Re: r320162 - Revert "Unify implementation of our two different flavours of -Wtautological-compare."

2017-12-08 Thread Bill Seurer via cfe-commits
It also caused all the sanitizer builds to fail. For example: http://lab.llvm.org:8011/builders/sanitizer-ppc64le-linux/builds/3654 /home/buildbots/ppc64le-sanitizer/sanitizer-ppc64le/build/llvm/utils/TableGen/X86RecognizableInstr.cpp:789:21: error: comparison of constant -1 with expression of

Re: [clang-tools-extra] r291446 - [include-fixer] Load symbol index asynchronously.

2017-01-12 Thread Bill Seurer via cfe-commits
On 01/10/2017 01:53 PM, Benjamin Kramer wrote: I didn't manage to reproduce this. Does adding ${PTHREAD_LIB} to LINK_LIBS of tools/clang/tools/extra/include-fixer/plugin/CMakeLists.txt help? That does seem to make it work. On Tue, Jan 10, 2017 at 8:31 PM, Bill Seurer wrote: On 01/09/2017 09

Re: [clang-tools-extra] r291446 - [include-fixer] Load symbol index asynchronously.

2017-01-10 Thread Bill Seurer via cfe-commits
On 01/09/2017 09:18 AM, Benjamin Kramer via cfe-commits wrote: Author: d0k Date: Mon Jan 9 09:18:28 2017 New Revision: 291446 URL: http://llvm.org/viewvc/llvm-project?rev=291446&view=rev Log: [include-fixer] Load symbol index asynchronously. We don't actually need the index until parse time, s

[clang-tools-extra] r289581 - [powerpc] deactivate readability-identifier-naming.cpp test on powerpc64le

2016-12-13 Thread Bill Seurer via cfe-commits
Author: seurer Date: Tue Dec 13 14:26:35 2016 New Revision: 289581 URL: http://llvm.org/viewvc/llvm-project?rev=289581&view=rev Log: [powerpc] deactivate readability-identifier-naming.cpp test on powerpc64le The test case clang-tidy/readability-identifier-naming.cpp segfaults on powerpc64 little

Re: r283537 - Revert "[analyzer] Try to re-apply r283092 "Extend bug reports with extra notes"

2016-10-07 Thread Bill Seurer via cfe-commits
On 10/07/16 05:56, Artem Dergachev via cfe-commits wrote: > Author: dergachev > Date: Fri Oct 7 05:56:44 2016 > New Revision: 283537 > > URL: http://llvm.org/viewvc/llvm-project?rev=283537&view=rev One of these recent changes introduced a problem in sanitizer testing. /home/seurer/llvm/build/l

Re: [PATCH] D25326: [StaticAnalyser] Don't merge different returns in ExplodedGraph

2016-10-07 Thread Bill Seurer via cfe-commits
On 10/07/16 08:39, Daniel Marjamäki via cfe-commits wrote: danielmarjamaki removed rL LLVM as the repository for this revision. danielmarjamaki updated this revision to Diff 73926. danielmarjamaki added a comment. Refactoring. https://reviews.llvm.org/D25326 Files: include/clang/Analysis/Pr

Re: r259622 - Fix miscompile and rejects-valids when disambiguating after an ambiguous

2016-02-03 Thread Bill Seurer via cfe-commits
On 02/03/16 11:30, Renato Golin via cfe-commits wrote: On 3 February 2016 at 02:58, Richard Smith via cfe-commits wrote: Author: rsmith Date: Tue Feb 2 20:58:20 2016 New Revision: 259622 URL: http://llvm.org/viewvc/llvm-project?rev=259622&view=rev Log: Fix miscompile and rejects-valids when d

r253706 - Fix test case function name checks

2015-11-20 Thread Bill Seurer via cfe-commits
Author: seurer Date: Fri Nov 20 14:47:34 2015 New Revision: 253706 URL: http://llvm.org/viewvc/llvm-project?rev=253706&view=rev Log: Fix test case function name checks This is similar to the earlier fix I did, r253702, expect that here it is function names that are being searched for. If the fun