[PATCH] D64239: [AArch64] Fix vsqadd scalar intrinsics operands

john.brawn accepted this revision. john.brawn added a comment. This revision is now accepted and ready to land. LGTM, with a couple of nitpicks. > The existing unsigned argument can cause faulty code as float to unsigned > conversion is undefined, > which llvm/clang optimizes away. It's specif

[PATCH] D64242: [AArch64] Fix scalar vuqadd intrinsics operands

john.brawn accepted this revision. john.brawn added a comment. This revision is now accepted and ready to land. LGTM, with one nitpick. Comment at: test/CodeGen/aarch64-neon-vuqadd-float-conversion-warning.c:2 +// RUN: %clang_cc1 -triple arm64-none-linux-gnu -target-feature +ne

[PATCH] D62152: [ARM][AArch64] Fix incorrect handling of alignment in va_arg code generation

john.brawn created this revision. john.brawn added reviewers: chill, efriedma, t.p.northover, rjmccall. Herald added subscribers: kristof.beyls, javed.absar. Herald added a project: clang. Overaligned and underaligned types (i.e. types where the alignment has been increased or decreased using the

[PATCH] D62152: [ARM][AArch64] Fix incorrect handling of alignment in va_arg code generation

john.brawn added a comment. In D62152#1508979 , @efriedma wrote: > Please verify my understanding of the following is correct: > > 1. getTypeUnadjustedAlign() is currently only used to compute calling > convention alignment for ARM and AArch64. Yes, the

[PATCH] D62152: [ARM][AArch64] Fix incorrect handling of alignment in va_arg code generation

This revision was automatically updated to reflect the committed changes. Closed by commit rC361372: [ARM][AArch64] Fix incorrect handling of alignment in va_arg code generation (authored by john.brawn, committed by ). Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[PATCH] D46050: [Frontend] Avoid running plugins during code completion parse

john.brawn added a comment. I know very little about how code completion works, but it's not immediately obvious to me that disabling plugin ast consumers when code completion is enabled is necessarily correct. In what kind of scenario would we both have a plugin loaded that wants to insert an

[PATCH] D80295: [Sema] Diagnose static data members in classes nested in unnamed classes

john.brawn created this revision. john.brawn added reviewers: rsmith, rjmccall. Herald added a project: clang. We currently diagnose static data members directly contained in unnamed classes, but we should also diagnose when they're in a class that is nested (directly or indirectly) in an unname

[PATCH] D80295: [Sema] Diagnose static data members in classes nested in unnamed classes

john.brawn updated this revision to Diff 265279. john.brawn added a comment. Adjusted to not give multiple errors when more than one error criteria is met. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80295/new/ https://reviews.llvm.org/D80295 Files: clang/lib/Sema/SemaDecl.cpp cl

[PATCH] D80295: [Sema] Diagnose static data members in classes nested in unnamed classes

john.brawn updated this revision to Diff 265486. john.brawn marked an inline comment as done. john.brawn added a comment. Use the tag of the anonymous struct when emitting a diagnostic. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80295/new/ https://reviews.llvm.org/D80295 Files: cl

[PATCH] D80295: [Sema] Diagnose static data members in classes nested in unnamed classes

john.brawn added inline comments. Comment at: clang/lib/Sema/SemaDecl.cpp:6904 << Name << RD->getTagKind(); Invalid = true; +} else if (RD->isLocalClass()) { rjmccall wrote: > This diagnostic actually ignores the tag kind that pass

[PATCH] D80295: [Sema] Diagnose more cases of static data members in local or unnamed classes

john.brawn marked an inline comment as done. john.brawn added inline comments. Comment at: clang/lib/Sema/SemaDecl.cpp:6904 << Name << RD->getTagKind(); Invalid = true; +} else if (RD->isLocalClass()) { rjmccall wrote: > john.brawn

[PATCH] D80295: [Sema] Diagnose more cases of static data members in local or unnamed classes

john.brawn updated this revision to Diff 265779. john.brawn retitled this revision from "[Sema] Diagnose static data members in classes nested in unnamed classes" to "[Sema] Diagnose more cases of static data members in local or unnamed classes". john.brawn edited the summary of this revision. jo

[PATCH] D80295: [Sema] Diagnose more cases of static data members in local or unnamed classes

This revision was automatically updated to reflect the committed changes. Closed by commit rG6c906f7785da: [Sema] Diagnose more cases of static data members in local or unnamed classes (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llv

[PATCH] D71775: [ThreadPool] On Windows, extend usage to all CPU sockets and all NUMA groups

john.brawn added a comment. Herald added subscribers: frgossen, grosul1. Herald added a reviewer: MaskRay. With this patch the Threading.PhysicalConcurrency unit test fails when run with an affinity less than the number of physical cpus. I've raised https://bugs.llvm.org/show_bug.cgi?id=45556.

[PATCH] D81672: [Driver] When forcing a crash call abort to get the correct diagnostic

john.brawn created this revision. john.brawn added reviewers: gbreynoo, jhenderson, probinson. Herald added a project: clang. Commit a945037e8fd0c30e250a62211469eea6765a36ae moved the printing of the "PLEASE submit a bug repor

[PATCH] D81672: [Driver] When forcing a crash call abort to get the correct diagnostic

john.brawn added reviewers: aganea, rnk. john.brawn added a comment. Added a couple of reviewers that have recently worked on CrashRecoveryContext. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81672/new/ https://reviews.llvm.org/D81672

[PATCH] D81672: [Driver] When forcing a crash call abort to get the correct diagnostic

john.brawn marked an inline comment as done. john.brawn added inline comments. Comment at: clang/tools/driver/driver.cpp:518 + CRC.DumpStackAndCleanupOnFailure = true; + CRC.RunSafely([&]() { abort(); }); } aganea wrote: > The only concern I have i

[PATCH] D81672: [Driver] When forcing a crash call abort to get the correct diagnostic

john.brawn updated this revision to Diff 271757. john.brawn edited the summary of this revision. john.brawn added a comment. Herald added subscribers: llvm-commits, hiraditya. Herald added a project: LLVM. Moved BugReportMsg printing into CrashRecoveryContext, and stopped printing of backtrace wh

[PATCH] D81672: [Driver] When forcing a crash print the bug report message

john.brawn updated this revision to Diff 272471. john.brawn retitled this revision from "[Driver] When forcing a crash call abort to get the correct diagnostic" to "[Driver] When forcing a crash print the bug report message". john.brawn edited the summary of this revision. john.brawn added a comm

[PATCH] D81672: [Driver] When forcing a crash print the bug report message

john.brawn marked an inline comment as done. john.brawn added inline comments. Comment at: llvm/lib/Support/PrettyStackTrace.cpp:36 +static const char *BugReportMsg = +"PLEASE submit a bug report to " BUG_REPORT_URL MaskRay wrote: > This variable is mutable

[PATCH] D81672: [Driver] When forcing a crash print the bug report message

john.brawn marked an inline comment as done. john.brawn added inline comments. Comment at: clang/tools/driver/driver.cpp:515 + + llvm::dbgs() << llvm::getBugReportMsg(); } MaskRay wrote: > Why ` llvm::dbgs() << llvm::getBugReportMsg();` when -gen-reprod

[PATCH] D81672: [Driver] When forcing a crash print the bug report message

john.brawn updated this revision to Diff 273816. john.brawn edited the summary of this revision. john.brawn added a comment. Don't print the diagnostic with -gen-reproducer. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81672/new/ https://reviews.llvm.org/D81672 Files: clang/test/Dri

[PATCH] D81672: [Driver] When forcing a crash print the bug report message

john.brawn marked an inline comment as done. john.brawn added inline comments. Comment at: clang/tools/driver/driver.cpp:515 + + llvm::dbgs() << llvm::getBugReportMsg(); } MaskRay wrote: > john.brawn wrote: > > MaskRay wrote: > > > Why ` llvm::dbgs() <<

[PATCH] D81672: [Driver] When forcing a crash print the bug report message

This revision was automatically updated to reflect the committed changes. Closed by commit rGce1fa201af77: [Driver] When forcing a crash print the bug report message (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81672/new/

[PATCH] D129231: [Builtins] Do not claim all libfuncs are readnone with trapping math.

john.brawn added a comment. Looking at the descriptions of maths functions in C99 (and I expect C11 will be the same) it looks like there are three kinds: - Those that can report error by errno and floating-point exeption, and may also raise the inexact exception - Those that don't set errno, b

[PATCH] D115620: [AArch64] Lowering and legalization of strict FP16

john.brawn updated this revision to Diff 419494. john.brawn added a comment. Adjusted formatting slightly. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115620/new/ https://reviews.llvm.org/D115620 Files: clang/test/CodeGen/aarch64-v8.2a-fp16-intrinsics-constrained.c llvm/lib/CodeG

[PATCH] D129231: [Builtins] Do not claim all libfuncs are readnone with trapping math.

john.brawn accepted this revision. john.brawn added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129231/new/ https://reviews.llvm.org/D129231 ___

[PATCH] D129231: [Builtins] Do not claim all libfuncs are readnone with trapping math.

john.brawn added inline comments. Comment at: clang/include/clang/Basic/Builtins.def:1409 -LIBBUILTIN(round, "dd", "fnc", "math.h", ALL_LANGUAGES) -LIBBUILTIN(roundf, "ff", "fnc", "math.h", ALL_LANGUAGES) -LIBBUILTIN(roundl, "LdLd", "fnc", "math.h", ALL_LANGUAGES) +LIBBUILTIN(r

[PATCH] D118259: [AArch64] Adjust aarch64-neon-intrinsics-constrained test and un-XFAIL

john.brawn added a comment. Ping. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118259/new/ https://reviews.llvm.org/D118259 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org

[PATCH] D115620: [AArch64] Lowering and legalization of strict FP16

john.brawn updated this revision to Diff 416141. john.brawn set the repository for this revision to rG LLVM Github Monorepo. john.brawn added a comment. Herald added a project: clang. Herald added a subscriber: cfe-commits. Rebase and ping. Repository: rG LLVM Github Monorepo CHANGES SINCE LA

[PATCH] D118259: [AArch64] Adjust aarch64-neon-intrinsics-constrained test and un-XFAIL

john.brawn added a comment. llvm/test/CodeGen/AArch64/fp-intrinsics-vector.ll is testing the asm generation, so it probably is fine to remove that here so I will. This test is specifically testing arm_neon.h intrinsics though, so wouldn't make sense unifying with other architectures. Reposito

[PATCH] D115620: [AArch64] Lowering and legalization of strict FP16

john.brawn added a comment. Ping. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115620/new/ https://reviews.llvm.org/D115620 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D115620: [AArch64] Lowering and legalization of strict FP16

john.brawn updated this revision to Diff 422257. john.brawn added a comment. Removed clang test changes. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115620/new/ https://reviews.llvm.org/D115620 Files: llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp llvm/lib/Target/AArch64/AArch64IS

[PATCH] D115620: [AArch64] Lowering and legalization of strict FP16

john.brawn added inline comments. Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:1386 + + IsStrictFPEnabled = true; } dmgreen wrote: > What are the ramifications of setting this to true? The behaviour with IsStrictFPEnabled = false is that strict f

[PATCH] D118259: [AArch64] Adjust aarch64 constrained intrinsics tests and un-XFAIL

john.brawn updated this revision to Diff 422285. john.brawn retitled this revision from "[AArch64] Adjust aarch64-neon-intrinsics-constrained test and un-XFAIL" to "[AArch64] Adjust aarch64 constrained intrinsics tests and un-XFAIL". john.brawn edited the summary of this revision. john.brawn adde

[PATCH] D115620: [AArch64] Lowering and legalization of strict FP16

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG12c1022679d4: [AArch64] Lowering and legalization of strict FP16 (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D118259: [AArch64] Adjust aarch64 constrained intrinsics tests and un-XFAIL

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1b1466c34669: [AArch64] Adjust aarch64 constrained intrinsics tests and un-XFAIL (authored by john.brawn). Repository: rG LLVM Github Monorepo CH

[PATCH] D118257: [AArch64] Generate fcmps when appropriate for neon intrinsics

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGbca998ed3c9a: [AArch64] Generate fcmps when appropriate for neon intrinsics (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D117795: [AArch64] Add some missing strict FP vector lowering

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG8e17c9613f36: [AArch64] Add some missing strict FP vector lowering (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES SINCE LA

[PATCH] D124255: [libc++] Make check_assertion.h use setjmp/longjmp instead of fork

john.brawn created this revision. john.brawn added reviewers: ldionne, Mordante, michaelplatings. Herald added a project: All. john.brawn requested review of this revision. Herald added a project: libc++. Herald added a subscriber: libcxx-commits. Herald added a reviewer: libc++. Currently check_a

[PATCH] D125775: [ARM] Don't Enable AES Pass for Generic Cores

john.brawn accepted this revision. john.brawn added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125775/new/ https://reviews.llvm.org/D125775 __

[PATCH] D86091: [cmake] Fix build of attribute plugin example on Windows

john.brawn accepted this revision. john.brawn added a comment. This revision is now accepted and ready to land. LGTM (looks like the other example plugins just use PRIVATE as well). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86091/new/ https://r

[PATCH] D88546: [ARM] Update NEON testcase with missing target string in

john.brawn accepted this revision. john.brawn added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88546/new/ https://reviews.llvm.org/D88546

[PATCH] D89573: [Driver] Incorporate -mfloat-abi in the computed triple on ARM

john.brawn created this revision. john.brawn added reviewers: efriedma, vhscampos, chill. Herald added subscribers: dexonsmith, steven_wu, hiraditya, kristof.beyls. Herald added a project: clang. john.brawn requested review of this revision. LLVM assumes that when it creates a call to a C library

[PATCH] D89573: [Driver] Incorporate -mfloat-abi in the computed triple on ARM

john.brawn updated this revision to Diff 299420. john.brawn edited the summary of this revision. john.brawn added a comment. Adjusted to give an error when the specified float abi is incompatible with the default. Was a bit trickier than I expected as it required extracting out the logic to dete

[PATCH] D89573: [Driver] Incorporate -mfloat-abi in the computed triple on ARM

This revision was automatically updated to reflect the committed changes. Closed by commit rG0c66606230df: [Driver] Incorporate -mfloat-abi in the computed triple on ARM (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89573/ne

[PATCH] D88645: [Annotation] Allows annotation to carry some additional constant arguments.

john.brawn added a comment. This also causes the AnnotateFunctions example to no longer build: /home/jb/work/llvm-project/clang/examples/AnnotateFunctions/AnnotateFunctions.cpp: In member function ‘virtual bool {anonymous}::AnnotateFunctionsConsumer::HandleTopLevelDecl(clang::DeclGroupRef)’:

[PATCH] D110065: [AArch64] Add support for the 'R' architecture profile.

john.brawn added a comment. In D110065#3087330 , @labrinea wrote: > Added v8.1a_ops on AppleA10. However, the target parser lists it as v8.0a, > which seems odd. Out of the scope of this patch anyway. Looking at the definition of A10 in target parser it

[PATCH] D110065: [AArch64] Add support for the 'R' architecture profile.

john.brawn accepted this revision. john.brawn added a comment. LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110065/new/ https://reviews.llvm.org/D110065 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi

[PATCH] D138143: [FPEnv] Enable strict fp for AArch64 in clang

john.brawn created this revision. john.brawn added reviewers: t.p.northover, fhahn, umesh.kalappa0. Herald added subscribers: arphaman, kristof.beyls. Herald added a project: All. john.brawn requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. T

[PATCH] D137980: [ARM] Pretend atomics are always lock-free for small widths.

john.brawn added a comment. Looking at GCC it looks like there (for cortex-m0 at least) the behaviour is that loads and stores are generated inline, but more complex operations go to the atomic library calls (not the sync library calls). e.g. for int x, y; int fn() { return __atomic_loa

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn added inline comments. Comment at: clang/lib/Sema/SemaLookup.cpp:542 +N = Decls.size(); + } + rjmccall wrote: > dexonsmith wrote: > > john.brawn wrote: > > > rjmccall wrote: > > > > john.brawn wrote: > > > > > rjmccall wrote: > > > > > > This is g

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn updated this revision to Diff 542893. john.brawn edited the summary of this revision. john.brawn added a comment. Restructured to check for hidden tags in the main loop. Also add a bunch of extra tests. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154503/new/ https://review

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGdfca88341794: [Sema] Fix handling of functions that hide classes (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn added a comment. The first version of this that I committed caused a failure in clang/test/Modules/stress1.cpp so I reverted it. I've now committed a new version that handles the removal of existing decl when isPreferredLookupResult is true in a slightly different way, which should

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn added a comment. I've committed a test for the behaviour when we have invalid decls in https://reviews.llvm.org/rG6244e3840694. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154503/new/ https://reviews.llvm.org/D154503

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn added a comment. I've committed an updated version of this that checks for invalid decls when deciding if a decl has been hidden. This stops the assertion failure happening. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154503/new/ http

[PATCH] D154502: [AST] Fix bug in UnresolvedSet::erase of last element

john.brawn created this revision. john.brawn added reviewers: dexonsmith, rsmith, rjmccall. Herald added a project: All. john.brawn requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. UnresolvedSet::erase works by popping the last element then

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn created this revision. john.brawn added reviewers: dexonsmith, rsmith, rjmccall. Herald added a project: All. john.brawn requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. When a function is declared in the same scope as a class wit

[PATCH] D154502: [AST] Fix bug in UnresolvedSet::erase of last element

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG4ade8b7ed997: [AST] Fix bug in UnresolvedSet::erase of last element (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES SINCE L

[PATCH] D154502: [AST] Fix bug in UnresolvedSet::erase of last element

john.brawn added a comment. The test I added here caused failures in buildbots that build with -Wall -Werror, fixed in https://reviews.llvm.org/rG258322105892. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154502/new/ https://reviews.llvm.org/D154

[PATCH] D154502: [AST] Fix bug in UnresolvedSet::erase of last element

john.brawn added a comment. Unfortunately my fix then broke builds using MSVC. Hopefully fixed for real in https://reviews.llvm.org/rG25784cd6a962. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154502/new/ https://reviews.llvm.org/D154502 ___

[PATCH] D154502: [AST] Fix bug in UnresolvedSet::erase of last element

john.brawn added inline comments. Comment at: clang/unittests/AST/UnresolvedSetTest.cpp:11 +}; +} // namespace clang + jroelofs wrote: > jroelofs wrote: > > This ODR violation broke the build for me because it's picking up the > > definition from Decl.h instead

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn updated this revision to Diff 537721. john.brawn added a comment. Avoid doing work when we don't have both decls that can hide and decls that can be hidden. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154503/new/ https://reviews.llvm.org/D154503 Files: clang/lib/Sema/Se

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn added inline comments. Comment at: clang/lib/Sema/SemaLookup.cpp:542 +N = Decls.size(); + } + rjmccall wrote: > This is going to fire on every single ordinary lookup that finds multiple > declarations, right? I haven't fully internalized the iss

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn updated this revision to Diff 537731. john.brawn added a comment. Same patch as previous, but with full context. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154503/new/ https://reviews.llvm.org/D154503 Files: clang/lib/Sema/SemaLookup.cpp clang/test/SemaCXX/using-hidin

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn added inline comments. Comment at: clang/lib/Sema/SemaLookup.cpp:507 + // C++ [basic.scope.hiding]p2: + // A class name or enumeration name can be hidden by the name of shafik wrote: > This section does not exist anymore, it was replaced in > [p

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn added inline comments. Comment at: clang/lib/Sema/SemaLookup.cpp:542 +N = Decls.size(); + } + rjmccall wrote: > john.brawn wrote: > > rjmccall wrote: > > > This is going to fire on every single ordinary lookup that finds multiple > > > declaratio

[PATCH] D154503: [Sema] Fix handling of functions that hide classes

john.brawn updated this revision to Diff 539142. john.brawn added a comment. Use BitVector, change how Decls are deleted, adjust test to avoid potentially-conflicting using-declarations. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154503/new/ https://reviews.llvm.org/D154503 Files:

[PATCH] D138143: [FPEnv] Enable strict fp for AArch64 in clang

john.brawn added inline comments. Comment at: clang/docs/ReleaseNotes.rst:760 +- Strict floating point has been enabled for AArch64, which means that + ``-ftrapping-math``, ``-frounding-math``, ``-ffp-model=strict``, and + ``-ffp-exception-behaviour=`` are now accepted. ---

[PATCH] D138143: [FPEnv] Enable strict fp for AArch64 in clang

This revision was automatically updated to reflect the committed changes. Closed by commit rG9e3264ab2021: [FPEnv] Enable strict fp for AArch64 in clang (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138143/new/ https://revi

[PATCH] D138679: [clang][CodeGen] Add default attributes to __clang_call_terminate

john.brawn created this revision. john.brawn added reviewers: rsmith, rjmccall, lebedev.ri, miyuki. Herald added a project: All. john.brawn requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. When generating __clang_call_terminate use SetLLVMFu

[PATCH] D138679: [clang][CodeGen] Add default attributes to __clang_call_terminate

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6b8900f7f91d: [clang][CodeGen] Add default attributes to __clang_call_terminate (authored by john.brawn). Changed prior to commit: https://reviews

[PATCH] D31342: Add ParsedAttrInfo::handleDeclAttribute

This revision was automatically updated to reflect the committed changes. Closed by commit rGfa0320dd8d5a: Add ParsedAttrInfo::handleDeclAttribute (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D31342/new/ https://reviews.llv

[PATCH] D76704: Don't normalise CXX11/C2X attribute names to start with ::

john.brawn created this revision. john.brawn added reviewers: erichkeane, aaron.ballman, rjmccall. Herald added a project: clang. john.brawn added a child revision: D31343: Add an attribute plugin example. Currently square-bracket-style (CXX11/C2X) attribute names are normalised to start with ::

[PATCH] D31343: Add an attribute plugin example

john.brawn updated this revision to Diff 252309. john.brawn added a comment. A few small changes: - Don't start C++11 spelling with :: (changes to name normalisation to allow this are in D76704 ) - Don't check that the Decl is null in diagAppertainsToDecl - Retur

[PATCH] D76704: Don't normalise CXX11/C2X attribute names to start with ::

This revision was automatically updated to reflect the committed changes. Closed by commit rGbc3f171090f6: Don't normalise CXX11/C2X attribute names to start with :: (authored by john.brawn). Changed prior to commit: https://reviews.llvm.org/D76704?vs=252308&id=252578#toc Repository: rG LLVM

[PATCH] D31343: Add an attribute plugin example

This revision was automatically updated to reflect the committed changes. Closed by commit rG3f03c12a51be: Add an attribute plugin example (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D31343/new/ https://reviews.llvm.org/D3

[PATCH] D31343: Add an attribute plugin example

john.brawn added a comment. Release note added: https://reviews.llvm.org/rG0cff81cff05d Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D31343/new/ https://reviews.llvm.org/D31343 ___ cfe-commits mailing

[PATCH] D31337: Use virtual functions in ParsedAttrInfo instead of function pointers

john.brawn marked 2 inline comments as done. john.brawn added inline comments. Comment at: clang/lib/Sema/ParsedAttr.cpp:144 + // otherwise return a default ParsedAttrInfo. + if (A.getKind() < sizeof(AttrInfoMap)/sizeof(AttrInfoMap[0])) +return *AttrInfoMap[A.getKind()]; --

[PATCH] D31337: Use virtual functions in ParsedAttrInfo instead of function pointers

This revision was automatically updated to reflect the committed changes. Closed by commit rG590dc8d02cd7: Use virtual functions in ParsedAttrInfo instead of function pointers (authored by john.brawn). Changed prior to commit: https://reviews.llvm.org/D31337?vs=242363&id=246765#toc Repository:

[PATCH] D31338: Move ParsedAttrInfos into a registry and point to one in ParsedAttr

john.brawn updated this revision to Diff 246763. john.brawn added a comment. Herald added a subscriber: krytarowski. This has been rewritten so that the registry is only used for attributes added by plugins, and everything else continues as before. This also removes the strange dependency I'd pr

[PATCH] D31342: Add ParsedAttrInfo::handleDeclAttribute

john.brawn marked 2 inline comments as done. john.brawn added inline comments. Comment at: clang/docs/ClangPlugins.rst:74 + + class ExampleAttrInfo : public ParsedAttrInfo { + public: aaron.ballman wrote: > We should be documenting the fields of `ParsedAttrInfo

[PATCH] D31342: Add ParsedAttrInfo::handleDeclAttribute

john.brawn updated this revision to Diff 246987. john.brawn added a comment. Update based on review comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D31342/new/ https://reviews.llvm.org/D31342 Files: clang/docs/ClangPlugins.rst clang/docs/InternalsManual.rst clang/include/c

[PATCH] D31343: Add an attribute plugin example

john.brawn updated this revision to Diff 246989. john.brawn marked an inline comment as done. john.brawn added a comment. Update based on review comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D31343/new/ https://reviews.llvm.org/D31343 Files: clang/examples/Attribute/Attribut

[PATCH] D31343: Add an attribute plugin example

john.brawn marked 5 inline comments as done. john.brawn added inline comments. Comment at: clang/examples/Attribute/Attribute.cpp:25 +struct ExampleAttrInfo : public ParsedAttrInfo { + ExampleAttrInfo() { +// Set the kind to NoSemaHandlerAttribute to make sure clang doesn't

[PATCH] D31338: Move ParsedAttrInfos into a registry and point to one in ParsedAttr

This revision was automatically updated to reflect the committed changes. Closed by commit rG75d4d4bd028f: Add an attribute registry so plugins can add attributes (authored by john.brawn). Changed prior to commit: https://reviews.llvm.org/D31338?vs=246763&id=247014#toc Repository: rG LLVM Gi

[PATCH] D31342: Add ParsedAttrInfo::handleDeclAttribute

john.brawn marked 6 inline comments as done. john.brawn added inline comments. Comment at: clang/docs/ClangPlugins.rst:89 +The members of ``ParsedAttrInfo`` that a plugin attribute must define are: + * ``AttrKind``, which must be set to ``ParsedAttr::NoSemaHandlerAttribute``. + *

[PATCH] D31342: Add ParsedAttrInfo::handleDeclAttribute

john.brawn updated this revision to Diff 247681. john.brawn added a comment. Update based on review comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D31342/new/ https://reviews.llvm.org/D31342 Files: clang/docs/ClangPlugins.rst clang/docs/InternalsManual.rst clang/include/c

[PATCH] D31343: Add an attribute plugin example

john.brawn updated this revision to Diff 247683. john.brawn added a comment. Rebased and added link in documentation. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D31343/new/ https://reviews.llvm.org/D31343 Files: clang/docs/ClangPlugins.rst clang/examples/Attribute/Attribute.cpp

[PATCH] D31343: Add an attribute plugin example

john.brawn marked 2 inline comments as done. john.brawn added inline comments. Comment at: clang/examples/Attribute/Attribute.cpp:35 +Spellings.push_back({ParsedAttr::AS_GNU, "example"}); +Spellings.push_back({ParsedAttr::AS_CXX11, "::example"}); +Spellings.push_back(

[PATCH] D31343: Add an attribute plugin example

john.brawn added a comment. I've been looking into attribute merging, and as far as I can tell there's two things going on: - In SemaDeclAttr.cpp, the various handleXyzAttr functions may call mergeXyzAttr to handle clashes with other attributes within the same declaration. - In SemaDecl.cpp, s

[PATCH] D31342: Add ParsedAttrInfo::handleDeclAttribute

john.brawn marked 3 inline comments as done. john.brawn added inline comments. Comment at: clang/lib/Sema/SemaDeclAttr.cpp:6701-6707 if (!AL.isStmtAttr()) { // Type attributes are handled elsewhere; silently move on. assert(AL.isTypeAttr() && "Non-type attribut

[PATCH] D31342: Add ParsedAttrInfo::handleDeclAttribute

john.brawn marked an inline comment as done. john.brawn added inline comments. Comment at: clang/lib/Sema/SemaDeclAttr.cpp:6701-6707 if (!AL.isStmtAttr()) { // Type attributes are handled elsewhere; silently move on. assert(AL.isTypeAttr() && "Non-type attribut

[PATCH] D31343: Add an attribute plugin example

john.brawn updated this revision to Diff 251114. john.brawn added a comment. Updated to match the changes to D31342 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D31343/new/ https://reviews.llvm.org/D31343 Files: clang/docs/ClangPlugins.rst clang/ex

[PATCH] D31342: Add ParsedAttrInfo::handleDeclAttribute

john.brawn updated this revision to Diff 251113. john.brawn added a comment. Use an enum for the return value. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D31342/new/ https://reviews.llvm.org/D31342 Files: clang/docs/ClangPlugins.rst clang/docs/InternalsManual.rst clang/include/

[PATCH] D31337: Use virtual functions in ParsedAttrInfo instead of function pointers

john.brawn marked 2 inline comments as done. john.brawn added inline comments. Comment at: clang/lib/Sema/ParsedAttr.cpp:141 +static ParsedAttrInfo DefaultParsedAttrInfo; static const ParsedAttrInfo &getInfo(const ParsedAttr &A) { aaron.ballman wrote: > Might

[PATCH] D31337: Use virtual functions in ParsedAttrInfo instead of function pointers

john.brawn updated this revision to Diff 242363. john.brawn added a comment. Move DefaultParsedAttrInfo out of file scope, and move custom appertains-to function generation out of the loop in EmitClangAttrParsedAttrImpl so we only need to use one output stream. CHANGES SINCE LAST ACTION http

[PATCH] D31338: Move ParsedAttrInfos into a registry and point to one in ParsedAttr

john.brawn marked an inline comment as done. john.brawn added inline comments. Comment at: clang/lib/Basic/Attributes.cpp:101-103 + for (ParsedAttrInfoRegistry::iterator it = ParsedAttrInfoRegistry::begin(), +ie = ParsedAttrInfoRegistry::e

[PATCH] D22221: Decide whether to enable plugin tests based on cmake variables

john.brawn abandoned this revision. john.brawn added a comment. Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Abandoning this old obsolete patch. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D1/new/ https://reviews.llvm.org/D1 _

  1   2   >