[PATCH] D125165: [Clang] Introduce clang-offload-packager tool to bundle device files

yaxunl added a comment. In D125165#3506448 , @thakis wrote: > We could add a "clang-offload-bundler and clang-offload-wrapper are > deprecated, replace them with $whatever" in the release notes and then remove > them a release later, assuming the replac

[PATCH] D125396: [clang] Fix KEYALL

delcypher accepted this revision. delcypher added a comment. This revision is now accepted and ready to land. @yaxunl Thanks for addressing my feedback so quickly. I think the commit message should also mention that `KEYCUDA` is now included in `KEYALL`. Other than that LGTM. CHANGES SINCE LAS

[PATCH] D125396: [clang] Fix KEYALL

yaxunl added a comment. In D125396#3506478 , @delcypher wrote: > @yaxunl Thanks for addressing my feedback so quickly. I think the commit > message should also mention that `KEYCUDA` is now included in `KEYALL`. Other > than that LGTM. will do when com

[PATCH] D123878: [AMDGPU] Add remarks to output some resource usage

afanfa added a comment. Herald added subscribers: kosarev, jsilvanus. If possible, I would like to keep some kind of delimiter. I like the idea of having it at the beginning and at the end of the section. The best option would be to convince clang to print new lines. Repository: rG LLVM Gith

[PATCH] D125165: [Clang] Introduce clang-offload-packager tool to bundle device files

thakis added a comment. In D125165#3506477 , @yaxunl wrote: > In D125165#3506448 , @thakis wrote: > >> We could add a "clang-offload-bundler and clang-offload-wrapper are >> deprecated, replace them with $whateve

[PATCH] D125165: [Clang] Introduce clang-offload-packager tool to bundle device files

yaxunl added a comment. In D125165#3506502 , @thakis wrote: > In D125165#3506477 , @yaxunl wrote: > >> In D125165#3506448 , @thakis wrote: >> >>> We could add a "clang-off

[PATCH] D125165: [Clang] Introduce clang-offload-packager tool to bundle device files

jhuber6 added a comment. In D125165#3506502 , @thakis wrote: > In D125165#3506477 , @yaxunl wrote: > >> In D125165#3506448 , @thakis wrote: >> >>> We could add a "clang-of

[clang] 65860a9 - Fix the Clang sphinx build

Author: Aaron Ballman Date: 2022-05-11T12:09:21-04:00 New Revision: 65860a9f5db1befcbd7dc73059b63b5cbce418f4 URL: https://github.com/llvm/llvm-project/commit/65860a9f5db1befcbd7dc73059b63b5cbce418f4 DIFF: https://github.com/llvm/llvm-project/commit/65860a9f5db1befcbd7dc73059b63b5cbce418f4.diff

[PATCH] D124447: [clang-tidy] Add infrastructure support for running on project-level information

whisperity added a comment. In D124447#3493996 , @whisperity wrote: > In D124447#3493446 , @aaron.ballman > wrote: > >> precommit CI is showing a fair amount of failures that I believe are related >> to your pa

[PATCH] D125362: [NFC][analyzer] Transitive interestingness in BugReporter

balazske added a comment. Is it problem if `markNotInteresting` is used on a symbol that was marked interesting and has subregions? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125362/new/ https://reviews.llvm.org/D125362 ___

[clang] f933c89 - [OpenMP] Add a check for alignment in the offload packager

Author: Joseph Huber Date: 2022-05-11T12:25:44-04:00 New Revision: f933c896d1abe9e0e7854e95023fe389c0e704eb URL: https://github.com/llvm/llvm-project/commit/f933c896d1abe9e0e7854e95023fe389c0e704eb DIFF: https://github.com/llvm/llvm-project/commit/f933c896d1abe9e0e7854e95023fe389c0e704eb.diff

[PATCH] D121733: Clean pathnames in FileManager.

ppluzhnikov updated this revision to Diff 428693. ppluzhnikov added a comment. Fix clangd test failure. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121733/new/ https://reviews.llvm.org/D121733 Files: clang-tools-extra/clang-tidy/google/Upgrade

[PATCH] D91000: [clang-tidy] Add bugprone-unsafe-functions checker.

whisperity added a comment. Just one question if you could try this out for me: what happens if you run `clang-tidy a.c b.c` (two TUs in the invocation) where **one of them** (preferably the later one, i.e. **`b.c`**) does //NOT// have Annex K enabled? I believe the cached `IsAnnexKAvailable` (

[PATCH] D124726: Suggest typoed directives in preprocessor conditionals

aaron.ballman added inline comments. Comment at: llvm/lib/Support/StringRef.cpp:102 +// Find a similar string in `Candidates`. +Optional StringRef::find_similar_str(const std::vector &Candidates, size_t Dist) const { + // We need to check if `rng` has the exact case-insensitive

[PATCH] D125011: [MSVC] Add support for pragma alloc_text

aaron.ballman added inline comments. Comment at: clang/test/Sema/pragma-ms-alloc-text.cpp:5 +#pragma alloc_text(a// expected-warning {{expected ',' in '#pragma alloc_text'}} +#pragma alloc_text(a, a // expected-warning {{missing ')' after '#pragma alloc_text'}} expe

[PATCH] D125378: [Attribute] Introduce shuffle attribute to be used for __shfl_sync like cross-lane APIs

nhaehnle requested changes to this revision. nhaehnle added a comment. I'm sorry to say that this patch seems confused about semantics. It lacks clear definitions, and in particular for the `shuffle` attribute in LLVM IR, you almost certainly just want the already existing `convergent` instead.

[PATCH] D125400: [clang][Analyzer] Add errno state to standard functions modeling.

balazske created this revision. Herald added subscribers: steakhal, manas, ASDenysPetrov, martong, gamesh411, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun. Herald added a reviewer: Szelethus. Herald added a project: All. balazske requested revie

[PATCH] D125338: [HLSL] add -D option for dxc mode.

python3kgae updated this revision to Diff 428698. python3kgae added a comment. Add check for macro. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125338/new/ https://reviews.llvm.org/D125338 Files: clang/include/clang/Driver/Options.td clang/l

[PATCH] D125165: [Clang] Introduce clang-offload-packager tool to bundle device files

yaxunl added a comment. In D125165#3506529 , @jhuber6 wrote: > In D125165#3506502 , @thakis wrote: > >> In D125165#3506477 , @yaxunl wrote: >> >>> In D125165#3506448

[PATCH] D118996: [clang-tidy] Support C++14 in bugprone-signal-handler.

whisperity added a comment. @njames93 What do you think about the current approach? It will under-approximate the problem-inducing node set but at least cover what we know about C++ now. (@balazske please mark //"Done"// the answered discussion threads.) Comment at: clang-to

[PATCH] D125362: [NFC][analyzer] Transitive interestingness in BugReporter

steakhal added a comment. In D125362#3506545 , @balazske wrote: > Is it problem if `markNotInteresting` is used on a symbol that was marked > interesting and has subregions? In that case, the sub-symbols will remain still interesting. We might need to t

[PATCH] D124815: [libclang] Fall back to getMainExecutable when dladdr fails

aykevl added a comment. This file is also linked to `libclang.a`. When linking `libclang.a` on Alpine Linux, `dladdr` is not available and the only fallback I can think of is `getMainExecutable`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D12481

[PATCH] D123773: [clang][analyzer][ctu] Make CTU a two phase analysis

xazax.hun added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/ExprEngineCallAndReturn.cpp:446 +} +const bool BState = State->contains(D); +if (!BState) { // This is the first time we see this foreign function. martong wrote: > xazax.hun w

[PATCH] D123773: [clang][analyzer][ctu] Make CTU a two phase analysis

xazax.hun added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/ExprEngineCallAndReturn.cpp:446 +} +const bool BState = State->contains(D); +if (!BState) { // This is the first time we see this foreign function. xazax.hun wrote: > martong w

[PATCH] D125338: [HLSL] add -D option for dxc mode.

aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM aside from a super minor nit with the test. Comment at: clang/test/Driver/dxc_D.hlsl:14 +// ERROR-NOT: error: \ No newline at end of file You should add the newline back to the end of th

[PATCH] D123773: [clang][analyzer][ctu] Make CTU a two phase analysis

xazax.hun added inline comments. Comment at: clang/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h:116 + const bool Foreign = false; // From CTU. + martong wrote: > martong wrote: > > xazax.hun wrote: > > > I feel that we use different terms for t

[PATCH] D124447: [clang-tidy] Add infrastructure support for running on project-level information

whisperity added a comment. In D124447#3506536 , @whisperity wrote: > In D124447#3493996 , @whisperity > wrote: > >> In D124447#3493446 , >> @aaron.ballman wrote: >> >>

[PATCH] D125178: Warn if using `elifdef` & `elifndef` in not C2x & C++2b mode

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM aside from a super minor nit in a test file. Can you please add a release note for the new diagnostics as well? I'm happy to land this for you when you're ready, but let me

[PATCH] D125401: [OpenCL] Do not guard vload/store_half builtins

svenvh created this revision. svenvh added a reviewer: Anastasia. svenvh added a project: clang. Herald added subscribers: Naghasan, ldrumm, yaxunl. Herald added a project: All. svenvh requested review of this revision. Herald added a subscriber: cfe-commits. The vload*_half* and vstore*_half* bui

[PATCH] D125142: [clang][auto-init] Remove -enable flag for "zero" mode

kees added a comment. In D125142#3505767 , @jfb wrote: > I think the most relevant post from @rsmith is: > https://discourse.llvm.org/t/making-ftrivial-auto-var-init-zero-a-first-class-option/55143/40 > > He has a prototype: https://reviews.llvm.org/D792

[PATCH] D123773: [clang][analyzer][ctu] Make CTU a two phase analysis

xazax.hun accepted this revision. xazax.hun added a comment. This revision is now accepted and ready to land. Overall looks good to me. I wish some parts would be simpler but it looks like sometimes it is not easy to extend the current code and we might need to do some refactoring at some point.

[PATCH] D125402: [clang][diag] warn if function returns class type by-const-value

nlee created this revision. nlee added a reviewer: lattner. Herald added a project: All. nlee requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Returning class or struct by value with const qualifier should be avoided since it prevents move

[PATCH] D124613: In MSVC compatibility mode, friend function declarations behave as function declarations

thakis added a comment. Given that: - This is not needed for system headers - Before the change, users had to fix their standards-noncompliant code, but with it, users have to change their previously-compiling, standards-compliant code - The change is a no-op for C++20 and up anyways - There's

[PATCH] D123319: Change how we handle auto return types for lambda operator() to be consistent with gcc

shafik added a comment. > Any update/further details here? David, apologies for not getting back to you sooner. The context is D105564 which I started working on again recently. I was having difficulties finding a solution that also worked for local lambdas.

[PATCH] D124613: In MSVC compatibility mode, friend function declarations behave as function declarations

frederic-tingaud-sonarsource added a comment. Thanks a lot for pointing out the reproducer, it made the debug a lot easier. I have pinpointed the problem to an unintended increase in ADL visibility for friend classes. I do have a test-case and fix for that. I'll make a last check tomorrow to mak

[PATCH] D125165: [Clang] Introduce clang-offload-packager tool to bundle device files

yaxunl added inline comments. Comment at: clang/docs/ClangOffloadPackager.rst:31-32 +uint32_t Flags; +StringMap StringData; +MemoryBufferRef Image; + }; This makes the file format depend on LLVM version and potentially standard C++ library version.

[PATCH] D125165: [Clang] Introduce clang-offload-packager tool to bundle device files

jhuber6 added inline comments. Comment at: clang/docs/ClangOffloadPackager.rst:31-32 +uint32_t Flags; +StringMap StringData; +MemoryBufferRef Image; + }; yaxunl wrote: > This makes the file format depend on LLVM version and potentially standard > C+

[clang] d9c1d3c - [clang][AIX] Don't ignore XCOFF visibility by default

Author: David Tenty Date: 2022-05-11T13:27:48-04:00 New Revision: d9c1d3cbcb9751a6a82cc5e4ada0533cbbc79a1a URL: https://github.com/llvm/llvm-project/commit/d9c1d3cbcb9751a6a82cc5e4ada0533cbbc79a1a DIFF: https://github.com/llvm/llvm-project/commit/d9c1d3cbcb9751a6a82cc5e4ada0533cbbc79a1a.diff L

[PATCH] D125141: [clang][AIX] Don't ignore XCOFF visibility by default

This revision was automatically updated to reflect the committed changes. Closed by commit rGd9c1d3cbcb97: [clang][AIX] Don't ignore XCOFF visibility by default (authored by daltenty). Herald added a project: clang. Herald added a subscriber: cfe-commits. Changed prior to commit: https://review

[PATCH] D124613: In MSVC compatibility mode, friend function declarations behave as function declarations

frederic-tingaud-sonarsource added a comment. In D124613#3506740 , @thakis wrote: > Should we revert this, at least for now, until the investigation is complete? I started typing my message before you sent yours so it was not a direct answer. We are wor

[PATCH] D125403: [Serialization] Delta-encode consecutive SourceLocations in TypeLoc

sammccall created this revision. sammccall added a reviewer: ilya-biryukov. Herald added a subscriber: mgorny. Herald added a project: All. sammccall requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Much of the size of PCH/PCM files comes fr

[PATCH] D125209: [clang-tidy] modernize-deprecated-headers check should respect extern "C" blocks

whisperity added inline comments. Comment at: clang-tools-extra/clang-tidy/modernize/DeprecatedHeadersCheck.cpp:70-73 +auto ExternCBlockBegin = +SM.getDecomposedExpansionLoc(LinkSpecDecl->getBeginLoc()); +auto ExternCBlockEnd = +SM.getDecomposedExpansionL

[PATCH] D121733: Clean pathnames in FileManager.

ilya-biryukov requested changes to this revision. ilya-biryukov added a comment. This revision now requires changes to proceed. Please allow some time for the clangd team to review this before landing. Changes to filenames used to cause unintended consequences for us before. We switched to using

[PATCH] D124613: In MSVC compatibility mode, friend function declarations behave as function declarations

thakis added a comment. It seems to not affect a ton of code, so I don't think there's a huge hurry. I would like to hear an answer about the question "what is this for?" though, and your thoughts on how this relates to the goals in https://docs.google.com/presentation/d/1oxNHaVjA9Gn_rTzX6HIpJH

[PATCH] D125006: [pseudo] Support parsing variant target symbols.

sammccall accepted this revision. sammccall added inline comments. Comment at: clang-tools-extra/pseudo/include/clang-pseudo/Grammar.h:188 +// Name must be a valid nonterminal symbol name in the grammar. +SymbolID findNonterminal(llvm::StringRef Name, + co

[PATCH] D121733: Clean pathnames in FileManager.

kadircet added inline comments. Comment at: clang/lib/Basic/FileManager.cpp:218 + llvm::sys::path::remove_dots(CleanFilename, /*remove_dot_dot=*/false); + Filename = CleanFilename; + this is actually breaking the [contract of FileEntryRef](https://github.com/l

[PATCH] D125395: [analyzer][solver] Handle UnarySymExpr in RangeConstraintSolver

steakhal added a comment. Great content! I've got a long list of nits though. Nothing personal :D Comment at: clang/lib/StaticAnalyzer/Core/RangeConstraintManager.cpp:1453 +const RangeSet *NegatedRange = nullptr; +if (const UnarySymExpr *USE = dyn_cast(Sym)) { + if

[PATCH] D125195: [asan][ARMCXXABI] Added missing asan poison array cookie hooks.

rsundahl updated this revision to Diff 428716. rsundahl added a comment. Implement suggestions from reviews. (Incremental update.) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125195/new/ https://reviews.llvm.org/D125195 Files: clang/lib/CodeGe

Re: [PATCH] D121733: Clean pathnames in FileManager.

Please note that this patch still breaks Winx64 tests, and that I marked it as "further changes required" / not ready for review some time ago. On Wed, May 11, 2022 at 10:37 AM Ilya Biryukov via Phabricator < revi...@reviews.llvm.org> wrote: > ilya-biryukov requested changes to this revision. > i

[PATCH] D121733: Clean pathnames in FileManager.

ppluzhnikov planned changes to this revision. ppluzhnikov added inline comments. Comment at: clang-tools-extra/clangd/index/CanonicalIncludes.cpp:24 + static auto *Mappings = + new std::array, 645>{{ + {"algorithm", ""}, ilya-biryukov wrote: > Don'

[PATCH] D125195: [asan][ARMCXXABI] Added missing asan poison array cookie hooks.

rsundahl updated this revision to Diff 428718. rsundahl added a comment. Revert ItaniumCXXABI.cpp for now (unintentional push) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125195/new/ https://reviews.llvm.org/D125195 Files: clang/lib/CodeGen/It

[PATCH] D125209: [clang-tidy] modernize-deprecated-headers check should respect extern "C" blocks

steakhal updated this revision to Diff 428724. steakhal added a comment. - Ignore `extern "C++"` blocks; also added a test for this. - Also ignore `LinkageSpecDecl`s without braces. - Clear the `IncludesToBeProcessed` list at the end of each translation unit. Repository: rG LLVM Github Monorep

[PATCH] D125165: [Clang] Introduce clang-offload-packager tool to bundle device files

yaxunl added inline comments. Comment at: clang/docs/ClangOffloadPackager.rst:31-32 +uint32_t Flags; +StringMap StringData; +MemoryBufferRef Image; + }; jhuber6 wrote: > yaxunl wrote: > > This makes the file format depend on LLVM version and potentia

[clang] 4dae38e - [HLSL] add -D option for dxc mode.

Author: Xiang Li Date: 2022-05-11T11:26:31-07:00 New Revision: 4dae38ebfba0d8583e52c3ded8f62f5f9fa77fda URL: https://github.com/llvm/llvm-project/commit/4dae38ebfba0d8583e52c3ded8f62f5f9fa77fda DIFF: https://github.com/llvm/llvm-project/commit/4dae38ebfba0d8583e52c3ded8f62f5f9fa77fda.diff LOG:

[PATCH] D125338: [HLSL] add -D option for dxc mode.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG4dae38ebfba0: [HLSL] add -D option for dxc mode. (authored by python3kgae). Changed prior to commit: https://reviews.llvm.org/D125338?vs=428698&id

[PATCH] D125209: [clang-tidy] modernize-deprecated-headers check should respect extern "C" blocks

steakhal marked 2 inline comments as done. steakhal added inline comments. Comment at: clang-tools-extra/clang-tidy/modernize/DeprecatedHeadersCheck.cpp:70-73 +auto ExternCBlockBegin = +SM.getDecomposedExpansionLoc(LinkSpecDecl->getBeginLoc()); +auto ExternCBlock

[clang] 84db355 - [clang] Fix KEYALL

Author: Yaxun (Sam) Liu Date: 2022-05-11T14:28:08-04:00 New Revision: 84db35594953a6f7aff7cbc007f1c5d2fd35b1a9 URL: https://github.com/llvm/llvm-project/commit/84db35594953a6f7aff7cbc007f1c5d2fd35b1a9 DIFF: https://github.com/llvm/llvm-project/commit/84db35594953a6f7aff7cbc007f1c5d2fd35b1a9.dif

[PATCH] D125396: [clang] Fix KEYALL

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG84db35594953: [clang] Fix KEYALL (authored by yaxunl). Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTIO

[PATCH] D125165: [Clang] Introduce clang-offload-packager tool to bundle device files

jhuber6 added inline comments. Comment at: clang/docs/ClangOffloadPackager.rst:31-32 +uint32_t Flags; +StringMap StringData; +MemoryBufferRef Image; + }; yaxunl wrote: > jhuber6 wrote: > > yaxunl wrote: > > > This makes the file format depend on LLVM

[PATCH] D125378: [Attribute] Introduce shuffle attribute to be used for __shfl_sync like cross-lane APIs

aaron.ballman added a comment. In D125378#3506446 , @jdoerfert wrote: > What I would suggest, if you want to use C/C++ attributes (which makes sense > to me), is an attribute that avoids undef or introduces frozen: I would prefer to avoid this approach

[PATCH] D125195: [asan][ARMCXXABI] Added missing asan poison array cookie hooks.

rsundahl updated this revision to Diff 428731. rsundahl marked 6 inline comments as done. rsundahl added a comment. Update diff back to dc7e02d4b4dc30d44ddebd832719a6e63396e718 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125195/new/ https://revie

[PATCH] D125378: [Attribute] Introduce shuffle attribute to be used for __shfl_sync like cross-lane APIs

jdoerfert added a comment. In D125378#3506954 , @aaron.ballman wrote: > In D125378#3506446 , @jdoerfert > wrote: > >> What I would suggest, if you want to use C/C++ attributes (which makes sense >> to me), is a

[PATCH] D125011: [MSVC] Add support for pragma alloc_text

steplong updated this revision to Diff 428727. steplong added a comment. - Reject `#pragma alloc_text(a, a` - On Line 1180, I didn't use `ExpectAndConsume()` there because I wanted to accept both identifiers and strings (i.e `pragma alloc_text(a, foo)` and `pragma alloc_text("a", foo)` - It look

[PATCH] D124701: [clang] Honor __attribute__((no_builtin("foo"))) on functions

aaron.ballman added inline comments. Comment at: clang/lib/CodeGen/CGExpr.cpp:5037 if (auto builtinID = FD->getBuiltinID()) { +std::string AttributeNoBuiltin = "no-builtin-" + FD->getName().str(); std::string FDInlineName = (FD->getName() + ".inline").str(); -

[clang] 3ca6328 - [clang][ppc] Creating Seperate Install Target for PPC htm Headers

Author: Qiongsi Wu Date: 2022-05-11T14:48:40-04:00 New Revision: 3ca6328637b3f42096c652e4df53282649956bdb URL: https://github.com/llvm/llvm-project/commit/3ca6328637b3f42096c652e4df53282649956bdb DIFF: https://github.com/llvm/llvm-project/commit/3ca6328637b3f42096c652e4df53282649956bdb.diff LO

[PATCH] D125386: [clang][ppc] Creating Seperate Install Target for PPC htm Headers

This revision was automatically updated to reflect the committed changes. Closed by commit rG3ca6328637b3: [clang][ppc] Creating Seperate Install Target for PPC htm Headers (authored by qiongsiwu1). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D12538

[PATCH] D124702: [MSVC] Add support for pragma function

steplong added inline comments. Comment at: clang/test/Preprocessor/pragma_microsoft.c:210 +#pragma function(main) // expected-warning {{'main' is not a recognized builtin; consider including }} +#pragma function(// expected-warning {{mi

[PATCH] D124836: [AArch64] Add support for -fzero-call-used-regs

nickdesaulniers added a comment. I still think this would be easier to review if the `isArgumentRegister` tablegen changes were separated out into a distinct parent patch and then the existing x86 implementation updated to use, then this would rebased on top of as a child patch.

[PATCH] D125338: [HLSL] add -D option for dxc mode.

thakis added a comment. Looks like this breaks tests on Mac: http://45.33.8.238/macm1/35103/step_7.txt Please take a look and revert for now if it takes a while to fix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125338/new/ https://reviews.llvm

[PATCH] D125137: [clang-format] Don't remove braces if a 1-statement body would wrap

curdeius accepted this revision. curdeius added a comment. This revision is now accepted and ready to land. LGTM with nits. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:766 + assert(!Tokens.empty()); + const auto *LastToken = Tokens.back().Tok; + assert(LastToken); -

[PATCH] D124836: [AArch64] Add support for -fzero-call-used-regs

peterwaller-arm added inline comments. Comment at: llvm/test/CodeGen/AArch64/zero-call-used-regs.ll:259-262 +; SVE-NEXT:pfalse p0.b +; SVE-NEXT:pfalse p1.b +; SVE-NEXT:pfalse p2.b +; SVE-NEXT:pfalse p3.b nickdesaulniers wrote: > N00b question abou

[PATCH] D125402: [clang][diag] warn if function returns class type by-const-value

lattner edited reviewers, added: rsmith; removed: lattner. lattner added a comment. I'm not a competent reviewer for this, Richard can you recommend someone? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125402/new/ https://reviews.llvm.org/D125402

[PATCH] D125137: [clang-format] Don't remove braces if a 1-statement body would wrap

owenpan added inline comments. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:766 + assert(!Tokens.empty()); + const auto *LastToken = Tokens.back().Tok; + assert(LastToken); curdeius wrote: > It might be a matter of taste but adding this variable makes

[PATCH] D123878: [AMDGPU] Add remarks to output some resource usage

scott.linder added a comment. In D123878#3506500 , @afanfa wrote: > If possible, I would like to keep some kind of delimiter. I like the idea of > having it at the beginning and at the end of the section. The best option > would be to convince clang to

[PATCH] D125137: [clang-format] Don't remove braces if a 1-statement body would wrap

curdeius added inline comments. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:787 + + const int Length = LastToken->TotalLength; + owenpan wrote: > curdeius wrote: > > Why not like this? > > Why not like this? > > See the assertion on line 781 above. We

[PATCH] D113793: Comment Sema: Run checks only when appropriate (NFC)

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

[PATCH] D123538: [symbolizer] Parse DW_TAG_variable DIs to show line info for globals

dblaikie added inline comments. Comment at: llvm/lib/DebugInfo/DWARF/DWARFContext.cpp:1044 + + // Unfortunately, debug_aranges by default don't inclue global variables. If + // we failed to find the CU using aranges, try and search for variables as Might be wor

[PATCH] D125338: [HLSL] add -D option for dxc mode.

python3kgae added a comment. In D125338#3507063 , @thakis wrote: > Looks like this breaks tests on Mac: http://45.33.8.238/macm1/35103/step_7.txt > > Please take a look and revert for now if it takes a while to fix. I'll revert it. Thanks. Repository:

[PATCH] D125414: Revert "[HLSL] add -D option for dxc mode."

python3kgae created this revision. Herald added a subscriber: Anastasia. Herald added a project: All. python3kgae requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. This reverts commit 4dae38ebfba0d8583e52c3ded8f62f5f9fa77fda

[clang] 2db7002 - [AMDGPU] Add llvm.amdgcn.sched.barrier intrinsic

Author: Austin Kerbow Date: 2022-05-11T13:22:51-07:00 New Revision: 2db700215a2eebce7358c0a81a3d52d0a9d4a997 URL: https://github.com/llvm/llvm-project/commit/2db700215a2eebce7358c0a81a3d52d0a9d4a997 DIFF: https://github.com/llvm/llvm-project/commit/2db700215a2eebce7358c0a81a3d52d0a9d4a997.diff

[PATCH] D124700: [AMDGPU] Add llvm.amdgcn.sched.barrier intrinsic

This revision was automatically updated to reflect the committed changes. Closed by commit rG2db700215a2e: [AMDGPU] Add llvm.amdgcn.sched.barrier intrinsic (authored by kerbowa). Herald added a subscriber: kosarev. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews

[PATCH] D124699: [DeadArgElim] Set unused arguments for internal functions

qcolombet added a comment. Thanks @thakis ! @fhahn are you okay with the clang tests update as well? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124699/new/ https://reviews.llvm.org/D124699 ___ cfe-co

[PATCH] D124836: [AArch64] Add support for -fzero-call-used-regs

void updated this revision to Diff 428764. void marked 4 inline comments as done. void added a comment. Fix think-o use of HasSVE. Use `--check-prefixes` in the testcase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124836/new/ https://reviews.llv

[PATCH] D124836: [AArch64] Add support for -fzero-call-used-regs

void added a comment. I'll split off the TableGen changes into a separate patch. It will supersede those changes here, so it shouldn't delay other reviews here. :-) Comment at: llvm/lib/Target/AArch64/AArch64FrameLowering.cpp:798 + + if (STI.hasSVE()) { +for (MCRegister P

[PATCH] D125418: [Arm64EC 6/?] Implement C/C++ mangling for Arm64EC function definitions.

efriedma created this revision. Herald added subscribers: zzheng, kristof.beyls. Herald added a project: All. efriedma requested review of this revision. Herald added a project: clang. Part of initial Arm64EC patchset. For the Arm64EC ABI, ARM64 functions have an alternate name. For C code, this

[PATCH] D124699: [DeadArgElim] Set unused arguments for internal functions

fhahn added a comment. In D124699#3507250 , @qcolombet wrote: > Thanks @thakis ! > > @fhahn are you okay with the clang tests update as well? Yes looks good, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llv

[PATCH] D125419: [Arm64EC 7/?] clang side of Arm64EC varargs ABI.

efriedma created this revision. Herald added subscribers: zzheng, kristof.beyls. Herald added a project: All. efriedma requested review of this revision. Herald added a project: clang. Part of initial Arm64EC patchset. Don't try to duplicate the existing logic; instead, just call the actual code

[clang] 42a1fb5 - [LinkerWrapper][Fix} Fix bad alignment from extracted archive members

Author: Joseph Huber Date: 2022-05-11T16:56:41-04:00 New Revision: 42a1fb5ca56c494e25419a97057a9526f3e8608d URL: https://github.com/llvm/llvm-project/commit/42a1fb5ca56c494e25419a97057a9526f3e8608d DIFF: https://github.com/llvm/llvm-project/commit/42a1fb5ca56c494e25419a97057a9526f3e8608d.diff

[clang] 6055837 - Revert "[HLSL] add -D option for dxc mode."

Author: Xiang Li Date: 2022-05-11T13:57:47-07:00 New Revision: 6055837f6d2941bbc09e44dc88b6a39b934a7453 URL: https://github.com/llvm/llvm-project/commit/6055837f6d2941bbc09e44dc88b6a39b934a7453 DIFF: https://github.com/llvm/llvm-project/commit/6055837f6d2941bbc09e44dc88b6a39b934a7453.diff LOG:

[PATCH] D125414: Revert "[HLSL] add -D option for dxc mode."

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6055837f6d29: Revert "[HLSL] add -D option for dxc mode." (authore

[PATCH] D124836: [AArch64] Add support for -fzero-call-used-regs

nickdesaulniers added a comment. In D124836#3507268 , @void wrote: > I'll split off the TableGen changes into a separate patch. It will supersede > those changes here, so it shouldn't delay other reviews here. :-) I'm referring to the changes to llvm/ut

[clang] 772b0c4 - [OpenMP] Fix mangling for linear parameters with negative stride

Author: Mike Rice Date: 2022-05-11T14:02:09-07:00 New Revision: 772b0c44a4296a34cbc072c2a7cf294410d07a1a URL: https://github.com/llvm/llvm-project/commit/772b0c44a4296a34cbc072c2a7cf294410d07a1a DIFF: https://github.com/llvm/llvm-project/commit/772b0c44a4296a34cbc072c2a7cf294410d07a1a.diff LOG

[PATCH] D125406: [OpenMP] Fix mangling for linear parameters with negative stride

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG772b0c44a429: [OpenMP] Fix mangling for linear parameters with negative stride (authored by mikerice). Herald added a project: clang. Herald added a

[clang] 6398f3f - [clang] Add the flag -ffile-reproducible

Author: Alan Zhao Date: 2022-05-11T23:04:36+02:00 New Revision: 6398f3f2e9045cb38c73425fcc4dddbfb8933a57 URL: https://github.com/llvm/llvm-project/commit/6398f3f2e9045cb38c73425fcc4dddbfb8933a57 DIFF: https://github.com/llvm/llvm-project/commit/6398f3f2e9045cb38c73425fcc4dddbfb8933a57.diff LOG

[PATCH] D122766: [clang] Add the flag -ffile-reproducible

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6398f3f2e904: [clang] Add the flag -ffile-reproducible (authored by ayzhao, committed by hans). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D123878: [AMDGPU] Add remarks to output some resource usage

vangthao added a comment. I am not sure if allowing clang to accept newlines is a good idea. It seems like clang wants to know what type of message is being outputted. For example whether this is a remark, warning, etc. but allowing for a diagnostic to output their own newline makes it ambiguou

[PATCH] D125420: [AArch64] Add support for -fzero-call-used-regs

void created this revision. void added reviewers: nickdesaulniers, nhaehnle, thakis, jankratochvil. Herald added subscribers: pengfei, hiraditya, kristof.beyls. Herald added a project: All. void requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits, MaskRay. Herald

[PATCH] D125420: [AArch64] Add support for -fzero-call-used-regs

void abandoned this revision. void added a comment. Ugh! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125420/new/ https://reviews.llvm.org/D125420 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D125422: Comment parsing: Specify argument numbers for some block commands

aaronpuchert created this revision. aaronpuchert added a reviewer: gribozavr2. Herald added a project: All. aaronpuchert requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The command traits have a member NumArgs for which all the parsing infr

[PATCH] D124836: [AArch64] Add support for -fzero-call-used-regs

void added a comment. In D124836#3507354 , @nickdesaulniers wrote: > In D124836#3507268 , @void wrote: > >> I'll split off the TableGen changes into a separate patch. It will supersede >> those changes here, so

<    1   2   3   >