[clang-tools-extra] 4782ccd - [test][clangd] Try to deflake with sanitizers

Author: Vitaly Buka Date: 2022-09-26T22:52:48-07:00 New Revision: 4782ccde2e1b44ea44a8e83cd1e7041fa31c5c18 URL: https://github.com/llvm/llvm-project/commit/4782ccde2e1b44ea44a8e83cd1e7041fa31c5c18 DIFF: https://github.com/llvm/llvm-project/commit/4782ccde2e1b44ea44a8e83cd1e7041fa31c5c18.diff

[PATCH] D134042: [clang-format] Fix alignment in #else preprocessor blocks

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

[PATCH] D134626: [clang-format] Correctly indent closing brace of compound requires

MyDeveloperDay added a comment. Hmm…it’s nice when it’s this simple, but I get an uneasy feeling about the knock on effects. I’d say go for it but perhaps we could run it over a larger code base to see the implications Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D134700: [clang-format] Fix a bug with C++ `export import `

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

[PATCH] D133863: [RISCV] Add MC support of RISCV zcmt Extension

VincentWu updated this revision to Diff 463097. VincentWu marked 5 inline comments as done. VincentWu added a comment. update & address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133863/new/ https://reviews.llvm.org/D133863 Files:

[PATCH] D134669: [clang][msan] Turn on -fsanitize-memory-param-retval by default

aeubanks updated this revision to Diff 463096. aeubanks added a comment. Herald added subscribers: pcwang-thead, s.egerton, simoncook. update test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134669/new/ https://reviews.llvm.org/D134669 Files:

[PATCH] D134523: [clang][Interp] Fix copy constructors with record array members

tbaeder updated this revision to Diff 463094. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134523/new/ https://reviews.llvm.org/D134523 Files: clang/lib/AST/Interp/ByteCodeExprGen.cpp clang/lib/AST/Interp/Interp.h clang/lib/AST/Interp/Pointer.cpp

[PATCH] D134702: [Clang] Don't warn if deferencing void pointers in unevaluated context

junaire created this revision. junaire added reviewers: aaron.ballman, nathanchance, nickdesaulniers. Herald added a project: All. junaire requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. After https://reviews.llvm.org/D134461, Clang will

[PATCH] D134700: [clang-format] Fix a bug with C++ `export import `

owenpan created this revision. owenpan added reviewers: MyDeveloperDay, HazardyKnusperkeks, curdeius. owenpan added a project: clang-format. Herald added a project: All. owenpan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fixes

[PATCH] D134057: [clang][Interp] Start implementing record types

tbaeder marked 2 inline comments as done. tbaeder added inline comments. Comment at: clang/lib/AST/Interp/ByteCodeExprGen.cpp:689 +const Decl *Callee = CE->getCalleeDecl(); +const Function *Func = P.getFunction(dyn_cast(Callee)); + aaron.ballman wrote: >

[PATCH] D134699: [clang][Interp] Implement This pointer passing to methods

tbaeder created this revision. tbaeder added reviewers: aaron.ballman, erichkeane, tahonermann, shafik. Herald added a project: All. tbaeder requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Seems like I completely forgot to post this to

[PATCH] D134410: [clang][CodeGen] Add noundef metadata to load instructions (preliminary)

vitalybuka added a comment. I tried to hook this patch into MemorySanitizer and it reduces instrumented code by ~30% ! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134410/new/ https://reviews.llvm.org/D134410

[PATCH] D134410: [clang][CodeGen] Add noundef metadata to load instructions (preliminary)

vitalybuka added a comment. Are there patches uploaded with arc tool? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134410/new/ https://reviews.llvm.org/D134410 ___ cfe-commits mailing list

[PATCH] D134461: [Clang] Warn when trying to deferencing void pointers in C

junaire added a comment. Sorry for missing the conversation, I was in sleeping mode at that time ;D > This warning is quite noisy for the Linux kernel due to a couple of places > where a void * is dereferenced as part of compile time checking. I'm actually surprised to see that, I thought

[PATCH] D131172: [clang][llvm][doc] Add more information for the ABI change in FP16

pengfei added inline comments. Comment at: clang/docs/ReleaseNotes.rst:633 +- If you are using downstream runtimes that provide FP16 conversions, update +them with the new ABI. + mehdi_amini wrote: > Can you add a link to a documentation that describe the new

[PATCH] D134685: Fix SourceManager::isBeforeInTranslationUnit bug with token-pasting

sammccall added a comment. > However I tried to avoid mixing these with subtle behavior changes, and will > send a followup instead. D134694 if you're interested. I guess I should try to get performance measures though... Repository: rG LLVM Github

[PATCH] D134694: Clean up isBeforeInTranslationUnit cache and add eviction.

sammccall created this revision. Herald added a project: All. sammccall requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Previously the first 300 entries would stay around forever, and any new queries would thrash on entry 301. The cache

[PATCH] D134693: [CMake] Add `CLANG_ENABLE_HLSL` CMake option

beanz created this revision. beanz added reviewers: phosek, smeenai, compnerd. Herald added a subscriber: Anastasia. Herald added a project: All. beanz requested review of this revision. Herald added a project: clang. The HLSL support in clang is in proress and not fully functioning. As such we

[clang] 1172bde - [HIP] Fix unbundling archive

Author: Yaxun (Sam) Liu Date: 2022-09-26T22:16:17-04:00 New Revision: 1172bdecfab364579d90e6aa5ba7fc64a5b96786 URL: https://github.com/llvm/llvm-project/commit/1172bdecfab364579d90e6aa5ba7fc64a5b96786 DIFF:

[PATCH] D134669: [clang][msan] Turn on -fsanitize-memory-param-retval by default

vitalybuka added a comment. diff --git a/clang/test/Driver/fsanitize-memory-param-retval.c b/clang/test/Driver/fsanitize-memory-param-retval.c index d82d20812186..79ade32178b6 100644 --- a/clang/test/Driver/fsanitize-memory-param-retval.c +++

[PATCH] D134235: [clang-doc] Clean up *Info constructors.

paulkirth accepted this revision. paulkirth added a comment. I'm good w/ this. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134235/new/ https://reviews.llvm.org/D134235 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D134235: [clang-doc] Clean up *Info constructors.

haowei accepted this revision. haowei added a comment. This revision is now accepted and ready to land. LGTM @paulkirth Do you have more comments? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134235/new/ https://reviews.llvm.org/D134235 ___

[PATCH] D134235: [clang-doc] Clean up *Info constructors.

brettw added inline comments. Comment at: clang-tools-extra/clang-doc/BitcodeReader.cpp:336 Reference *I, FieldId ) { switch (ID) { case REFERENCE_USR: haowei wrote: > This block ID came from L582, which read from a bitcode file

[PATCH] D134688: MSVC AArch64 ABI: Homogeneous aggregates

dblaikie created this revision. dblaikie added reviewers: hansw, rnk. Herald added subscribers: mstorsjo, kristof.beyls. Herald added a project: All. dblaikie requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fixes: Protected members, HFA:

[clang] f3e0298 - [NFC][Clang] Remove dead code

Author: Yuanfang Chen Date: 2022-09-26T18:32:52-07:00 New Revision: f3e02989e6e5d6fa90d5ac1ea21c592197a0ae7d URL: https://github.com/llvm/llvm-project/commit/f3e02989e6e5d6fa90d5ac1ea21c592197a0ae7d DIFF: https://github.com/llvm/llvm-project/commit/f3e02989e6e5d6fa90d5ac1ea21c592197a0ae7d.diff

[PATCH] D86049: RFC: Implement optional exportable wrapper function generation for objc_direct methods.

plotfi updated this revision to Diff 463064. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86049/new/ https://reviews.llvm.org/D86049 Files: clang/include/clang/AST/DeclObjC.h clang/lib/AST/Mangle.cpp clang/lib/CodeGen/CGObjC.cpp

[PATCH] D133574: [C2x] reject type definitions in offsetof

inclyc updated this revision to Diff 463063. inclyc added a comment. This revision fixes: **anonymous struct** > You should be able to pass in the TagDecl directly because the diagnostics > engine knows how to print a NamedDecl. I've switch back to using `Context.getTagDeclType(New)` because

[PATCH] D134687: [clang] Ensure correct metadata for relative vtables

leonardchan created this revision. leonardchan added reviewers: phosek, mcgrathr, tejohnson, pcc. leonardchan added a project: clang. Herald added a subscriber: Prazek. Herald added a project: All. leonardchan requested review of this revision. Prior to this, metadata pertaining to the size or

[PATCH] D134669: [clang][msan] Turn on -fsanitize-memory-param-retval by default

aeubanks updated this revision to Diff 463061. aeubanks added a comment. add release notes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134669/new/ https://reviews.llvm.org/D134669 Files: clang/docs/ReleaseNotes.rst

[PATCH] D134669: [clang][msan] Turn on -fsanitize-memory-param-retval by default

vitalybuka updated this revision to Diff 463058. vitalybuka added a comment. Herald added a subscriber: MaskRay. Update Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134669/new/ https://reviews.llvm.org/D134669 Files:

[PATCH] D134669: [clang][msan] Turn on -fsanitize-memory-param-retval by default

vitalybuka added inline comments. Comment at: clang/include/clang/Driver/Options.td:1767 MarshallingInfoEnum, "Global">; defm sanitize_memory_param_retval : BoolFOption<"sanitize-memory-param-retval", you need to update

[PATCH] D134235: [clang-doc] Clean up *Info constructors.

haowei added inline comments. Comment at: clang-tools-extra/clang-doc/BitcodeReader.cpp:336 Reference *I, FieldId ) { switch (ID) { case REFERENCE_USR: This block ID came from L582, which read from a bitcode file when

[clang] b2d7a0d - [AArch64] Check target feature support for __builtin_arm_crc*

Author: Fangrui Song Date: 2022-09-26T17:16:44-07:00 New Revision: b2d7a0dcf1ff1c08cac173f408843a8659c6f3cd URL: https://github.com/llvm/llvm-project/commit/b2d7a0dcf1ff1c08cac173f408843a8659c6f3cd DIFF: https://github.com/llvm/llvm-project/commit/b2d7a0dcf1ff1c08cac173f408843a8659c6f3cd.diff

[PATCH] D134685: Fix SourceManager::isBeforeInTranslationUnit bug with token-pasting

sammccall created this revision. sammccall added reviewers: ilya-biryukov, hokein. Herald added subscribers: kadircet, arphaman. Herald added a project: All. sammccall requested review of this revision. Herald added projects: clang, clang-tools-extra. Herald added a subscriber: cfe-commits.

[PATCH] D133993: [HLSL] Remove global ctor/dtor variable for non-lib profile.

python3kgae added a comment. In D133993#3816526 , @efriedma wrote: > Why are we using different mechanisms for global constructors in "libraries" > vs. other code? If we need a mechanism in LLVM already, we might as well use > it all the time? For

[PATCH] D134669: [clang][msan] Turn on -fsanitize-memory-param-retval by default

vitalybuka added a comment. In D134669#3816519 , @aeubanks wrote: > do you want the tests to all properly work with > -fsanitize-memory-param-retval, or just pin them to > -fno-sanitize-memory-param-retval if they're failing? I have some draft patch

[PATCH] D134671: [Driver] Prevent Mips specific code from claiming -mabi argument on other targets.

craig.topper updated this revision to Diff 463049. craig.topper added a comment. Herald added subscribers: pcwang-thead, s.egerton, simoncook. Address review comments. I didn't test the -march because all of the target I'm familiar with use -march. Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D134456: [PGO] Consider parent context when weighing branches with likelyhood.

wenlei added a comment. There's also subtle difference between different variants of PGO. For instrumentation PGO where profile quality is higher, profile counts always take precedence. But for sampling PGO (AutoFDO, CSSPGO), it tries to not overwrite any existing branch weights, which means

[PATCH] D133993: [HLSL] Remove global ctor/dtor variable for non-lib profile.

efriedma added a comment. Why are we using different mechanisms for global constructors in "libraries" vs. other code? If we need a mechanism in LLVM already, we might as well use it all the time? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D134669: [clang][msan] Turn on -fsanitize-memory-param-retval by default

aeubanks added a comment. do you want the tests to all properly work with -fsanitize-memory-param-retval, or just pin them to -fno-sanitize-memory-param-retval if they're failing? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134669/new/

[PATCH] D134671: [Driver] Prevent Mips specific code from claiming -mabi argument on other targets.

craig.topper added inline comments. Comment at: clang/lib/Driver/Driver.cpp:637 // accordingly to provided ABI name. - A = Args.getLastArg(options::OPT_mabi_EQ); + A = Args.getLastArgNoClaim(options::OPT_mabi_EQ); if (A && Target.isMIPS()) { arichardson

[PATCH] D134669: [clang][msan] Turn on -fsanitize-memory-param-retval by default

vitalybuka added reviewers: thakis, hans, glider. vitalybuka added a comment. LGTM. I was thinking about this recently as well. Main inconvenience is users like Chromium. They have recompiled libraries. They will need to use -fno-sanitize-memory-param-retval, if recompiling is not an option.

[PATCH] D134671: [Driver] Prevent Mips specific code from claiming -mabi argument on other targets.

MaskRay added inline comments. Comment at: clang/test/Driver/mabi.c:1 +// RUN: %clang -target i386-unknown-linux -mabi=ms -S %s -### 2>&1 | FileCheck --check-prefix=CHECK %s + `-target ` is legacy. Use `--target=` Repository: rG LLVM Github Monorepo

[PATCH] D132379: [Support] Class for response file expansion

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

[PATCH] D130096: [Clang][AMDGPU] Emit AMDGPU library control constants in clang

jhuber6 updated this revision to Diff 463042. jhuber6 added a comment. Adding test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130096/new/ https://reviews.llvm.org/D130096 Files: clang/lib/CodeGen/CodeGenAction.cpp

[PATCH] D134352: [AArch64] Add Neoverse V2 CPU support

paulwalker-arm accepted this revision. paulwalker-arm added inline comments. Comment at: clang/docs/ReleaseNotes.rst:376 them, which it cannot. +- Add driver and tuning support for Neoverse V2 support via the flag + ``-mcpu=neoverse-v2``. Native detection is also supported

[PATCH] D134454: [Driver][Distro] Fix ArchLinux triplet and sysroot detection

MaskRay added a comment. I am nervous as well as Arch Linux has many derivatives. They likely don't use `ID=arch` in `/etc/os-release`. The patch won't work for them. The last few comments of https://discourse.llvm.org/t/rfc-adding-a-default-file-location-to-config-file-support/63606 discuss

Re: [PATCH] D134235: [clang-doc] Clean up *Info constructors.

Please avoid directly replying to the email as they won't be logged in the phabricator. This boolean is logged into YAML output, while I don't have data, I could bet there will be users depending on it even if it looks so unlikely. We don't own the tool in the first place, in that case, I felt if

[PATCH] D134235: [clang-doc] Clean up *Info constructors.

brettw added a comment. I can guarantee that nobody uses this for anything useful because it doesn't work well enough. For example, before I started changing it, requesting HTML output crashed on startup and many of the most basic syntactic things were broken. My previous patch changed enums

[PATCH] D129802: [DRAFT] Implementing new atomic orderings in LLVM and generate barriers for legacy __sync builtins. Support corresponding memory model in outline atomics as well.

ilinpv added a comment. @sebpop could you ellaborate on __sync_* operations usage, are you getting issues with current Clang implementation? Do Clang need to keep supporting them and fix introducing new memory model? It seems we need compelling reasons to do that. >> However if sync

Re: [PATCH] D134235: [clang-doc] Clean up *Info constructors.

On Mon, Sep 26, 2022 at 1:42 PM Haowei Wu via Phabricator < revi...@reviews.llvm.org> wrote: > haowei added a comment. > > @brettw Thanks for the clarification. I was oncall for buildgardener last > week and got quite busy due to breakages. > > Thanks for your clarification, that is really

[PATCH] D134222: [clang][deps] Report module map describing compiled module

w2yehia added a comment. In D134222#3815956 , @jansvoboda11 wrote: > In D134222#3815793 , @w2yehia wrote: > >> @jansvoboda11 >> FYI, this is breaking AIX buildbot: >>

[PATCH] D134681: [Clang][AArch64] Add SME outer product intrinsics

sagarkulkarni19 created this revision. sagarkulkarni19 added reviewers: sdesmalen, rsandifo-arm, david-arm. sagarkulkarni19 added a project: clang. Herald added a subscriber: kristof.beyls. Herald added a project: All. sagarkulkarni19 requested review of this revision. Herald added a subscriber:

[PATCH] D134680: [Clang][AArch64] Add SME svaddha and svaddva intrinsics

sagarkulkarni19 created this revision. sagarkulkarni19 added reviewers: sdesmalen, david-arm, rsandifo-arm. sagarkulkarni19 added a project: clang. Herald added a subscriber: kristof.beyls. Herald added a project: All. sagarkulkarni19 requested review of this revision. Herald added a subscriber:

[PATCH] D134454: [Driver][Distro] Fix ArchLinux triplet and sysroot detection

nickdesaulniers requested changes to this revision. nickdesaulniers added a comment. This revision now requires changes to proceed. It's not clear to me why Arch is special here. The path may be different, but normalizing the triple seems unnecessary. Blocking this until I'm confident that

[PATCH] D134679: [Clang][AArch64] Add SME svcntsb/h/w/d C intrinsics

sagarkulkarni19 created this revision. sagarkulkarni19 added reviewers: sdesmalen, rsandifo-arm, david-arm. sagarkulkarni19 added a project: clang. Herald added a subscriber: kristof.beyls. Herald added a project: All. sagarkulkarni19 requested review of this revision. Herald added a subscriber:

[PATCH] D134678: [Clang][AArch64] Add SME ldr and str intrinsic

sagarkulkarni19 created this revision. sagarkulkarni19 added reviewers: sdesmalen, rsandifo-arm, david-arm. sagarkulkarni19 added a project: clang. Herald added a subscriber: kristof.beyls. Herald added a project: All. sagarkulkarni19 requested review of this revision. Herald added a subscriber:

[PATCH] D134671: [Driver] Prevent Mips specific code from claiming -mabi argument on other targets.

arichardson added inline comments. Comment at: clang/lib/Driver/Driver.cpp:637 // accordingly to provided ABI name. - A = Args.getLastArg(options::OPT_mabi_EQ); + A = Args.getLastArgNoClaim(options::OPT_mabi_EQ); if (A && Target.isMIPS()) { Would it make

[PATCH] D133993: [HLSL] Remove global ctor/dtor variable for non-lib profile.

aaron.ballman edited reviewers, added: rjmccall, efriedma; removed: aaron.ballman. aaron.ballman added a comment. Switching up code reviewers for the codegen questions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133993/new/

[PATCH] D133705: [HIP] Fix unbundling archive

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

[PATCH] D134550: [Clang] Make Clang driver suggest '-Xclang' for CC1 options passed to the driver

aaron.ballman added a comment. In D134550#3816255 , @jhuber6 wrote: > In D134550#3816239 , @aaron.ballman > wrote: > >> In terms of this patch -- is there sentiment to revert (even temporarily >> while we

[PATCH] D131172: [clang][llvm][doc] Add more information for the ABI change in FP16

mehdi_amini added inline comments. Comment at: llvm/docs/ReleaseNotes.rst:206 +When you find failures with ``half`` type, check the calling conversion of the +code and switch it to the new ABI. + (same here: users reading "switch it to the new ABI" will be

[PATCH] D131172: [clang][llvm][doc] Add more information for the ABI change in FP16

mehdi_amini added inline comments. Comment at: clang/docs/ReleaseNotes.rst:633 +- If you are using downstream runtimes that provide FP16 conversions, update +them with the new ABI. + Can you add a link to a documentation that describe the new ABI? Repository:

[PATCH] D134020: [clang][Interp] Handle enums

aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134020/new/ https://reviews.llvm.org/D134020 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D134677: [Clang][AArch64] Add SME zero intrinsic

sagarkulkarni19 created this revision. sagarkulkarni19 added reviewers: sdesmalen, rsandifo-arm, david-arm. sagarkulkarni19 added a project: clang. Herald added a subscriber: kristof.beyls. Herald added a project: All. sagarkulkarni19 requested review of this revision. Herald added a subscriber:

[PATCH] D134550: [Clang] Make Clang driver suggest '-Xclang' for CC1 options passed to the driver

jhuber6 added a comment. In D134550#3816239 , @aaron.ballman wrote: > In terms of this patch -- is there sentiment to revert (even temporarily > while we discuss)? I think alternatively we could just change the message to mention that the provided

[PATCH] D134550: [Clang] Make Clang driver suggest '-Xclang' for CC1 options passed to the driver

aaron.ballman added a comment. In D134550#3813285 , @jyknight wrote: > In D134550#3813269 , @aaron.ballman > wrote: > >> Alternatively, perhaps those experimental options should be exposed from the >> driver

[PATCH] D133711: [Sema] Reject array element types whose sizes aren't a multiple of their alignments

ahatanak added a comment. Fixed in 0ca1051bf . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133711/new/ https://reviews.llvm.org/D133711

[PATCH] D127910: [Clang][AArch64] Add SME C intrinsics for load and store

sagarkulkarni19 updated this revision to Diff 463023. sagarkulkarni19 added a comment. Update testcases by adding "arm_streaming" attribute to the callee. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127910/new/ https://reviews.llvm.org/D127910

[PATCH] D134458: [AST] Add C++11 attribute 'msvc::no_unique_address'

aaron.ballman added inline comments. Comment at: clang/include/clang/Basic/Attr.td:3505 + let LangOpts = [MicrosoftExt]; + let Spellings = [CXX11<"msvc", "no_unique_address", 201803>]; + let Subjects = SubjectList<[NonBitField], ErrorDiag>; Does the `201803`

[PATCH] D134556: [Clang][OpenMP] Fix run time crash when use_device_addr is used.

jyu2 added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:8561-8617 for (const auto *Cl : Clauses) { const auto *C = dyn_cast(Cl); if (!C) continue; for (const auto L : C->component_lists()) {

[PATCH] D130096: [Clang][AMDGPU] Emit AMDGPU library control constants in clang

yaxunl added inline comments. Comment at: clang/lib/CodeGen/CodeGenAction.cpp:299-308 + if (!LinkModules.empty() && Gen->CGM().getTriple().isAMDGCN() && + !Gen->CGM().getLangOpts().GPURelocatableDeviceCode) { +const StringRef GVS[] = {"__oclc_daz_opt",

[PATCH] D133683: [c++] implements tentative DR1432 for partial ordering of function template

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

[PATCH] D133683: [c++] implements tentative DR1432 for partial ordering of function template

ychen updated this revision to Diff 463019. ychen added a comment. - update comments suggested by @mizvekov Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133683/new/ https://reviews.llvm.org/D133683 Files:

[PATCH] D134057: [clang][Interp] Start implementing record types

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM aside from a minor nit, we can handle the other cases in follow-ups. Comment at: clang/lib/AST/Interp/ByteCodeExprGen.cpp:642 + +for (const auto *Init

[clang] 0ca1051 - Check whether the allocated type is an array type before calling

Author: Akira Hatanaka Date: 2022-09-26T14:21:21-07:00 New Revision: 0ca1051bfc6e9548763f0a52641733393c33a427 URL: https://github.com/llvm/llvm-project/commit/0ca1051bfc6e9548763f0a52641733393c33a427 DIFF:

[PATCH] D133574: [C2x] reject type definitions in offsetof

aaron.ballman added a comment. In D133574#3816066 , @nathanchance wrote: > In D133574#3815951 , @aaron.ballman > wrote: > >> LGTM, thank you! Please don't land until you have some indication from the >> kernel

[PATCH] D134673: [Driver][PS4] pass -fcrash-diagnostics-dir to LTO

ychen added a comment. In D134673#3816166 , @probinson wrote: > I'm guessing the non-PS4 targets would also want this, so a similar change is > needed in tools::addLTOOptions. This can be a separate patch if you wish. That's right. It was submitted

[PATCH] D134673: [Driver][PS4] pass -fcrash-diagnostics-dir to LTO

probinson added a comment. I'm guessing the non-PS4 targets would also want this, so a similar change is needed in tools::addLTOOptions. This can be a separate patch if you wish. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134673/new/

[PATCH] D133705: [HIP] Fix unbundling archive

yaxunl updated this revision to Diff 463016. yaxunl marked an inline comment as done. yaxunl added a comment. check file magic and only unbundle real archives CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133705/new/ https://reviews.llvm.org/D133705 Files:

[PATCH] D133705: [HIP] Fix unbundling archive

yaxunl marked an inline comment as done. yaxunl added inline comments. Comment at: clang/test/Driver/clang-offload-bundler.c:410-412 +// Check clang-offload-bundler extracts empty archives if the input file +// is not an archive when --allow-missing-bundles is specified,

[PATCH] D130096: [Clang][AMDGPU] Emit AMDGPU library control constants in clang

arsenm added a comment. In D130096#3815529 , @jhuber6 wrote: > The best solution would be to handle these per-TU variables in the backend. > Or maybe even all of these could be placed in the backend where the code > paths that currently require a

[PATCH] D134673: [Driver][PS4] pass -fcrash-diagnostics-dir to LTO

ychen created this revision. ychen added a reviewer: probinson. Herald added a subscriber: inglorion. Herald added a project: All. ychen requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. Also refactor the existing code a little bit.

[PATCH] D134235: [clang-doc] Clean up *Info constructors.

haowei added a comment. @brettw Thanks for the clarification. I was oncall for buildgardener last week and got quite busy due to breakages. Thanks for your clarification, that is really helpful to understand the your intent. While I still don't see the way these constructors are implemented is

[PATCH] D133574: [C2x] reject type definitions in offsetof

nickdesaulniers added a comment. In D133574#3816066 , @nathanchance wrote: > In D133574#3815951 , @aaron.ballman > wrote: > >> LGTM, thank you! Please don't land until you have some indication from the >>

[PATCH] D134671: [Driver] Prevent Mips specific code from claiming -mabi argument on other targets.

craig.topper created this revision. craig.topper added reviewers: aaron.ballman, erichkeane. Herald added subscribers: StephenFan, atanasyan, arichardson, sdardis. Herald added a project: All. craig.topper requested review of this revision. Herald added a subscriber: MaskRay. Herald added a

[PATCH] D133574: [C2x] reject type definitions in offsetof

nathanchance added a comment. In D133574#3815951 , @aaron.ballman wrote: > LGTM, thank you! Please don't land until you have some indication from the > kernel folks that this won't be super disruptive for them. CC > @nickdesaulniers and @nathanchance

[PATCH] D134544: [clang-cl] Implement /ZH: flag

dblaikie added inline comments. Comment at: clang/include/clang/Driver/Options.td:3220 Values<"simple,mangled">, Flags<[CC1Option, NoDriverOption]>; +def gsrc_hash_EQ : Joined<["-"], "gsrc-hash=">, + Group, Flags<[CC1Option, NoDriverOption]>, This is a

[PATCH] D134669: [clang][msan] Turn on -fsanitize-memory-param-retval by default

aeubanks added a comment. before looking into those, is it ok to proceed with this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134669/new/ https://reviews.llvm.org/D134669 ___ cfe-commits mailing

[PATCH] D133036: [InstCombine] Treat passing undef to noundef params as UB

aeubanks added a comment. In D133036#3816002 , @aeubanks wrote: > @vitalybuka can we turn on -fsanitize-memory-param-retval by default upstream? attempt in https://reviews.llvm.org/D134669 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D134669: [clang][msan] Turn on -fsanitize-memory-param-retval by default

aeubanks added a comment. hmm, looks like this breaks a bunch of msan tests with `-DLLVM_ENABLE_RUNTIMES=compiler-rt` and `ninja && ninja check-runtimes` MemorySanitizer-X86_64 :: Linux/swapcontext_annotation_reset.cpp MemorySanitizer-X86_64 :: bsearch.cpp MemorySanitizer-X86_64 ::

[PATCH] D134669: [clang][msan] Turn on -fsanitize-memory-param-retval by default

aeubanks created this revision. aeubanks added a reviewer: vitalybuka. Herald added a project: All. aeubanks requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This eagerly reports use of undef values when passed to noundef parameters or

[PATCH] D134652: [clang-format] Add Really Basic Carbon Support/Infrastructure

MyDeveloperDay updated this revision to Diff 463001. MyDeveloperDay added a comment. Remove .clang-format change CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134652/new/ https://reviews.llvm.org/D134652 Files: clang/docs/ClangFormatStyleOptions.rst clang/docs/ReleaseNotes.rst

[PATCH] D134652: [clang-format] Add Really Basic Carbon Support/Infrastructure

MyDeveloperDay updated this revision to Diff 463000. MyDeveloperDay added a comment. Fix pointer function arguments Add release notes Fix more `TT_TrailingReturnArrow` cases CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134652/new/ https://reviews.llvm.org/D134652 Files:

[PATCH] D86049: RFC: Implement optional exportable wrapper function generation for objc_direct methods.

plotfi added a comment. @dmaclach @ahatanak @mwyman How do things look from here? Do you want something for properties as well or would it be ok if we did this in a later commit? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86049/new/

[PATCH] D133036: [InstCombine] Treat passing undef to noundef params as UB

aeubanks added a comment. @vitalybuka can we turn on -fsanitize-memory-param-retval by default upstream? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133036/new/ https://reviews.llvm.org/D133036 ___

[PATCH] D86049: RFC: Implement optional exportable wrapper function generation for objc_direct methods.

plotfi updated this revision to Diff 462999. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86049/new/ https://reviews.llvm.org/D86049 Files: clang/include/clang/AST/DeclObjC.h clang/lib/AST/DeclObjC.cpp clang/lib/AST/Mangle.cpp

[PATCH] D134668: [LTO][clang] Using Single Dash Consistently when Passing LTO Options

qiongsiwu1 created this revision. qiongsiwu1 added reviewers: w2yehia, MaskRay. qiongsiwu1 added a project: clang. Herald added subscribers: ormris, StephenFan, inglorion. Herald added a project: All. qiongsiwu1 requested review of this revision. Herald added a subscriber: cfe-commits. The

[PATCH] D134222: [clang][deps] Report module map describing compiled module

jansvoboda11 added a comment. In D134222#3815793 , @w2yehia wrote: > @jansvoboda11 > FYI, this is breaking AIX buildbot: > https://lab.llvm.org/buildbot/#/builders/214/builds/3436/steps/6/logs/FAIL__Clang__modules-header-sharing_m Thanks for reporting

[PATCH] D133574: [C2x] reject type definitions in offsetof

aaron.ballman accepted this revision. aaron.ballman added a subscriber: nathanchance. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM, thank you! Please don't land until you have some indication from the kernel folks that this won't be super disruptive for

  1   2   3   >