Buildbot numbers for the week of 9/04/2016 - 9/10/201

2016-09-19 Thread Galina Kistanova via cfe-commits
Hello everyone, Below are some buildbot numbers for the week of 9/04/2016 - 9/10/2016. Please see the same data in attached csv files: The longest time each builder was red during the last week; "Status change ratio" by active builder (percent of builds that changed the builder status from greed

Buildbot numbers for the week of 9/11/2016 - 9/17/2016

2016-09-19 Thread Galina Kistanova via cfe-commits
Hello everyone, Below are some buildbot numbers for the last week of 9/11/2016 - 9/17/2016. Please see the same data in attached csv files: The longest time each builder was red during the last week; "Status change ratio" by active builder (percent of builds that changed the builder status from

Re: Submodule semantics with macro guard

2016-09-19 Thread Manman via cfe-commits
> On Sep 19, 2016, at 5:55 PM, Richard Smith via cfe-commits > wrote: > > Your c.h is not correct. It would introduce a definition of c in every file > where it's included, so it's not a modular header. Hi Richard, What do you mean by c.h is not correct? It is guarded by a macro, so if we ar

[PATCH] D24751: [cleanup] Remove excessive padding from TextTokenRetokenizer::Position

2016-09-19 Thread Alexander Shaposhnikov via cfe-commits
alexshap created this revision. alexshap added reviewers: gribozavr, craig.topper. alexshap added a subscriber: cfe-commits. alexshap set the repository for this revision to rL LLVM. alexshap changed the visibility of this Differential Revision from "Public (No Login Required)" to "All Users". Re

Re: [PATCH] D24751: [cleanup] Remove excessive padding from TextTokenRetokenizer::Position

2016-09-19 Thread Alexander Shaposhnikov via cfe-commits
alexshap added a comment. F2434937: Screen Shot 2016-09-19 at 7.33.38 PM.png the order BufferStart, BufferEnd,BufferPtr,BufferStartLoc,CurToken works as well Repository: rL LLVM https://reviews.llvm.org/D24751 ___

Re: [PATCH] D24751: [cleanup] Remove excessive padding from TextTokenRetokenizer::Position

2016-09-19 Thread David Majnemer via cfe-commits
majnemer added a subscriber: majnemer. majnemer accepted this revision. majnemer added a reviewer: majnemer. majnemer added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM https://reviews.llvm.org/D24751 ___ c

Re: r281277 - [Sema] Fix PR30346: relax __builtin_object_size checks.

2016-09-19 Thread George Burgess IV via cfe-commits
WFM; I'll put together a patch that only allows this under -fno-strict-aliasing. I'm entirely unfamiliar with struct-path-tbaa, so Hal, do you see a reason why struct-path-tbaa wouldn't play nicely with flexible arrays at the end of types? Glancing at it, I don't think it should cause problems, bu

Re: [PATCH] D24609: [ARM] Add missing Interlocked intrinsics

2016-09-19 Thread Saleem Abdulrasool via cfe-commits
compnerd accepted this revision. compnerd added a comment. This revision is now accepted and ready to land. That sounds reasonable. https://reviews.llvm.org/D24609 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin

[PATCH] D24752: [Modules] Add missing dependencies to clang builtins modulemap

2016-09-19 Thread Elad Cohen via cfe-commits
eladcohen created this revision. eladcohen added a reviewer: rsmith. eladcohen added a subscriber: cfe-commits. X86 intrinsic header files mm3dnow.h and wmmintrin.h include and depend on mmintrin.h and emmintrin.h respectively. This patch adds these missing dependencies to the corresponding submo

Re: [clang-tools-extra] r281954 - clang-change-namespace: Update libdeps.

2016-09-19 Thread Eric Liu via cfe-commits
Thanks for the fix! On Tue, Sep 20, 2016, 02:53 NAKAMURA Takumi via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: chapuni > Date: Mon Sep 19 19:44:45 2016 > New Revision: 281954 > > URL: http://llvm.org/viewvc/llvm-project?rev=281954&view=rev > Log: > clang-change-namespace: Update l

Re: Submodule semantics with macro guard

2016-09-19 Thread Richard Smith via cfe-commits
On 19 Sep 2016 6:53 pm, "Manman via cfe-commits" wrote: On Sep 19, 2016, at 5:55 PM, Richard Smith via cfe-commits < cfe-commits@lists.llvm.org> wrote: Your c.h is not correct. It would introduce a definition of c in every file where it's included, so it's not a modular header. Hi Richard, W

<    1   2