[PATCH] D68720: Support -fstack-clash-protection for x86

serge-sans-paille updated this revision to Diff 224254. serge-sans-paille added a comment. Added documentation + release note entry Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68720/new/ https://reviews.llvm.org/D68720 Files:

[PATCH] D68193: In openFileForRead don't cache erroneous entries if the error relates to them being directories. Add tests.

dexonsmith accepted this revision. dexonsmith added a comment. LGTM too. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68193/new/ https://reviews.llvm.org/D68193 ___ cfe-commits mailing list

[PATCH] D68377: [Builtins] Teach Clang about memccpy

xbolva00 abandoned this revision. xbolva00 added a comment. Abandoned since D67986 was abandoned too. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68377/new/ https://reviews.llvm.org/D68377 ___

[PATCH] D68716: [clang] prevent crash for nonnull attribut in constant context (Bug 43601)

This revision was automatically updated to reflect the committed changes. Closed by commit rG59c6df9b2c52: [clang] prevent crash for nonnull attribut in constant context (Bug 43601) (authored by Tyker). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D67901: [clangd] Improve semantic highlighting in dependent contexts (fixes #154)

ilya-biryukov added inline comments. Comment at: clang-tools-extra/clangd/SemanticHighlighting.cpp:157 +if (canHighlightName(E->getName())) + addToken(E->getNameLoc(), HighlightingKind::DependentName); +return true; Could we highlighting based on

[PATCH] D68710: Remove time-trace message as it is inconsistent style

russell.gallop marked 2 inline comments as done. russell.gallop added inline comments. Comment at: clang/include/clang/Driver/Options.td:1797 + HelpText<"Turn on time profiler. Generates JSON file based on output filename. " + "Results can be analyzed with

[PATCH] D68028: [clang] Add no_builtin attribute

gchatelet added a comment. It should be ready to submit, @aaron.ballman @courbet can you take a final look? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68028/new/ https://reviews.llvm.org/D68028 ___

Re: r374006 - Reland 'Add VFS support for sanitizers' blacklist'

Thanks, Jan. And one more time, sorry about the inconvenience. On Thu, Oct 10, 2019 at 12:47 AM Jan Korous wrote: > Ok, no worries. I understand the reason for the revert and will rewrite > the patch. > > > On Oct 9, 2019, at 7:19 AM, Ilya Biryukov wrote: > > Fair enough, sorry about that. > >

[PATCH] D68028: [clang] Add no_builtin attribute

courbet added inline comments. Comment at: clang/include/clang/Basic/Attr.td:3418 + let Spellings = [Clang<"no_builtin">]; + let Args = [VariadicStringArgument<"FunctionNames">]; + let Subjects = SubjectList<[Function]>; [nit] Should this be `BuiltinNames` ?

[PATCH] D66199: [docs] loop pragmas

SjoerdMeijer added a comment. I have commit all my pragma patches, so now back to the last bit, this doc update. This doc change should now reflect our implementation. Are we happy for this to go in? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66199/new/

[PATCH] D68756: [libc++][test] Change IsSmallObject's calculation for std::any's small object buffer

EricWF accepted this revision. EricWF added inline comments. This revision is now accepted and ready to land. Comment at: libcxx/test/support/any_helpers.h:30 {}; template bool containsType(std::any const& a) { LGTM. Repository: rG LLVM Github

[PATCH] D67159: [clang] New __attribute__((__clang_arm_mve_alias)).

simon_tatham updated this revision to Diff 224288. simon_tatham marked 3 inline comments as done. simon_tatham added a comment. Removed spuriously inserted blank line and redundant `checkAttributeNumArgs`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D68710: Remove time-trace message as it is inconsistent style

This revision was automatically updated to reflect the committed changes. russell.gallop marked an inline comment as done. Closed by commit rG9d9ac46a08d7: Remove rest of time-trace message as it is inconsistent style (authored by russell.gallop). Changed prior to commit:

[PATCH] D68720: Support -fstack-clash-protection for x86

serge-sans-paille added a subscriber: eli.friedman. serge-sans-paille added a comment. @efriedma the free probe algorithm requires more testing, and I'd like to take into account memset and memcpy as free probes too. To showcase this algorithm, consider the following LLVM bitcode: define

[PATCH] D68554: [clang-format] Proposal for clang-format to give compiler style warnings

klimek added a comment. In D68554#1702090 , @MyDeveloperDay wrote: > As this behaviour is not something clang-format should do because there are > external tool that can do this for us, I thought I'd set myself a challenge > of trying to write this

[PATCH] D68028: [clang] Add no_builtin attribute

gchatelet updated this revision to Diff 224285. gchatelet marked 2 inline comments as done. gchatelet added a comment. - Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68028/new/ https://reviews.llvm.org/D68028 Files:

[PATCH] D68028: [clang] Add no_builtin attribute

gchatelet updated this revision to Diff 224286. gchatelet added a comment. - Fix missing rename Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68028/new/ https://reviews.llvm.org/D68028 Files: clang/include/clang/Basic/Attr.td

r374294 - Remove rest of time-trace message as it is inconsistent style

Author: russell_gallop Date: Thu Oct 10 02:33:53 2019 New Revision: 374294 URL: http://llvm.org/viewvc/llvm-project?rev=374294=rev Log: Remove rest of time-trace message as it is inconsistent style Other options which create output files don't produce output messages. Improve documentation to

[PATCH] D68554: [clang-format] Proposal for clang-format to give compiler style warnings

MyDeveloperDay marked an inline comment as done. MyDeveloperDay added a comment. > I think it's easy enough to do as a kind of driver, either in python or C++, > but then again, at that point putting it into ClangFormat seems fine. One > thing that I find on the one side kinda nice, but on the

[PATCH] D68076: [AArch64] Enable unwind tables by default for Gnu targets

tellenbach added a comment. This patch would fix PR37240 but is probably not the most optimal solution. For other ideas see http://lists.llvm.org/pipermail/llvm-dev/2019-September/135433.html . I'll leave this patch open

r374285 - [clang] prevent crash for nonnull attribut in constant context (Bug 43601)

Author: tyker Date: Thu Oct 10 00:13:20 2019 New Revision: 374285 URL: http://llvm.org/viewvc/llvm-project?rev=374285=rev Log: [clang] prevent crash for nonnull attribut in constant context (Bug 43601) Summary: bug : https://bugs.llvm.org/show_bug.cgi?id=43601 Reviewers: rnk Reviewed By: rnk

[PATCH] D29039: [clang-format] Proposal for clang-format -r option

MyDeveloperDay added a comment. Comment at: tools/clang-format/ClangFormat.cpp:345 + + Regex regex(FileNames[i]); + I could imagine the path you supplying being just a directory . or ../../include/Format as much as a wild card, once in that directory I

[PATCH] D68710: Remove time-trace message as it is inconsistent style

russell.gallop updated this revision to Diff 224278. russell.gallop added a comment. Update so --help helps user find trace file. Documentation has the further details on how to view it. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68710/new/ https://reviews.llvm.org/D68710 Files:

[PATCH] D68720: Support -fstack-clash-protection for x86

serge-sans-paille marked 3 inline comments as done. serge-sans-paille added inline comments. Comment at: llvm/lib/Target/X86/X86FrameLowering.cpp:400 +!(STI.isOSWindows() && !STI.isTargetMachO()); +if (InlineStackClashProtector && !InEpilogue) { + const uint64_t

[PATCH] D68710: Remove time-trace message as it is inconsistent style

anton-afanasyev accepted this revision. anton-afanasyev added a comment. This revision is now accepted and ready to land. Yes, I agree. LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68710/new/ https://reviews.llvm.org/D68710 ___

[PATCH] D68694: [clang-tidy] hicpp-signed-bitwise: Do not show "use of a signed integer operand with a binary bitwise operator" for positive integer operands

JonasToth added a comment. In D68694#1701772 , @aaron.ballman wrote: > > In my opinion this check should be disabled in case of integer literals, > > since there are a lot of existing code (even in system libraries) where > > user can do nothing, e.g.:

[PATCH] D68554: [clang-format] Proposal for clang-format to give compiler style warnings

MyDeveloperDay marked 2 inline comments as done. MyDeveloperDay added inline comments. Comment at: clang/tools/clang-format/ClangFormat.cpp:371 + if (!Replaces.empty()) { +IntrusiveRefCntPtr DiagOpts = +new DiagnosticOptions(); klimek

[PATCH] D67122: [UBSan][clang][compiler-rt] Applying non-zero offset to nullptr is undefined behaviour

russell.gallop added a comment. This is failing the sanitizer lint check: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/23819/steps/64-bit%20check-sanitizer/logs/stdio /compiler-rt/lib/ubsan/ubsan_checks.inc:22: Lines should be <= 80 characters long [whitespace/line_length]

[PATCH] D67160: [clang, ARM] Default to -fno-lax-vector-conversions in ARM v8.1-M.

dmgreen added a subscriber: rsmith. dmgreen accepted this revision. dmgreen added a comment. Yeah, OK. D67678 is the patch that will change the default, but only to "int", not to "none" for the moment. An earlier version of that patch had a different way of

r374288 - Recommit "[Clang] Pragma vectorize_width() implies vectorize(enable)"

Author: sjoerdmeijer Date: Thu Oct 10 01:27:14 2019 New Revision: 374288 URL: http://llvm.org/viewvc/llvm-project?rev=374288=rev Log: Recommit "[Clang] Pragma vectorize_width() implies vectorize(enable)" This was further discussed at the llvm dev list:

[PATCH] D63932: [GlobalDCE] Dead Virtual Function Elimination

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

[PATCH] D68660: [tooling] Teach Tooling to understand compilation with offloading.

klimek added inline comments. Comment at: clang/lib/Tooling/Tooling.cpp:117 // The one job we find should be to invoke clang again. const auto = cast(*Jobs.begin()); if (StringRef(Cmd.getCreator().getName()) != "clang") { tra wrote: > hliao wrote: > >

[PATCH] D68743: [ARM] Fix arm_neon.h with -flax-vector-conversions=none, part 2.

SjoerdMeijer accepted this revision. SjoerdMeijer added a comment. This revision is now accepted and ready to land. Thanks again, looks good. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68743/new/ https://reviews.llvm.org/D68743

[PATCH] D68028: [clang] Add no_builtin attribute

gchatelet updated this revision to Diff 224271. gchatelet added a comment. - rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68028/new/ https://reviews.llvm.org/D68028 Files: clang/include/clang/Basic/Attr.td

[PATCH] D68767: [clang-format] NFC - Move functionality into functions to help code structure

MyDeveloperDay created this revision. MyDeveloperDay added reviewers: klimek, mitchell-stellar, owenpan. MyDeveloperDay added a project: clang-format. Herald added a project: clang. I don't think its the main()'s responsibility to be working out how to dump the config, or the format()'s

[PATCH] D68581: Include leading attributes in DeclStmt's SourceRange

sberg updated this revision to Diff 224303. sberg edited the summary of this revision. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68581/new/ https://reviews.llvm.org/D68581 Files: clang/lib/Parse/ParseStmt.cpp clang/test/AST/sourceranges.cpp Index:

[PATCH] D67122: [UBSan][clang][compiler-rt] Applying non-zero offset to nullptr is undefined behaviour

lebedev.ri added a comment. In D67122#1703482 , @russell.gallop wrote: > This is failing the sanitizer lint check: > > http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/23819/steps/64-bit%20check-sanitizer/logs/stdio > >

r374324 - Revert "Use -fdebug-compilation-dir to form absolute paths in coverage mappings"

Author: kadircet Date: Thu Oct 10 05:20:11 2019 New Revision: 374324 URL: http://llvm.org/viewvc/llvm-project?rev=374324=rev Log: Revert "Use -fdebug-compilation-dir to form absolute paths in coverage mappings" This reverts commit f6777964bde28c349d3e289ea37ecf5f5eeedbc4. Because the absolute

[PATCH] D67122: [UBSan][clang][compiler-rt] Applying non-zero offset to nullptr is undefined behaviour

lebedev.ri added a comment. Not unexpectedly, this broke sanitizer-x86_64-linux-bootstrap-ubsan. I'm going to attempt to address uncovered issues Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67122/new/ https://reviews.llvm.org/D67122

[PATCH] D68554: [clang-format] Proposal for clang-format to give compiler style warnings

klimek added a comment. In D68554#1703358 , @MyDeveloperDay wrote: > > I think it's easy enough to do as a kind of driver, either in python or > > C++, but then again, at that point putting it into ClangFormat seems fine. > > One thing that I find on

[PATCH] D63131: arm64_32: implement the desired ABI for the ILP32 triple.

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

r374328 - [AST] ASTReader::ReadSLocEntry(): move computation of FirstDecl into the branch where it's used

Author: lebedevri Date: Thu Oct 10 05:22:42 2019 New Revision: 374328 URL: http://llvm.org/viewvc/llvm-project?rev=374328=rev Log: [AST] ASTReader::ReadSLocEntry(): move computation of FirstDecl into the branch where it's used The existing code is not defined, you are not allowed to produce

[PATCH] D67122: [UBSan][clang][compiler-rt] Applying non-zero offset to nullptr is undefined behaviour

lebedev.ri added a comment. http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap-ubsan/builds/15329/steps/annotate/logs/stdio appears to contain "only" 3 distinct issues. I've pushed those 3 fixes, but maybe that is not enough, we'll see. Repository: rG LLVM Github Monorepo

[PATCH] D67608: [ARM] Preserve fpu behaviour for '-crypto'

peter.smith accepted this revision. peter.smith added a comment. This revision is now accepted and ready to land. Thanks for the update. That looks good to me. Will be good to wait for a day before committing to give US timezone a chance to object. Repository: rG LLVM Github Monorepo

[PATCH] D67608: [ARM] Preserve fpu behaviour for '-crypto'

dnsampaio updated this revision to Diff 224323. dnsampaio added a comment. Attending review request: - Fixed to only add '-crypto' when not passing -fno-integrated-as - Fixed test to use arm-none-none-eabi - Changed comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D61675: [WIP] Update IRBuilder::CreateFNeg(...) to return a UnaryOperator

gribozavr added a comment. Sorry, but this commit broke OCaml tests: http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/19014 I reverted it in r374354. Please test before re-landing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D66049: [analyzer] PR41729: Fix some false positives and improve strlcat and strlcpy modeling

Szelethus accepted this revision. Szelethus added a comment. This revision is now accepted and ready to land. LGTM! In D66049#1701730 , @dkrupp wrote: > I also analyzed openssl with the baseline and this version, but did not find > any new warnings. >

[PATCH] D61675: [WIP] Update IRBuilder::CreateFNeg(...) to return a UnaryOperator

cameron.mcinally added a comment. In D61675#1703749 , @cameron.mcinally wrote: > > I reverted it in r374354. Please test before re-landing. > > Hmm, how do I run those tests? I did not see that failure with check-all. > > That's a pretty straightforward

r374363 - [OPENMP50]Register vendor name only once in vendor context selector.

Author: abataev Date: Thu Oct 10 08:15:26 2019 New Revision: 374363 URL: http://llvm.org/viewvc/llvm-project?rev=374363=rev Log: [OPENMP50]Register vendor name only once in vendor context selector. No need to store multiple copies of the same vendor names in the context selector, keep only

[PATCH] D62731: Add support for options -frounding-math, ftrapping-math, -fp-model=, and -fp-exception-behavior=, : Specify floating point behavior

mibintc marked an inline comment as done. mibintc added a comment. I looked over the codegen testcase fpconstrained.c and it looks pretty good, so i think this is ready for your review comments. I'll be off the grid for a couple days but looking forward to receiving your feedback. I inserted

[PATCH] D68792: [Alignment] Migrate Attribute::getWith(Stack)Alignment

gchatelet created this revision. gchatelet added a reviewer: courbet. Herald added subscribers: llvm-commits, cfe-commits, hiraditya, nhaehnle, jvesely, arsenm. Herald added a reviewer: jdoerfert. Herald added projects: clang, LLVM. This is patch is part of a series to introduce an Alignment

[PATCH] D68193: In openFileForRead don't cache erroneous entries if the error relates to them being directories. Add tests.

This revision was automatically updated to reflect the committed changes. Closed by commit rG4abac5330277: In openFileForRead dont cache erroneous entries if the error relates to them… (authored by kousikk). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D68795: [libTooling] Introduce a single, "main" header file for Transformer.

gribozavr added inline comments. Comment at: clang/include/clang/Tooling/Transformer/Transformer.h:28 +// also update their code to reference `TransformerTool`. +#include "clang/Tooling/Transformer/TransformerTool.h" +// Temporary alias to assist clients in migrating to new

[PATCH] D68792: [Alignment] Migrate Attribute::getWith(Stack)Alignment

gchatelet added a comment. @courbet there's a slight difference in the way the field is serialized in the mir / ir format (hence not [NFC]) but there's no change in semantics. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68792/new/

[PATCH] D66199: [docs] loop pragmas

hsaito added inline comments. Comment at: docs/LanguageExtensions.rst:3069 +There are loop hints that control transformations (e.g. vectorization, loop +unrolling) and there loop hints that set transformation options (e.g. +``vectorize_width``, ``unroll_count``). Pragmas

[PATCH] D68795: [libTooling] Introduce a single, "main" header file for Transformer.

ymandel marked an inline comment as done. ymandel added inline comments. Comment at: clang/include/clang/Tooling/Transformer/Transformer.h:28 +// also update their code to reference `TransformerTool`. +#include "clang/Tooling/Transformer/TransformerTool.h" +// Temporary alias to

[PATCH] D68818: [hip][cuda] Fix the extended lambda name mangling issue.

hliao created this revision. hliao added reviewers: tra, rsmith, yaxunl. Herald added a reviewer: martong. Herald added a reviewer: shafik. Herald added a project: clang. Herald added a subscriber: cfe-commits. - HIP/CUDA host side needs to use device kernel symbol name to match the device side

[PATCH] D68818: [hip][cuda] Fix the extended lambda name mangling issue.

hliao added a comment. this's a patch address the same issue previously proposed to be worked around in https://reviews.llvm.org/D63164 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68818/new/ https://reviews.llvm.org/D68818

[PATCH] D68377: [Builtins] Teach Clang about memccpy

xbolva00 reclaimed this revision. xbolva00 added a comment. (still interested in this patch) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68377/new/ https://reviews.llvm.org/D68377 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D61675: [WIP] Update IRBuilder::CreateFNeg(...) to return a UnaryOperator

cameron.mcinally added a comment. > I reverted it in r374354. Please test before re-landing. Hmm, how do I run those tests? I did not see that failure with check-all. That's a pretty straightforward failure. It's just a one-line IR change: `fsub float {{.*}}0{{.*}}, %F1` -> `fneg float %F1`

[PATCH] D67551: [AArch64][SVE] Implement sdot and udot (lane) intrinsics

greened added a comment. LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67551/new/ https://reviews.llvm.org/D67551 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D67122: [UBSan][clang][compiler-rt] Applying non-zero offset to nullptr is undefined behaviour

lebedev.ri added a comment. In D67122#1703616 , @lebedev.ri wrote: > http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap-ubsan/builds/15329/steps/annotate/logs/stdio > appears to contain "only" 3 distinct issues. > I've pushed those 3

[PATCH] D68746: [Clang][OpenMP Offload] Move offload registration code to the wrapper

ABataev added inline comments. Comment at: clang/tools/clang-offload-wrapper/ClangOffloadWrapper.cpp:74 + IntegerType *getSizeTTy() { +switch (M.getDataLayout().getPointerTypeSize(Type::getInt8PtrTy(C))) { +case 4u: Same question as before: maybe better

[PATCH] D68795: [libTooling] Introduce a single, "main" header file for Transformer.

ymandel created this revision. ymandel added a reviewer: gribozavr. Herald added a subscriber: jfb. Herald added a project: clang. Split the existing `Transformer.h` into two new files, with more focused purpose: `RewriteRule.h` and `TransformerTool.h`. The file `Transformer.h` is repurposed to

[PATCH] D68807: [ClangTidy] Separate tests for infrastructure and checkers

gribozavr created this revision. Herald added subscribers: cfe-commits, lebedev.ri, jfb, arphaman, mgrang, christof, kbarton, aheejin, nemanjai, srhines. Herald added a reviewer: jfb. Herald added a reviewer: jdoerfert. Herald added a reviewer: lebedev.ri. Herald added a project: clang. gribozavr

[PATCH] D68554: [clang-format] Proposal for clang-format to give compiler style warnings

MyDeveloperDay updated this revision to Diff 224403. MyDeveloperDay added a comment. builds ontop of D68767: [clang-format] NFC - Move functionality into functions to help code structure move replacement warnings into a separate function and move outputXML out

[PATCH] D68554: [clang-format] Proposal for clang-format to give compiler style warnings

MyDeveloperDay added a comment. In D68554#1703596 , @klimek wrote: > I ran through the same line of reasoning in my head when writing my last > comment :) The argument about the readability of error messages by tools is > definitely a good one, so I'd

[PATCH] D68818: [hip][cuda] Fix the extended lambda name mangling issue.

hliao marked 2 inline comments as done. hliao added a comment. minor comment to help review. BTW, as the AST ser/deser is changed as well, not sure we have compatibility issue and, if there is, how to handle that. please advice me on that concern. Thanks. Comment at:

[PATCH] D68193: In openFileForRead don't cache erroneous entries if the error relates to them being directories. Add tests.

arphaman accepted this revision. arphaman 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/D68193/new/ https://reviews.llvm.org/D68193

[PATCH] D67550: [AArch64][SVE] Implement unpack intrinsics

greened added inline comments. Comment at: lib/Target/AArch64/SVEInstrFormats.td:836 class sve_int_perm_unpk sz16_64, bits<2> opc, string asm, -ZPRRegOp zprty1, ZPRRegOp zprty2> +ZPRRegOp zprty1, ZPRRegOp zprty2, SDPatternOperator

[PATCH] D29039: [clang-format] Proposal for clang-format -r option

zturner added a comment. In Windows you just write a Python script to do this, and this works everywhere, not just on one platform or the other, so bash isn't even necessary and Python is easy to write so I wouldn't really say it's "even harder". If you google for `run-clang-format.py` you'll

r374387 - [OPENMP50]Support for declare variant directive for NVPTX target.

Author: abataev Date: Thu Oct 10 10:28:10 2019 New Revision: 374387 URL: http://llvm.org/viewvc/llvm-project?rev=374387=rev Log: [OPENMP50]Support for declare variant directive for NVPTX target. NVPTX does not support global aliases. Instead, we have to copy the full body of the variant function

r374399 - [clang-format] throws an incorrect assertion in consumeToken() formatting the MSVC stl

Author: paulhoad Date: Thu Oct 10 10:54:47 2019 New Revision: 374399 URL: http://llvm.org/viewvc/llvm-project?rev=374399=rev Log: [clang-format] throws an incorrect assertion in consumeToken() formatting the MSVC stl Summary: An incorrect assertion is thrown when clang-formatting MSVC's STL

[PATCH] D68562: [clangd] Add RemoveUsingNamespace tweak.

ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. LGTM, many thanks! A few last NITs too Comment at: clang-tools-extra/clangd/refactor/tweaks/RemoveUsingNamespace.cpp:31 +/// std::vector foo(std::map); +//

[PATCH] D68795: [libTooling] Introduce a single, "main" header file for Transformer.

ymandel updated this revision to Diff 224414. ymandel added a comment. Herald added a subscriber: mgorny. Removed umbrella-header, as per discussion. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68795/new/ https://reviews.llvm.org/D68795 Files:

[PATCH] D68795: [libTooling] Introduce a single, "main" header file for Transformer.

ymandel marked an inline comment as done. ymandel added inline comments. Comment at: clang/include/clang/Tooling/Transformer/Transformer.h:28 +// also update their code to reference `TransformerTool`. +#include "clang/Tooling/Transformer/TransformerTool.h" +// Temporary alias to

[PATCH] D68743: [ARM] Fix arm_neon.h with -flax-vector-conversions=none, part 2.

This revision was automatically updated to reflect the committed changes. Closed by commit rG30a96d3fcb76: [ARM] Fix arm_neon.h with -flax-vector-conversions=none, part 2. (authored by efriedma). Changed prior to commit: https://reviews.llvm.org/D68743?vs=224229=224432#toc Repository: rG

[PATCH] D68824: Fix __builtin_assume_aligned with too large values.

erichkeane created this revision. erichkeane added reviewers: lebedev.ri, hfinkel, joey, jdoerfert. lebedev.ri added a comment. Thanks, i like it. Comment at: clang/lib/CodeGen/CodeGenFunction.cpp:2058-2073 +/* void CodeGenFunction::EmitAlignmentAssumption(llvm::Value

[PATCH] D68753: [CUDA][HIP} Add a test for constexpr default ctor

yaxunl marked an inline comment as done. yaxunl added inline comments. Comment at: test/SemaCUDA/constexpr-ctor.cu:14-27 +template struct B { + T a; + constexpr B() = default; +}; + +template struct C { + T a; tra wrote: > Do we really need three identical

[PATCH] D68824: Fix __builtin_assume_aligned with too large values.

erichkeane updated this revision to Diff 224449. erichkeane added a comment. Remove unused code CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68824/new/ https://reviews.llvm.org/D68824 Files: clang/include/clang/Basic/DiagnosticSemaKinds.td clang/lib/CodeGen/CGBuiltin.cpp

[PATCH] D68824: Fix __builtin_assume_aligned with too large values.

hfinkel added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:2854 +def warn_assume_align_too_big : Warning<"alignments greater than 1 << 29 are " + "unsupported by LLVM">, InGroup; Should this be in the IgnoredAttributes group?

[PATCH] D68660: [tooling] Teach Tooling to understand compilation with offloading.

hliao updated this revision to Diff 224476. hliao marked an inline comment as done. hliao added a comment. revice comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68660/new/ https://reviews.llvm.org/D68660 Files:

[PATCH] D68807: [ClangTidy] Separate tests for infrastructure and checkers

lebedev.ri added a comment. That's a lot of moved files. How about *only* moving the infrastructure tests? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68807/new/ https://reviews.llvm.org/D68807 ___

[PATCH] D29039: [clang-format] Proposal for clang-format -r option

MyDeveloperDay added a comment. In D29039#1704166 , @zturner wrote: > In Windows you just write a Python script to do this, and this works > everywhere, not just on one platform or the other, so bash isn't even > necessary and Python is easy to write so

[PATCH] D67536: [WIP] [clangd] Add support for an inactive regions notification

nridge updated this revision to Diff 224480. nridge added a comment. Updated to use as isInactive flag on SemanticHighlightingInformation Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67536/new/ https://reviews.llvm.org/D67536 Files:

[PATCH] D68824: Fix __builtin_assume_aligned with too large values.

thakis added a comment. The new test fails on Windows: -- Testing: 16032 tests, 32 workers -- Testing: 0.. 10.. 20.. 30.. 40.. 50 FAIL: Clang :: Sema/builtin-assume-aligned.c (8862 of 16032) TEST 'Clang :: Sema/builtin-assume-aligned.c' FAILED

[PATCH] D68824: Fix __builtin_assume_aligned with too large values.

thakis added a comment. Since you just left IRC, I reverted this in 374456 for now. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68824/new/ https://reviews.llvm.org/D68824 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D68746: [Clang][OpenMP Offload] Move offload registration code to the wrapper

sdmitriev added inline comments. Comment at: clang/tools/clang-offload-wrapper/ClangOffloadWrapper.cpp:74 + IntegerType *getSizeTTy() { +switch (M.getDataLayout().getPointerTypeSize(Type::getInt8PtrTy(C))) { +case 4u: ABataev wrote: > Same question as

[PATCH] D67536: [WIP] [clangd] Add support for an inactive regions notification

nridge added a comment. Ok, I updated the patch to convey the line highlight separately from the token highlights. I did use the simplified approach I mentioned, where I use a single `isInactive` flag. If you'd prefer the more general approach where we send an array of line styles, I can do

[PATCH] D68832: [tsan,msan] Insert module constructors in a module pass

leonardchan accepted this revision. leonardchan added a comment. This revision is now accepted and ready to land. Thanks for finding the root cause of this! Comment at: llvm/lib/Transforms/Instrumentation/ThreadSanitizer.cpp:143 + +static void insertModuleCtor(Module ) { +

[PATCH] D68838: [ARM] Fix arm_neon.h with -flax-vector-conversions=none, part 3

rsmith added a comment. Thanks again! In my testing, this is enough to get the clang testsuite to pass with the default changed to `-flax-vector-conversions=integer`. Comment at: test/CodeGen/aarch64-v8.2a-neon-intrinsics.c:149-170 +uint16x4_t test_vcvt_u16_f16 (float16x4_t

[PATCH] D68807: [ClangTidy] Separate tests for infrastructure and checkers

gribozavr added a comment. It would look weird if we have a ton of tests for checkers in a directory, and then a subdirectory for infra tests. Why are we trying to optimize the number of renamed files? I already did the renaming, so we are not saving any work. I think we should optimize for

[PATCH] D68746: [Clang][OpenMP Offload] Move offload registration code to the wrapper

sdmitriev added a subscriber: grokos. sdmitriev added inline comments. Comment at: clang/tools/clang-offload-wrapper/ClangOffloadWrapper.cpp:74 + IntegerType *getSizeTTy() { +switch (M.getDataLayout().getPointerTypeSize(Type::getInt8PtrTy(C))) { +case 4u:

r374465 - PR43629: Fix crash evaluating constexpr placement new on a subobject of

Author: rsmith Date: Thu Oct 10 15:31:17 2019 New Revision: 374465 URL: http://llvm.org/viewvc/llvm-project?rev=374465=rev Log: PR43629: Fix crash evaluating constexpr placement new on a subobject of an out-of-lifetime object. Modified: cfe/trunk/include/clang/Basic/DiagnosticASTKinds.td

[PATCH] D68753: [CUDA][HIP} Add a test for constexpr default ctor

yaxunl marked 2 inline comments as done. yaxunl added inline comments. Comment at: test/SemaCUDA/constexpr-ctor.cu:14-27 +template struct B { + T a; + constexpr B() = default; +}; + +template struct C { + T a; tra wrote: > yaxunl wrote: > > tra wrote: > > >

[PATCH] D68720: Support -fstack-clash-protection for x86

efriedma added inline comments. Comment at: llvm/lib/Target/X86/X86FrameLowering.cpp:423 + AbsOffset - CurrentAbsOffset + PageSize); +if (FreeProbeIterator != MBB.end()) { + NumFrameFreeProbe++; Each probe has to

r374467 - Fix help message for -ffp-contract

Author: yaxunl Date: Thu Oct 10 15:43:00 2019 New Revision: 374467 URL: http://llvm.org/viewvc/llvm-project?rev=374467=rev Log: Fix help message for -ffp-contract Differential Revision: https://reviews.llvm.org/D68823 Modified: cfe/trunk/include/clang/Driver/Options.td Modified:

[PATCH] D68838: [ARM] Fix arm_neon.h with -flax-vector-conversions=none, part 3

efriedma updated this revision to Diff 224496. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68838/new/ https://reviews.llvm.org/D68838 Files: test/CodeGen/aarch64-v8.2a-neon-intrinsics.c utils/TableGen/NeonEmitter.cpp Index: utils/TableGen/NeonEmitter.cpp

[PATCH] D68823: Fix help message for -ffp-contract

This revision was automatically updated to reflect the committed changes. Closed by commit rGc10a64718ed2: Fix help message for -ffp-contract (authored by yaxunl). Herald added a project: clang. Changed prior to commit: https://reviews.llvm.org/D68823?vs=224436=224497#toc Repository: rG

[PATCH] D63932: [GlobalDCE] Dead Virtual Function Elimination

pcc accepted this revision. pcc 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/D63932/new/ https://reviews.llvm.org/D63932 ___

  1   2   >