[PATCH] D87981: [X86] AMX programming model prototype.

pengfei added inline comments. Comment at: clang/lib/Headers/amxintrin.h:69 +#undef __DEFAULT_FN_ATTRS + Better to use `__DEFAULT_FN_ATTRS_TILE`, `__DEFAULT_FN_ATTRS_INT8`, then you don't need to undef it in the middle. Comment at: clang/lib

[clang] 17c8251 - [OpenMP][CUDA][FIX] Use the new `remquo` overload only for OpenMP

Author: Johannes Doerfert Date: 2020-10-27T23:52:59-05:00 New Revision: 17c8251bca83072d2f3e00f936d6ce24500e6b02 URL: https://github.com/llvm/llvm-project/commit/17c8251bca83072d2f3e00f936d6ce24500e6b02 DIFF: https://github.com/llvm/llvm-project/commit/17c8251bca83072d2f3e00f936d6ce24500e6b02.d

[PATCH] D87981: [X86] AMX programming model prototype.

LuoYuanke updated this revision to Diff 301172. LuoYuanke added a comment. Herald added a subscriber: dexonsmith. Rebase and Model tile configure register. Configure register is the user of all the amx instruction. Add X86PreTileConfig pass to insert psuedo ldtilecfg and build the data depenedcy b

[PATCH] D87946: [OpenMP] Add Location Fields to Libomptarget Runtime for Debugging

jhuber6 updated this revision to Diff 301167. jhuber6 added a comment. Updating after D90172 and D89802 landed. I don't see it failing the tests anymore but I'll look into it more. I should probably make a test for the source l

[PATCH] D90099: [Clang][OpenMP][WIP] Added the support for target data nowait

tianshilei1992 updated this revision to Diff 301164. tianshilei1992 added a comment. Fixed a rebase issue Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90099/new/ https://reviews.llvm.org/D90099 Files: clang/lib/CodeGen/CGOpenMPRuntime.cpp cla

Re: [llvm-dev] Upcoming upgrade of LLVM buildbot

Hello Florian, Thanks for pointing to the problem. Appreciate this. It has been fixed and now failed builds reported with the blamelist. Thanks Galina On Wed, Oct 21, 2020 at 5:30 AM Florian Hahn wrote: > > > On Oct 20, 2020, at 03:35, Galina Kistanova via llvm-dev < > llvm-...@lists.llvm.or

Re: Upcoming upgrade of LLVM buildbot

Hello everyone, The LLVM buildbot is upgraded. Thanks for your patience and support! llvm-zorg source code has been updated, and now is open for new changes. If you have pending patches, please rebase them on top of the llvm-zorg master and update your reviews on the Phabricator. I'll start revi

[PATCH] D90129: Source location for -Wignored-qualifiers on lambda trailing return type

aaronpuchert updated this revision to Diff 301161. aaronpuchert marked an inline comment as done. aaronpuchert added a comment. Collect location of a trailing return type in the parser, use that for the warning. I resisted clang-format's urge to reflow the large parameter lists, I hope that's t

[PATCH] D89158: [NewPM] Run all EP callbacks under -O0

ychen added inline comments. Comment at: llvm/lib/Passes/PassBuilder.cpp:1659 + bool DebugLogging) { + for (auto &C : PipelineStartEPCallbacks) +C(MPM); aeubanks wrote: > ychen wrote: > > asbirlea wrote: > > > aeub

[PATCH] D90278: [ThinLTO] Fix .llvmcmd emission

mtrofin created this revision. mtrofin added a reviewer: tejohnson. Herald added subscribers: llvm-commits, cfe-commits, steven_wu, hiraditya, inglorion. Herald added projects: clang, LLVM. mtrofin requested review of this revision. llvm::EmbedBitcodeInModule needs (what used to be called) EmbedM

[PATCH] D89158: [NewPM] Run all EP callbacks under -O0

aeubanks added inline comments. Comment at: llvm/lib/Passes/PassBuilder.cpp:1659 + bool DebugLogging) { + for (auto &C : PipelineStartEPCallbacks) +C(MPM); ychen wrote: > asbirlea wrote: > > aeubanks wrote: > > > y

[PATCH] D89158: [NewPM] Run all EP callbacks under -O0

ychen added inline comments. Comment at: llvm/lib/Passes/PassBuilder.cpp:1659 + bool DebugLogging) { + for (auto &C : PipelineStartEPCallbacks) +C(MPM); asbirlea wrote: > aeubanks wrote: > > ychen wrote: > > > What

[PATCH] D88603: [WebAssembly] Add support for DWARF type units

dschuff added a comment. This broke the bots for some strange reason that didn't reproduce locally. But because it was ~all of them, I probably just did something stupid. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88603/new/ https://reviews.llv

[clang] 44eea0b - Revert "[WebAssembly] Add support for DWARF type units"

Author: Derek Schuff Date: 2020-10-27T17:57:32-07:00 New Revision: 44eea0b1a7301de4c284afed1c023014c391372f URL: https://github.com/llvm/llvm-project/commit/44eea0b1a7301de4c284afed1c023014c391372f DIFF: https://github.com/llvm/llvm-project/commit/44eea0b1a7301de4c284afed1c023014c391372f.diff

[PATCH] D90275: [clang][IR] Add support for leaf attribute

leonardchan added a comment. I'd recommend splitting this into two patches, one for LLVM changes and another for Clang changes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90275/new/ https://reviews.llvm.org/D90275 _

[PATCH] D87528: Enable '#pragma STDC FENV_ACCESS' in frontend

pengfei added a comment. I agreed with Craig. Emitting constrained intrinsics on an unsupported target may result in problems. It's better to check if it is supported and prevent from the front end. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87

Re: [clang] e97e985 - [c++20] For P0732R2: permit class template argument deduction for non-type template parameters.

On Tue, 27 Oct 2020 at 17:08, Nico Weber wrote: > gcc points out that DTST is unused in > > +if (auto *DTST = > dyn_cast(DeducedT)) { > > Should it be used? Or should we delete the lhs of that assignment? > The latter. Done in af2f5f99bd144ae2f70e21d53b23fc18a3d9d7d0. > On Wed, Oct 21, 202

[PATCH] D90099: [Clang][OpenMP][WIP] Added the support for target data nowait

tianshilei1992 updated this revision to Diff 301151. tianshilei1992 added a comment. Fixed a rebase error Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90099/new/ https://reviews.llvm.org/D90099 Files: clang/lib/CodeGen/CGOpenMPRuntime.cpp cla

[clang] af2f5f9 - Remove unused variable name. NFC.

Author: Richard Smith Date: 2020-10-27T17:48:50-07:00 New Revision: af2f5f99bd144ae2f70e21d53b23fc18a3d9d7d0 URL: https://github.com/llvm/llvm-project/commit/af2f5f99bd144ae2f70e21d53b23fc18a3d9d7d0 DIFF: https://github.com/llvm/llvm-project/commit/af2f5f99bd144ae2f70e21d53b23fc18a3d9d7d0.diff

[PATCH] D90099: [Clang][OpenMP][WIP] Added the support for target data nowait

tianshilei1992 updated this revision to Diff 301150. tianshilei1992 added a comment. Rebased and fixed one test case Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90099/new/ https://reviews.llvm.org/D90099 Files: clang/lib/CodeGen/CGOpenMPRuntim

[PATCH] D90275: [clang][IR] Add support for leaf attribute

jdoerfert added a comment. In D90275#2357951 , @gulfem wrote: > In D90275#2357945 , @jdoerfert wrote: > >> In D90275#2357931 , @mcgrathr wrote: >> >>> There is an RFC going o

[PATCH] D90275: [clang][IR] Add support for leaf attribute

gulfem added a comment. In D90275#2357945 , @jdoerfert wrote: > In D90275#2357931 , @mcgrathr wrote: > >> There is an RFC going out with this prototype as reference. When there is >> consensus on the RFC, this wil

[PATCH] D89488: FileManager: Shrink FileEntryRef to the size of a pointer

dexonsmith added inline comments. Comment at: clang/include/clang/Basic/FileManager.h:101 + /// Type used in the StringMap. + using MapEntry = llvm::StringMapEntry>; + thakis wrote: > thakis wrote: > > It looks like this is too clever for gcc5.3: > > https://b

[PATCH] D89158: [NewPM] Run all EP callbacks under -O0

asbirlea added inline comments. Comment at: llvm/lib/Passes/PassBuilder.cpp:1659 + bool DebugLogging) { + for (auto &C : PipelineStartEPCallbacks) +C(MPM); aeubanks wrote: > ychen wrote: > > What I have in mind is

[PATCH] D90275: [clang][IR] Add support for leaf attribute

jdoerfert added a comment. In D90275#2357931 , @mcgrathr wrote: > There is an RFC going out with this prototype as reference. When there is > consensus on the RFC, this will get in shape for landing with complete tests > and all. Great. I didn't find a

[PATCH] D90053: Serialization: Change InputFile to use FileEntryRef and add getVirtualFileRef, NFC

dexonsmith updated this revision to Diff 301148. dexonsmith edited the summary of this revision. dexonsmith added a comment. Rebase. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90053/new/ https://reviews.llvm.org/D90053 Files: clang/include/clang/Basic/FileManager.h clang/include

[PATCH] D89158: [NewPM] Run all EP callbacks under -O0

aeubanks marked an inline comment as done. aeubanks added inline comments. Comment at: llvm/lib/Passes/PassBuilder.cpp:1659 + bool DebugLogging) { + for (auto &C : PipelineStartEPCallbacks) +C(MPM); ychen wrote: >

[PATCH] D85810: [clang] Pass-through remarks options to linker

weiwang closed this revision. weiwang added a comment. Diff was committed, but did not close automatically. Manual close it now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85810/new/ https://reviews.llvm.org/D85810 _

[PATCH] D90253: [WebAssembly] Prototype extending multiplication SIMD instructions

tlively updated this revision to Diff 301147. tlively added a comment. - Use V8 encodings: https://chromium-review.googlesource.com/c/v8/v8/+/2469156/4/src/wasm/wasm-opcodes.h Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90253/new/ https://review

[PATCH] D89836: Change Module::ASTFile and ModuleFile::File => Optional, NFC

dexonsmith updated this revision to Diff 301146. dexonsmith retitled this revision from "Change Module::ASTFile and ModuleFile::File => MaybeFileEntryRef, NFC" to "Change Module::ASTFile and ModuleFile::File => Optional, NFC". dexonsmith edited the summary of this revision. dexonsmith added a com

[PATCH] D90275: [clang][IR] Add support for leaf attribute

mcgrathr added a comment. There is an RFC going out with this prototype as reference. When there is consensus on the RFC, this will get in shape for landing with complete tests and all. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90275/new/ ht

[PATCH] D89158: [NewPM] Run all EP callbacks under -O0

aeubanks updated this revision to Diff 301145. aeubanks added a comment. run -> invoke Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89158/new/ https://reviews.llvm.org/D89158 Files: clang/lib/CodeGen/BackendUtil.cpp clang/test/CodeGen/bpf-O0.

[PATCH] D90275: [clang][IR] Add support for leaf attribute

jdoerfert added a comment. (To clarify, I am not opposing this, my comment before was brief but not meant to be negative. I'm generally for new attributes and this one is already in existing source so that is even better.) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION http

[clang] c486870 - [clang] Pass-through remarks options to linker

Author: Wei Wang Date: 2020-10-27T17:23:32-07:00 New Revision: c4868700c58078d0f4c71fab5af2bb73270d2d9f URL: https://github.com/llvm/llvm-project/commit/c4868700c58078d0f4c71fab5af2bb73270d2d9f DIFF: https://github.com/llvm/llvm-project/commit/c4868700c58078d0f4c71fab5af2bb73270d2d9f.diff LOG:

[PATCH] D90275: [clang][IR] Add support for leaf attribute

jdoerfert added a comment. Tests missing (on all levels), was there an RFC? What is the expected usage of this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90275/new/ https://reviews.llvm.org/D90275 _

[PATCH] D89834: FileManager: Improve the FileEntryRef API and customize Optional

dexonsmith updated this revision to Diff 301138. dexonsmith retitled this revision from "FileManager: Improve the FileEntryRef API and add MaybeFileEntryRef" to "FileManager: Improve the FileEntryRef API and customize Optional". dexonsmith edited the summary of this revision. dexonsmith added a c

[PATCH] D90275: [clang][IR] Add support for leaf attribute

gulfem created this revision. Herald added subscribers: llvm-commits, cfe-commits, dexonsmith, jdoerfert, steven_wu, hiraditya. Herald added a reviewer: aaron.ballman. Herald added projects: clang, LLVM. gulfem requested review of this revision. GCC supports __attribute__((leaf)) as an optimizati

[PATCH] D85810: [clang] Pass-through remarks options to linker

weiwang updated this revision to Diff 301140. weiwang added a comment. fix typo Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85810/new/ https://reviews.llvm.org/D85810 Files: clang/lib/Driver/ToolChains/CommonArgs.cpp clang/test/Driver/opt-re

[PATCH] D89158: [NewPM] Run all EP callbacks under -O0

ychen added inline comments. Comment at: llvm/include/llvm/Passes/PassBuilder.h:626 + /// typically run at the very beginning. + void runRegisteredO0EPCallbacks(ModulePassManager &MPM, bool DebugLogging); + invokeO0EPCallbacks? I saw there is `PassBuilder::invo

[PATCH] D88603: [WebAssembly] Add support for DWARF type units

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGbcb8a119df21: [WebAssembly] Add support for DWARF type units (authored by dschuff). Changed prior to commit: https://reviews.llvm.org/D88603?vs=30

[clang] bcb8a11 - [WebAssembly] Add support for DWARF type units

Author: Derek Schuff Date: 2020-10-27T17:13:41-07:00 New Revision: bcb8a119df210753c5f1a3ac346d49597fef0f51 URL: https://github.com/llvm/llvm-project/commit/bcb8a119df210753c5f1a3ac346d49597fef0f51 DIFF: https://github.com/llvm/llvm-project/commit/bcb8a119df210753c5f1a3ac346d49597fef0f51.diff

[PATCH] D89971: [OpenMP][CUDA] Add missing overload for `remquo(float,float,int*)`

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGb1a90e1599a6: [OpenMP][CUDA] Add missing overload for `remquo(float,float,int*)` (authored by jdoerfert). Repository: rG LLVM Github Monorepo CHA

[clang] b1a90e1 - [OpenMP][CUDA] Add missing overload for `remquo(float,float,int*)`

Author: Johannes Doerfert Date: 2020-10-27T19:12:51-05:00 New Revision: b1a90e1599a65bccf9af985603e198af3876b59a URL: https://github.com/llvm/llvm-project/commit/b1a90e1599a65bccf9af985603e198af3876b59a DIFF: https://github.com/llvm/llvm-project/commit/b1a90e1599a65bccf9af985603e198af3876b59a.d

[clang] 739d4bf - Unbreak build with gcc5.3 after 917acac

Author: Nico Weber Date: 2020-10-27T20:10:48-04:00 New Revision: 739d4bf8f43d0a5a3a99e07b1647ded1ce1d81b3 URL: https://github.com/llvm/llvm-project/commit/739d4bf8f43d0a5a3a99e07b1647ded1ce1d81b3 DIFF: https://github.com/llvm/llvm-project/commit/739d4bf8f43d0a5a3a99e07b1647ded1ce1d81b3.diff LO

[PATCH] D89158: [NewPM] Run all EP callbacks under -O0

aeubanks added a comment. In D89158#2357850 , @ychen wrote: > IIUC, this is the NPM version of EP_EarlyAsPossible in legacy PM for O0, > right? Between the choice of (1) [this patch and ] reusing all existing EP > callbacks and letting optnone filtering

[PATCH] D88603: [WebAssembly] Add support for DWARF type units

dschuff updated this revision to Diff 301133. dschuff added a comment. use GenericSectionID instead of ~0 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88603/new/ https://reviews.llvm.org/D88603 Files: llvm/lib/MC/MCObjectFileInfo.cpp Index: l

Re: [clang] e97e985 - [c++20] For P0732R2: permit class template argument deduction for non-type template parameters.

gcc points out that DTST is unused in +if (auto *DTST = dyn_cast(DeducedT)) { Should it be used? Or should we delete the lhs of that assignment? On Wed, Oct 21, 2020 at 6:03 PM Richard Smith via cfe-commits < cfe-commits@lists.llvm.org> wrote: > > Author: Richard Smith > Date: 2020-10-21T15

[PATCH] D89743: Support Attr in DynTypedNode and ASTMatchers.

njames93 added inline comments. Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:6763 +/// (but not "nodiscard" or "clang::warn_unused_result"). +AST_MATCHER_P(Attr, hasAttrName, llvm::StringRef, Name) { + return Node.getAttrName() && Node.getAttrName()->isStr(Name); --

[PATCH] D89158: [NewPM] Run all EP callbacks under -O0

aeubanks updated this revision to Diff 301132. aeubanks added a comment. only run passes in PipelineStartEPCallbacks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89158/new/ https://reviews.llvm.org/D89158 Files: clang/lib/CodeGen/BackendUtil.cp

[PATCH] D89158: [NewPM] Run all EP callbacks under -O0

ychen added a comment. IIUC, this is the NPM version of EP_EarlyAsPossible in legacy PM for O0, right? Between the choice of (1) [this patch and ] reusing all existing EP callbacks and letting optnone filtering out non-required passes (which I think does not work for `-disable-O0-optnone`) and

[PATCH] D89488: FileManager: Shrink FileEntryRef to the size of a pointer

thakis added inline comments. Comment at: clang/include/clang/Basic/FileManager.h:101 + /// Type used in the StringMap. + using MapEntry = llvm::StringMapEntry>; + thakis wrote: > It looks like this is too clever for gcc5.3: > https://bugs.chromium.org/p/chrom

Re: [Lldb-commits] Upcoming upgrade of LLVM buildbot

Hello Vitaly, This should be fixed. Please let me know if you still see the issue. Thanks Galina On Tue, Oct 27, 2020 at 4:45 PM Vitaly Buka wrote: > It looks fixed. I haven't seen them in recent reports. > > On Tue, 27 Oct 2020 at 16:32, Vitaly Buka wrote: > >> Any news regarding false-n

[PATCH] D90271: [clang][ToolChains] explicitly return LangOptions::StackProtectorMode

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

Re: [Lldb-commits] Upcoming upgrade of LLVM buildbot

It looks fixed. I haven't seen them in recent reports. On Tue, 27 Oct 2020 at 16:32, Vitaly Buka wrote: > Any news regarding false-negatives? > > +Mitch Phillips > > On Mon, 19 Oct 2020 at 22:07, Galina Kistanova > wrote: > >> Thanks, Vitaly. >> Somebody is looking at this. >> >> >> On Mon, Oc

[PATCH] D90109: [clang-tidy] Use ANSI escape codes for --use-color on Windows

dblaikie added a comment. In D90109#2356317 , @aaron.ballman wrote: > In D90109#2352180 , @dsanders11 > wrote: > >> Added a few inline comments for clarification. >> >> **Note**: I was not able to get a debug buil

Re: [Lldb-commits] Upcoming upgrade of LLVM buildbot

Any news regarding false-negatives? +Mitch Phillips On Mon, 19 Oct 2020 at 22:07, Galina Kistanova wrote: > Thanks, Vitaly. > Somebody is looking at this. > > > On Mon, Oct 19, 2020 at 9:56 PM Vitaly Buka wrote: > >> it's not just staging, this build must be marked as failed >> http://lab.llv

[PATCH] D89488: FileManager: Shrink FileEntryRef to the size of a pointer

thakis added inline comments. Herald added a subscriber: ormris. Comment at: clang/include/clang/Basic/FileManager.h:101 + /// Type used in the StringMap. + using MapEntry = llvm::StringMapEntry>; + It looks like this is too clever for gcc5.3: https://bugs.chr

[PATCH] D90194: [Driver] differentiate -stack-protector 0 from being unspecified

nickdesaulniers planned changes to this revision. nickdesaulniers added a comment. I split out the first logical change into https://reviews.llvm.org/D90271. I think I'll rework this to both be rebased, and to not add the new enum value. Repository: rG LLVM Github Monorepo CHANGES SINCE LAS

[PATCH] D90271: [clang][ToolChains] explicitly return LangOptions::StackProtectorMode

nickdesaulniers created this revision. nickdesaulniers added reviewers: void, rnk, phosek, srhines. Herald added a project: clang. Herald added a subscriber: cfe-commits. nickdesaulniers requested review of this revision. Make the virtual method Toolchain::GetDefaultStackProtectorLevel() return an

[PATCH] D90188: Add support for attribute 'using_if_exists'

rsmith added a comment. In addition to moving the diagnostic to `DiagnoseUseOfDecl`, you may well get better error recovery if you teach `ClassifyName` about the new kind of declaration, and have it `DiagnoseUseOfDecl` it immediately and return `NameClassification::Error()` -- that way, the par

[PATCH] D88239: [clang-format] Fix spaces around */& in multi-variable declarations

djasper added a comment. Sorry for being a bit late here and thanks @klimek for bringing this to my attention. This has been years ago, but if I reconstruct my thinking (and look at the test cases), then I'd say that "left" alignment should not be applied to multi-variable decl statements ever.

[PATCH] D90270: [clangd] Handle absolute/relative path specifications in Config

kadircet created this revision. kadircet added a reviewer: sammccall. Herald added subscribers: cfe-commits, usaxena95, arphaman. Herald added a project: clang. kadircet requested review of this revision. Herald added subscribers: ormris, MaskRay, ilya-biryukov. This introduces a mechanism for pro

[PATCH] D83448: [CodeGen] Emit destructor calls to destruct non-trivial C struct temporaries created by conditional and assignment operators

dblaikie added a comment. In D83448#2357574 , @ahatanak wrote: > C structs with ObjC pointer fields are non-trivial when ARC is enabled since > construction, destruction, and copy are non-trivial (see > https://reviews.llvm.org/D41228). As far as I know,

[PATCH] D83448: [CodeGen] Emit destructor calls to destruct non-trivial C struct temporaries created by conditional and assignment operators

ahatanak added a comment. C structs with ObjC pointer fields are non-trivial when ARC is enabled since construction, destruction, and copy are non-trivial (see https://reviews.llvm.org/D41228). As far as I know, this is limited to ObjC. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

Re: [PATCH] D89500: Fix the error message with -fbasic-block-sections=list=

On Tue, Oct 27, 2020 at 2:14 PM David Blaikie via Phabricator < revi...@reviews.llvm.org> wrote: > dblaikie added a comment. > > @tmsriram ping on the follow-up here > I checked in the patch that emits llvm instead of obj which spews garbage to the terminal as I wasn't redirecting it to /dev/null

[PATCH] D89500: Fix the error message with -fbasic-block-sections=list=

dblaikie added a comment. @tmsriram ping on the follow-up here Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89500/new/ https://reviews.llvm.org/D89500 ___ cfe-commits mailing list cfe-commits@lists.llvm

[PATCH] D83448: [CodeGen] Emit destructor calls to destruct non-trivial C struct temporaries created by conditional and assignment operators

dblaikie added a comment. Idle curiosity: The patch description says "non-trivial C struct temporaries" - can C have non-trivial structs? Or is this limited to Objective C (judging/guessing by the tests)? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D89794: [SyntaxTree] Implement "by-pointer output parameter to return value" refactoring.

dblaikie added a comment. I imagine test coverage would be handy (I'm probably not the right reviewer for this in general, but was/am curious to see how it goes - you might want to search for other contributors/reviewers to the code you're changing and request/add them as reviewers for this pat

[PATCH] D90244: [clang-tidy][NFC] IdentifierNaming: Remove unnecessary string allocations

njames93 added a comment. In D90244#2357130 , @aaron.ballman wrote: > Was this caused by a performance concern when profiling something? I'm not > opposed to the changes, but I do think the original formulation is easier to > read. It's not a huge perf

[PATCH] D89893: [Clang] [TableGen] Clean up !if(!eq(boolean, 1) and related booleans

dblaikie added a comment. In D89893#2357480 , @Paul-C-Anagnostopoulos wrote: > I'll remember this, too. > > @dblaikie How did you happen to notice? Reading the mailing list & seeing the phab review and commit without any emails from anyone else in the r

[PATCH] D90054: Added remotely ran compiler-rt tests.

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

[PATCH] D89893: [Clang] [TableGen] Clean up !if(!eq(boolean, 1) and related booleans

Paul-C-Anagnostopoulos added a comment. I'll remember this, too. @dblaikie How did you happen to notice? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89893/new/ https://reviews.llvm.org/D89893 ___ cfe-

[PATCH] D90188: Add support for attribute 'using_if_exists'

Mordante added a comment. A few questions. I'm not familiar enough with the code to accept the patch. Comment at: clang/include/clang/Basic/AttrDocs.td:5266 + let Content = [{ +The using_if_exists attribute applies to a using-declaration. It allows +programmers to import a dec

[PATCH] D89998: [c++20] For P0732R2 / P1907R1: Basic code generation and name mangling supportfor non-type template parameters of class type and template parameter objects.

rjmccall added inline comments. Comment at: clang/lib/AST/APValue.cpp:1047 + if (MergeLV(getLVForType(*TI.getType(), computation))) +break; +} else if (const Expr *E = V.getLValueBase().dyn_cast()) { I'm not sure what ABIs you're thinking about,

[PATCH] D89899: [CodeGen] Implement [[likely]] and [[unlikely]] for the iteration statements

aaron.ballman added inline comments. Comment at: clang/include/clang/Basic/AttrDocs.td:1866 + while(true) [[unlikely]] { +... // The attribute has no effect + } // Clang elides comparison and generates an infinite loop Mordan

[clang] 3700556 - [HIP][NFC] Use correct max in cuda_complex_builtins

Author: Aaron En Ye Shi Date: 2020-10-27T19:35:09Z New Revision: 3700556ecb062abf786ad52a5474c27b22c19107 URL: https://github.com/llvm/llvm-project/commit/3700556ecb062abf786ad52a5474c27b22c19107 DIFF: https://github.com/llvm/llvm-project/commit/3700556ecb062abf786ad52a5474c27b22c19107.diff LO

[PATCH] D89899: [CodeGen] Implement [[likely]] and [[unlikely]] for the iteration statements

Mordante marked 10 inline comments as done. Mordante added a comment. Thanks for the feedback. I'll update the patch after making the requested changes. Comment at: clang/include/clang/Basic/AttrDocs.td:1866 + while(true) [[unlikely]] { +... // The attribute

[clang] e025d09 - Revert multiple patches based on "Introduce CfgTraits abstraction"

Author: Nicolai Hähnle Date: 2020-10-27T20:33:30+01:00 New Revision: e025d09b216dc2239e1b502f4f277abb6fb4648a URL: https://github.com/llvm/llvm-project/commit/e025d09b216dc2239e1b502f4f277abb6fb4648a DIFF: https://github.com/llvm/llvm-project/commit/e025d09b216dc2239e1b502f4f277abb6fb4648a.diff

[PATCH] D90188: Add support for attribute 'using_if_exists'

erik.pilkington added inline comments. Comment at: clang/test/SemaCXX/using-if-exists-attr.cpp:16 +template +using template_alias UIE = NS::x; // expected-warning {{'using_if_exists' attribute only applies to named declarations, types, and value declarations}} + ---

[PATCH] D89072: [CodeView] Emit static data members as S_CONSTANTs.

akhuang updated this revision to Diff 301080. akhuang added a comment. Herald added a subscriber: jfb. Moved isUnsignedDIType to DebugHandlerBase to avoid copying it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89072/new/ https://reviews.llvm.org

[PATCH] D89072: [CodeView] Emit static data members as S_CONSTANTs.

akhuang reopened this revision. akhuang added a comment. This revision is now accepted and ready to land. Reverted because my copied `isUnsignedDIType` function had problems (guess I copied it at first but then was messing with the code); I moved it to DebugHandlerBase.h so it doesn't need to be

[PATCH] D90188: Add support for attribute 'using_if_exists'

ldionne added inline comments. Comment at: clang/test/SemaCXX/using-if-exists-attr.cpp:16 +template +using template_alias UIE = NS::x; // expected-warning {{'using_if_exists' attribute only applies to named declarations, types, and value declarations}} + I assu

[PATCH] D89893: [Clang] [TableGen] Clean up !if(!eq(boolean, 1) and related booleans

lattner added a comment. Oops, thanks for the head's up @dblaikie ! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89893/new/ https://reviews.llvm.org/D89893 ___ cfe-commits mailing list cfe-commits@lists

[clang] 917acac - FileManager: Shrink FileEntryRef to the size of a pointer

Author: Duncan P. N. Exon Smith Date: 2020-10-27T14:55:41-04:00 New Revision: 917acac960d40280ea02ea453e594034b1be1f6b URL: https://github.com/llvm/llvm-project/commit/917acac960d40280ea02ea453e594034b1be1f6b DIFF: https://github.com/llvm/llvm-project/commit/917acac960d40280ea02ea453e594034b1be

[PATCH] D89488: FileManager: Shrink FileEntryRef to the size of a pointer

This revision was automatically updated to reflect the committed changes. dexonsmith marked 2 inline comments as done. Closed by commit rG917acac960d4: FileManager: Shrink FileEntryRef to the size of a pointer (authored by dexonsmith). Herald added a project: clang. Changed prior to commit: htt

[PATCH] D89998: [c++20] For P0732R2 / P1907R1: Basic code generation and name mangling supportfor non-type template parameters of class type and template parameter objects.

rsmith added inline comments. Comment at: clang/lib/AST/MicrosoftMangle.cpp:657-660 // ::= $1? // ::= $H? // ::= $I? // ::= $J? For what it's worth, I'm fairly co

[PATCH] D90180: [clang-tidy] find/fix unneeded semicolon after switch

aaron.ballman added a comment. In D90180#2357178 , @trixirt wrote: > Discussion of change on LKML > https://lkml.org/lkml/2020/10/27/2538 > > On why the existing clang fixit is not practical. > tl;dr > 10,000 changes to look for a treewide fix > The fixit

[PATCH] D89893: [Clang] [TableGen] Clean up !if(!eq(boolean, 1) and related booleans

dblaikie added subscribers: mehdi_amini, dblaikie. dblaikie added a comment. @lattner minor detail/annoyance: Phabricator doesn't email the mailing lists if a review is "approved" with no comments. To ensure the approval is recorded on the list, please include some text (a stub "." would suffice

[PATCH] D89998: [c++20] For P0732R2 / P1907R1: Basic code generation and name mangling support for non-type template parameters of class type and template parameter objects.

rsmith updated this revision to Diff 301070. rsmith added a comment. - Ignore cv-qualifiers in member types for Itanium mangling. - Add MSVC-compatible mangling support. - Fix bug where template argument value mangling would be used when passing a template parameter object as a pointer or r

[PATCH] D89936: [clang-tidy] adding "--clang-tidy-config=" to specify custom config file

Hiralo added a comment. In D89936#2357088 , @njames93 wrote: > If you plan on contributing quite a lot then it would be wise to upload your > patches with arcanist - https://llvm.org/docs/Phabricator.html. It will help > prevent issues with diffs being r

[clang] 5046153 - Revert "[CodeView] Emit static data members as S_CONSTANTs."

Author: Amy Huang Date: 2020-10-27T11:29:58-07:00 New Revision: 504615353f31136dd6bf7a971b6c236fd70582be URL: https://github.com/llvm/llvm-project/commit/504615353f31136dd6bf7a971b6c236fd70582be DIFF: https://github.com/llvm/llvm-project/commit/504615353f31136dd6bf7a971b6c236fd70582be.diff LOG

[PATCH] D89936: [clang-tidy] adding "--clang-tidy-config=" to specify custom config file

Hiralo marked 4 inline comments as done. Hiralo added inline comments. Comment at: clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp:290 DefaultOptions.Checks = DefaultChecks; + DefaultOptions.ConfigFile = ""; DefaultOptions.WarningsAsErrors = ""; njames

[PATCH] D89980: [hip] Remove kernel argument coercion.

hliao added a comment. Besides the unpromotable `alloca` issue due to indirect accesses, such coercion to GLOBAL pointer directly is not safe as, in HIP/CUDA, both CONSTANT and GLOBAL pointers would be passed as the kernel arguments. Without introducing a new address space combing GLOBAL/CONSTA

[PATCH] D89936: [clang-tidy] adding "--clang-tidy-config=" to specify custom config file

Hiralo updated this revision to Diff 301065. Hiralo added a comment. Latest patch with Lit test-case. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89936/new/ https://reviews.llvm.org/D89936 Files: clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp clang-tools-extra/test/clang-ti

[PATCH] D88381: [Flang][Driver] Add PrintPreprocessed FrontendAction

awarzynski updated this revision to Diff 301066. awarzynski added a comment. Rebase on top of master + rename accessor method Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88381/new/ https://reviews.llvm.org/D88381 Files: clang/include/clang/Dri

[PATCH] D90174: [HIP] Fix regressions due to fp contract change

rjmccall added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:486 + if (LangOpts.HIP) +Options.AllowFPOpFusion = llvm::FPOpFusion::Standard; + tra wrote: > yaxunl wrote: > > tra wrote: > > > yaxunl wrote: > > > > tra wrote: > > > > > I don't

[PATCH] D90180: [clang-tidy] find/fix unneeded semicolon after switch

Eugene.Zelenko added a comment. In D90180#2357178 , @trixirt wrote: > Discussion of change on LKML > https://lkml.org/lkml/2020/10/27/2538 > > On why the existing clang fixit is not practical. > tl;dr > 10,000 changes to look for a treewide fix > The fixi

[PATCH] D90180: [clang-tidy] find/fix unneeded semicolon after switch

trixirt added a comment. Discussion of change on LKML https://lkml.org/lkml/2020/10/27/2538 On why the existing clang fixit is not practical. tl;dr 10,000 changes to look for a treewide fix The fixit has whitespace issue Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https:

[PATCH] D39086: Performance tracing facility for clangd.

RKSimon added inline comments. Herald added subscribers: usaxena95, kadircet, arphaman, MaskRay. Comment at: clang-tools-extra/trunk/unittests/clangd/TraceTests.cpp:57 +} +std::string VS = V->getValue(Tmp).str(); +if (VS != I->second) { @sammccall PVS

[PATCH] D89980: [hip] Remove kernel argument coercion.

hliao added inline comments. Comment at: clang/test/CodeGenCUDA/amdgpu-kernel-arg-pointer-type.cu:19 +// COMMON-LABEL: define amdgpu_kernel void @_Z7kernel1Pi(i32*{{.*}} %x) +// OPT: [[VAL:%.*]] = load i32, i32* %x, align 4 // OPT: [[INC:%.*]] = add nsw i32 [[VAL]], 1 --

  1   2   3   >