[clang] d4758d4 - Fix a spelling mistake in a couple of intrinsic description comments. NFC

2019-10-27 Thread Greg Bedwell via cfe-commits
Author: Greg Bedwell Date: 2019-10-27T09:42:14Z New Revision: d4758d4a8d842275f4319ae278808be0bcd2ede3 URL: https://github.com/llvm/llvm-project/commit/d4758d4a8d842275f4319ae278808be0bcd2ede3 DIFF: https://github.com/llvm/llvm-project/commit/d4758d4a8d842275f4319ae278808be0bcd2ede3.diff LOG:

Re: r351316 - Reapply [Tooling] Make clang-tool find libc++ dir on mac when running on a file without compilation database.

2019-01-16 Thread Greg Bedwell via cfe-commits
> > Unfortunately this trips up a variety of buildbots: > Platform REQUIRES added in r351360, please do revert & fix if this test is supposed to work elsewhere. Looks like the test fail just made it into the release-8.0.0 branch by a few commits. I think we'll need Jeremy's fix r351360 or/and any

Re: [llvm-dev] LLVM buildmaster is back to work now but two builders remain OFF

2017-11-29 Thread Greg Bedwell via cfe-commits
Hopefully these two should be good to re-enable as of r319330. -Greg On 29 November 2017 at 16:23, Greg Bedwell wrote: > > Two slaves remain off for now as they produce a lot of warnings and > their log files are way too big. > > I think https://reviews.llvm.org/D40603 will fix this issue. > >

Re: [llvm-dev] LLVM buildmaster is back to work now but two builders remain OFF

2017-11-29 Thread Greg Bedwell via cfe-commits
> Two slaves remain off for now as they produce a lot of warnings and their log files are way too big. I think https://reviews.llvm.org/D40603 will fix this issue. -Greg On 28 November 2017 at 22:13, Galina Kistanova via llvm-dev < llvm-...@lists.llvm.org> wrote: > Hello everyone, > > LLVM bui

Re: r300001 - Revert r298824 & r298816, recommit r298742 & r298754

2017-04-12 Thread Greg Bedwell via cfe-commits
We're seeing the same assertion failing in our internal testing (I've not bisected to a specific change yet though, but it seems likely it's the same thing as in the log from the modules bot). Here's a reduced example that triggers it: $ cat 2.h // tem

Re: [PATCH] D17550: Adding doxygen comments to the LLVM intrinsics (part 6, popcntintrin.h)

2016-02-23 Thread Greg Bedwell via cfe-commits
gbedwell added a subscriber: gbedwell. gbedwell added a comment. In http://reviews.llvm.org/D17550#360177, @probinson wrote: > One question I have, which shouldn't block this (as we've done several like > this already): > Is is okay to be using C++ style comments in these headers? > (Is there

Re: [PATCH] D14583: Report Windows error code in a fatal error after a system call

2015-11-11 Thread Greg Bedwell via cfe-commits
gbedwell added a subscriber: gbedwell. gbedwell added a comment. > > Also, I am really curious to know why CryptAcquireContext fails in those > > cases! > > > Me too. Once is a fluke, twice is a trend. Probably some obscure > system-resource problem but it would be nice to be sure. I'm

Re: r248546 - This patch adds missing pieces to clang, including the PS4 toolchain

2015-09-25 Thread Greg Bedwell via cfe-commits
Temporarily reverted in r248578 until we have implemented a solution for the failure on the llvm-clang-lld-x86_64-scei-ps4-ubuntu-fast bot. On 24 September 2015 at 23:06, Ekaterina Romanova via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: kromanova > Date: Thu Sep 24 17:06:52 2015 >

r248578 - Revert "This patch adds missing pieces to clang, including the PS4 toolchain definition, added warnings, PS4 defaults, and Driver changes needed for our compiler."

2015-09-25 Thread Greg Bedwell via cfe-commits
Author: gbedwell Date: Fri Sep 25 11:11:00 2015 New Revision: 248578 URL: http://llvm.org/viewvc/llvm-project?rev=248578&view=rev Log: Revert "This patch adds missing pieces to clang, including the PS4 toolchain definition, added warnings, PS4 defaults, and Driver changes needed for our compiler

Re: [PATCH] D12368: Add a clang release note about raising the minimum Windows version for the next major release

2015-08-27 Thread Greg Bedwell via cfe-commits
gbedwell added a comment. In http://reviews.llvm.org/D12368#233680, @hans wrote: > Committed in r246090. Thanks! Repository: rL LLVM http://reviews.llvm.org/D12368 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/c

[PATCH] D12368: Add a clang release note about raising the minimum Windows version for the next major release

2015-08-26 Thread Greg Bedwell via cfe-commits
gbedwell created this revision. gbedwell added reviewers: hans, rnk, aaron.ballman. gbedwell added a subscriber: cfe-commits. This is a clang release note to match the LLVM release note in D12367. As we're planning on raising the minimum Windows version to Windows 7 for LLVM 3.8 as per this disc