Re: r298027 - [Serialization] Follow-up to r297972, deserialize name/loc in separate statements to make sure they deserialize in defined order.

2017-03-16 Thread Argyrios Kyrtzidis via cfe-commits
*b* at the gcc bots, those are the worst… :-P > On Mar 16, 2017, at 6:39 PM, Eric Christopher wrote: > > Also gcc bots fwiw :) > > -eric > > On Thu, Mar 16, 2017 at 6:01 PM Argyrios Kyrtzidis via cfe-commits > mailto:cfe-commits@lists.llvm.org>> wrote: > Author: akirtzidis > Date: Thu Mar

Re: r298027 - [Serialization] Follow-up to r297972, deserialize name/loc in separate statements to make sure they deserialize in defined order.

2017-03-16 Thread Eric Christopher via cfe-commits
;) On Thu, Mar 16, 2017 at 7:13 PM Argyrios Kyrtzidis wrote: > *b* at the gcc bots, those are the worst… :-P > > On Mar 16, 2017, at 6:39 PM, Eric Christopher wrote: > > Also gcc bots fwiw :) > > -eric > > On Thu, Mar 16, 2017 at 6:01 PM Argyrios Kyrtzidis via cfe-commits < > cfe-commits@li

Re: [PATCH] D30415: Fix -mno-altivec cannot overwrite -maltivec option

2017-03-16 Thread Hal Finkel via cfe-commits
On 03/16/2017 08:11 PM, Eric Christopher wrote: On Thu, Mar 16, 2017 at 5:45 PM Hal Finkel > wrote: On 03/16/2017 07:40 PM, Eric Christopher wrote: On Thu, Mar 16, 2017 at 5:37 PM Hal Finkel via Phabricator mailto:revi...@reviews.llvm.org>> wrote:

Re: [PATCH] D30415: Fix -mno-altivec cannot overwrite -maltivec option

2017-03-16 Thread Eric Christopher via cfe-commits
Great, thanks! Can you make this change instead Sean? :) -eric On Thu, Mar 16, 2017 at 7:20 PM Hal Finkel wrote: > > On 03/16/2017 08:11 PM, Eric Christopher wrote: > > > > On Thu, Mar 16, 2017 at 5:45 PM Hal Finkel wrote: > > > On 03/16/2017 07:40 PM, Eric Christopher wrote: > > > > On Thu, M

[PATCH] D30806: [nonnull] Teach Clang to attach the nonnull LLVM attribute to declarations and calls instead of just definitions, and then teach it to *not* attach such attributes even if the source c

2017-03-16 Thread Hal Finkel via Phabricator via cfe-commits
hfinkel added inline comments. Comment at: include/clang/AST/ASTContext.h:1868 + bool *OverrideNonnullReturn = nullptr, + unsigned *OverrideNonnullArgs = nullptr, unsigned *IntegerConstantArgs = nullptr)

[PATCH] D31069: Don't warn about an unreachable fallthrough annotation in a template function

2017-03-16 Thread Richard Smith via Phabricator via cfe-commits
rsmith added a comment. This needs a test case, but the change itself looks fine to me. Repository: rL LLVM https://reviews.llvm.org/D31069 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe

[PATCH] D31069: Don't warn about an unreachable fallthrough annotation in a template function

2017-03-16 Thread Ahmed Asadi via Phabricator via cfe-commits
ahmedasadi updated this revision to Diff 92104. ahmedasadi added a comment. Added a test case. https://reviews.llvm.org/D31069 Files: lib/Sema/AnalysisBasedWarnings.cpp test/SemaCXX/P30636.cpp Index: test/SemaCXX/P30636.cpp =

r298040 - [AVX-512] Add missing typecasts and parentheses to _mm512_mask_i64gather_ps. My macro cleanup script I used on the others last year must have missed it.

2017-03-16 Thread Craig Topper via cfe-commits
Author: ctopper Date: Fri Mar 17 00:14:37 2017 New Revision: 298040 URL: http://llvm.org/viewvc/llvm-project?rev=298040&view=rev Log: [AVX-512] Add missing typecasts and parentheses to _mm512_mask_i64gather_ps. My macro cleanup script I used on the others last year must have missed it. Modified:

[PATCH] D31069: Don't warn about an unreachable fallthrough annotation in a template function

2017-03-16 Thread Richard Smith via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. LGTM, do you need someone to commit for you? https://reviews.llvm.org/D31069 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists

r298041 - [AVX-512] Fix test cases that were using the builtins directly without typecasts instead of the intrinsic header.

2017-03-16 Thread Craig Topper via cfe-commits
Author: ctopper Date: Fri Mar 17 00:59:22 2017 New Revision: 298041 URL: http://llvm.org/viewvc/llvm-project?rev=298041&view=rev Log: [AVX-512] Fix test cases that were using the builtins directly without typecasts instead of the intrinsic header. Modified: cfe/trunk/test/CodeGen/avx512bw-bu

r298042 - [AVX-512] Change the input type for some load intrinsics to take void type like the spec (and the test cases say).

2017-03-16 Thread Craig Topper via cfe-commits
Author: ctopper Date: Fri Mar 17 00:59:25 2017 New Revision: 298042 URL: http://llvm.org/viewvc/llvm-project?rev=298042&view=rev Log: [AVX-512] Change the input type for some load intrinsics to take void type like the spec (and the test cases say). Modified: cfe/trunk/lib/Headers/avx512fintr

<    1   2