[clang] dd50104 - [Clang][Bundler] Error reporting improvements

2019-10-25 Thread Sergey Dmitriev via cfe-commits
Author: Sergey Dmitriev Date: 2019-10-25T16:29:57-07:00 New Revision: dd501045cdea1c80b6788f0266d2a79f8b412eea URL: https://github.com/llvm/llvm-project/commit/dd501045cdea1c80b6788f0266d2a79f8b412eea DIFF: https://github.com/llvm/llvm-project/commit/dd501045cdea1c80b6788f0266d2a79f8b412eea.dif

[PATCH] D67031: [Clang][Bundler] Error reporting improvements

2019-10-25 Thread Sergey Dmitriev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGdd501045cdea: [Clang][Bundler] Error reporting improvements (authored by sdmitriev). Changed prior to commit: https://reviews.llvm.org/D67031?vs=224726&id=226521#toc Repository: rG LLVM Github Monore

[clang] baf91d0 - [NFC] Add a tablegen node for the root of the AST node hierarchies.

2019-10-25 Thread John McCall via cfe-commits
Author: John McCall Date: 2019-10-25T16:39:21-07:00 New Revision: baf91d02da6e68c4ee6723ef68911fcd80ece6a5 URL: https://github.com/llvm/llvm-project/commit/baf91d02da6e68c4ee6723ef68911fcd80ece6a5 DIFF: https://github.com/llvm/llvm-project/commit/baf91d02da6e68c4ee6723ef68911fcd80ece6a5.diff L

[PATCH] D68937: [clangd] Add parameter renaming to define-inline code action

2019-10-25 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. Build result: fail - 59631 tests passed, 1 failed and 805 were skipped. failed: LLVM.tools/llvm-ar/mri-utf8.test Log files: cmake-log.txt , ninja_check_all-log.txt

[PATCH] D69292: Proposal to add -Wtautological-compare to -Wall

2019-10-25 Thread Richard Trieu via Phabricator via cfe-commits
rtrieu added a comment. In D69292#1718415 , @thakis wrote: > Abstractly this lgtm. Do you have any data on true / false positive rates? Looking at the various warnings in -Wtautological-compare, only -Wtautological-overlap-compare and -Wtautological-bit

[clang] 93b29d3 - Revert "[Clang][Bundler] Error reporting improvements"

2019-10-25 Thread Sergey Dmitriev via cfe-commits
Author: Sergey Dmitriev Date: 2019-10-25T17:57:55-07:00 New Revision: 93b29d3882baf7df42e4e9bc26b977b00373ef56 URL: https://github.com/llvm/llvm-project/commit/93b29d3882baf7df42e4e9bc26b977b00373ef56 DIFF: https://github.com/llvm/llvm-project/commit/93b29d3882baf7df42e4e9bc26b977b00373ef56.dif

[PATCH] D68725: [analyzer] MemoryBlockRegion: Generalize AllocaRegion

2019-10-25 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. > This patch generalizes the `AllocaRegion` to store metadata about the > expression of the allocation of a memory block. Mmm, but you can easily obtain that expression from the existing `AllocaRegion`. I.e., you can obtain the expression on which the memory was allocated,

[clang] edb1a1d - Reland "[Clang][Bundler] Error reporting improvements"

2019-10-25 Thread Sergey Dmitriev via cfe-commits
Author: Sergey Dmitriev Date: 2019-10-25T19:00:06-07:00 New Revision: edb1a1de1b79c190ed7b76cf889a295c72e73729 URL: https://github.com/llvm/llvm-project/commit/edb1a1de1b79c190ed7b76cf889a295c72e73729 DIFF: https://github.com/llvm/llvm-project/commit/edb1a1de1b79c190ed7b76cf889a295c72e73729.dif

[PATCH] D69308: [analyzer] Test cases for the unsupported features for Clang Static Analyzer

2019-10-25 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. Thanks for the tests! Both of these features are relatively hard. Operator `new[]` requires invoking multiple (potentially unknown) amount of constructors with the same construct-expression. Apart from the technical difficulties of juggling program points around correctly

[PATCH] D69292: Proposal to add -Wtautological-compare to -Wall

2019-10-25 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. This is imho basic enough that it doesn't need a test. A test for this doesn't add any value that I can see. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69292/new/ https://reviews.llvm.org/D69292 ___ cfe-commits m

[PATCH] D67787: Add 8548 CPU definition and attributes

2019-10-25 Thread Justin Hibbits via Phabricator via cfe-commits
jhibbits updated this revision to Diff 226529. jhibbits added a comment. Make 8548 actually denote e500 LLVM target, add SPE checks to 8548 preprocessor test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67787/new/ https://reviews.llvm.org/D67787

<    1   2