[3.6 Release] Time to fix the release notes

From: "Hans Wennborg" <hans@chromium.org>
To: "george burgess iv" <george.burgess.iv@gmail.com>, "Nick Lewycky" <nicholas@mxc.ca>, "Daniel Berlin"
<dannyb@google.com>
Cc: "llvm cfe" <cfe-commits@cs.uiuc.edu>, "llvmdev" <llvmdev@cs.uiuc.edu>
Sent: Wednesday, February 18, 2015 6:39:17 PM
Subject: Re: [3.6 Release] Time to fix the release notes

> The 3.6 release is drawing nearer, but the release notes could
> still
> use a lot of improvement:
>
> LLVM:
> https://github.com/llvm-mirror/llvm/blob/release_36/docs/ReleaseNotes.rst
> Clang:
> https://github.com/llvm-mirror/clang/blob/release_36/docs/ReleaseNotes.rst
>
> The good news is that they can be updated all the way up until
> release. If you have a patch for the release notes, commit it
> directly
> to the branch, or send it to me and I'll commit it for you. It
> doesn't
> even have to be a patch proper, just send me some text!
>
> I've been going through Alex's excellent weekly news
> (http://llvmweekly.org) to figure out what changed.
>
> Here's a list of things that might be worth mentioning in the
> notes:
>
> LLVM:
> - the new CFL alias analysis

Is this hooked up by default? Should we mention it in the notes?

No, not yet. There are still bugs that are being worked out.

-Hal

You can request that CFLAA is run in opt/clang by passing in a few flags, but it’s not on by default, and there are known correctness issues that we’re currently working through.

Unless anyone feels strongly otherwise, I think it would be best to not add CFLAA until we’re highly confident that it doesn’t give incorrect results. Preferably it would be enabled by default (and/or shown to Make Things Faster™) at the time of mention as well, but correctness comes first. :slight_smile:

George

You can request that CFLAA is run in opt/clang by passing in a few flags, but it’s not on by default, and there are known correctness issues that we’re currently working through.

Unless anyone feels strongly otherwise, I think it would be best to not add CFLAA until we’re highly confident that it doesn’t give incorrect results. Preferably it would be enabled by default (and/or shown to Make Things Faster™) at the time of mention as well, but correctness comes first. :slight_smile:

Sounds good to me. Thanks!

Clang:
- fpic and -fPIC are now different

Justin, Rafael: should we mention this in the release notes?

Probably. What do you guys think of the attached patch?

Cheers,
Rafael

t.patch (363 Bytes)

lgtm

Thanks,
Hans

Looks good to me too.

- Justin

I think we're all good here. I've added some notes about C11 changes in
r229939.

C++14 support used to be feature-complete in Clang 3.4. It still is, but it
used to too. =)