[PATCH] D81459: [SveEmitter] Add SVE tuple types and builtins for svundef.

This revision was automatically updated to reflect the committed changes. Closed by commit rG91a4a592edb0: [SveEmitter] Add SVE tuple types and builtins for svundef. (authored by sdesmalen). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81459/new/ h

[PATCH] D81721: [SVE] Ensure proper mangling of ACLE tuple types

This revision was automatically updated to reflect the committed changes. Closed by commit rG98100353d784: [SVE] Ensure proper mangling of ACLE tuple types (authored by sdesmalen). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81721/new/ https://rev

Re: [clang] 3e39760 - Revert "Return "[InstCombine] Simplify compare of Phi with constant inputs against a constant""

Reproducers? On Mon, Jun 15, 2020 at 9:47 AM Sam Parker via cfe-commits wrote: > > > Author: Sam Parker > Date: 2020-06-15T07:46:28+01:00 > New Revision: 3e39760f8eaad4770efa05824768e67237915cf5 > > URL: > https://github.com/llvm/llvm-project/commit/3e39760f8eaad4770efa05824768e67237915cf5 > DIF

[clang] 3e39760 - Revert "Return "[InstCombine] Simplify compare of Phi with constant inputs against a constant""

Author: Sam Parker Date: 2020-06-15T07:46:28+01:00 New Revision: 3e39760f8eaad4770efa05824768e67237915cf5 URL: https://github.com/llvm/llvm-project/commit/3e39760f8eaad4770efa05824768e67237915cf5 DIFF: https://github.com/llvm/llvm-project/commit/3e39760f8eaad4770efa05824768e67237915cf5.diff LO

[clang] 91a4a59 - [SveEmitter] Add SVE tuple types and builtins for svundef.

Author: Sander de Smalen Date: 2020-06-15T07:36:01+01:00 New Revision: 91a4a592edb01ca1968b5e5abe3da21709bb72d6 URL: https://github.com/llvm/llvm-project/commit/91a4a592edb01ca1968b5e5abe3da21709bb72d6 DIFF: https://github.com/llvm/llvm-project/commit/91a4a592edb01ca1968b5e5abe3da21709bb72d6.di

[clang] 9810035 - [SVE] Ensure proper mangling of ACLE tuple types

Author: Sander de Smalen Date: 2020-06-15T07:36:12+01:00 New Revision: 98100353d784e599fca502a34490603942f1930c URL: https://github.com/llvm/llvm-project/commit/98100353d784e599fca502a34490603942f1930c DIFF: https://github.com/llvm/llvm-project/commit/98100353d784e599fca502a34490603942f1930c.di

[PATCH] D81816: [PowerPC] Add support for vector bool __int128 for Power10

saghir created this revision. saghir added reviewers: PowerPC, hfinkel. saghir added projects: LLVM, PowerPC. Herald added subscribers: cfe-commits, shchenz, nemanjai. Herald added a project: clang. saghir added a parent revision: D80758: [PowerPC] Add -m[no-]power10-vector clang and llvm option.

[PATCH] D81543: [CodeGen][TLS] Set TLS Model for __tls_guard as well.

junparser added a comment. @rnk @aaron.ballman any comments? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81543/new/ https://reviews.llvm.org/D81543 ___ cfe-commits mailing list cfe-commits@lists.

[PATCH] D81420: Fix size for _ExtInt types with builtins

jtmott-intel added a comment. I don't have commit access. For whoever performs the commit, here's a (draft) commit message: [clang] Fix or emit diagnostic for checked arithmetic builtins with _ExtInt types - Fix computed size for _ExtInt types passed to checked arithmetic builtins. - E

[PATCH] D81336: [clang-tidy] simplify-bool-expr ignores template instantiations

lebedev.ri added a comment. In D81336#2091833 , @aaron.ballman wrote: > In D81336#2090708 , > @LegalizeAdulthood wrote: > > > Yeah, the restriction to the header file is a bug. It was a misconception > > on my p

[PATCH] D81552: [ASTMatchers] Added hasDirectBase and hasClass Matchers

aaron.ballman added reviewers: sammccall, dblaikie. aaron.ballman added subscribers: sammccall, dblaikie. aaron.ballman added a comment. Pinging @klimek , @sammccall , and @dblaikie to see if there are some opinions about overloading `hasName` (and possibly other naming related questions). ===

[PATCH] D81336: [clang-tidy] simplify-bool-expr ignores template instantiations

aaron.ballman added a comment. In D81336#2090708 , @LegalizeAdulthood wrote: > Yeah, the restriction to the header file is a bug. It was a misconception on > my part when I originally wrote the matcher. Excellent, thank you for weighing in! I'm fine w

[clang-tools-extra] 7fc533a - [clangd] Fix windows builds failing on check-clangd

Author: njames93 Date: 2020-06-14T13:29:17+01:00 New Revision: 7fc533a1d8de25e74c4829a9d868dbebd498492c URL: https://github.com/llvm/llvm-project/commit/7fc533a1d8de25e74c4829a9d868dbebd498492c DIFF: https://github.com/llvm/llvm-project/commit/7fc533a1d8de25e74c4829a9d868dbebd498492c.diff LOG:

[PATCH] D81786: [clang][utils] Modify make-ast-dump-check.sh to generate AST serialization dump tests

riccibruno added a reviewer: lebedev.ri. riccibruno added a comment. Summarising a quick discussion with @lebedev.ri on IRC yesterday: this is not necessarily the best way to test the serialization of AST nodes, in that we could instead perform some kind of structural equivalence test. However

[PATCH] D81407: [Analyzer][StreamChecker] Add note tags for file opening.

Szelethus added a comment. I'd still like to see more `NoteTag`s such as "File read failed, end-of-file indicator set on 'F'", and a final evaluation would be nice, but otherwise this checker looks amazing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm

[PATCH] D80699: [Analyzer][StreamChecker] Add check for pointer escape.

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. > @NoQ @baloghadamsoftware Escapes are more within your realm of expertise. > Anything to add? This looks like a fairly standard approach to take, no objections. Comment at: clan

[PATCH] D81407: [Analyzer][StreamChecker] Add note tags for file opening.

NoQ added a reviewer: NoQ. NoQ added a comment. Do i understand correctly that the checker is no longer "missing limbs" and we should consider turning it on by default? If so, @balazske could you prioritize hunting down the remaining false positives above adding new checks / hunting down false

[PATCH] D81315: [analyzer] Warning for default constructed unique pointer dereferences

NoQ added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/SmartPtrModeling.cpp:124 +return; + updateTrackedRegion(Call, C, ThisValRegion); +} NoQ wrote: > Not all constructors behave this way. In particular, if it's a copy/move > constructor

[PATCH] D81315: [analyzer] Warning for default constructed unique pointer dereferences

xazax.hun added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/SmartPtrModeling.cpp:35 bool isNullAfterMoveMethod(const CallEvent &Call) const; + BugType NullDereferenceBugType{this, "Null-smartPtr-deref", + "C++ smart pointer"}