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

gulfem added inline comments. Comment at: clang/include/clang/Basic/Attr.td:1435 + let Spellings = [GCC<"leaf">]; + let Subjects = SubjectList<[Function]>; + let Documentation = [Undocumented]; aaron.ballman wrote: > gulfem wrote: > > aaron.ballman wrote: > >

[PATCH] D90765: [ARM][AArch64] Adding Neoverse V1 CPU support

dmgreen added inline comments. Comment at: llvm/lib/Target/AArch64/AArch64Subtarget.cpp:198 break; + case NeoverseV1: +PrefFunctionLogAlignment = 4; Please put with the other cpus above. Comment at: llvm/lib/Target/AArch64/AArch64Subt

[clang] 77a9e0a - Basic: Split out DirectoryEntry.h, NFC

Author: Duncan P. N. Exon Smith Date: 2020-11-04T14:11:57-05:00 New Revision: 77a9e0a4af5aea3795ed79a0db61ee5b19b837e4 URL: https://github.com/llvm/llvm-project/commit/77a9e0a4af5aea3795ed79a0db61ee5b19b837e4 DIFF: https://github.com/llvm/llvm-project/commit/77a9e0a4af5aea3795ed79a0db61ee5b19b8

[PATCH] D90478: Basic: Split out DirectoryEntry.h, NFC

This revision was automatically updated to reflect the committed changes. Closed by commit rG77a9e0a4af5a: Basic: Split out DirectoryEntry.h, NFC (authored by dexonsmith). Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D9

[PATCH] D88859: APINotes: add APINotesYAMLCompiler

compnerd added a comment. Thanks for the reviews! I'll reformat the rest of the code before merging. Im going to be working on merging this downstream first before merging this upstream, so it will take a couple of days still. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D82035: [PowerPC] Add Sema checks for MMA types

bsaleil updated this revision to Diff 302918. bsaleil added a comment. Change function name and improve comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82035/new/ https://reviews.llvm.org/D82035 Files: clang/include/clang/Basic/DiagnosticS

[PATCH] D90221: Include more attribute details when dumping AST in JSON

aronsky added a comment. > It seems surprising to me that you'd have to add those declarations; I think > that's a layering violation. There's something somewhat strange going on here > that I think is related. Given: > [...] > Notice how the annotate attribute has an `inner` array with the argu

[PATCH] D89790: [clangd] Add basic conflict detection for the rename.

hokein updated this revision to Diff 302928. hokein marked 2 inline comments as done. hokein added a comment. address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89790/new/ https://reviews.llvm.org/D89790 Files: clang-tools-extra/cla

[PATCH] D89790: [clangd] Add basic conflict detection for the rename.

hokein added inline comments. Comment at: clang-tools-extra/clangd/refactor/Rename.cpp:263 + DeclContextLookupResult LookupResult; + switch (DC->getDeclKind()) { + case Decl::TranslationUnit: sammccall wrote: > sammccall wrote: > > explain this list somewhat?

[PATCH] D82035: [PowerPC] Add Sema checks for MMA types

amyk accepted this revision as: amyk. amyk added a comment. This revision is now accepted and ready to land. Overall LGTM, thanks for addressing comments Baptiste! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82035/new/ https://reviews.llvm.org/D8

[PATCH] D90540: [Syntax] Add minimal TableGen for syntax nodes. NFC

hokein accepted this revision. hokein added inline comments. Comment at: clang/utils/TableGen/ClangSyntaxEmitter.cpp:107 +llvm::raw_ostream &OS) { + OS << "// Generated from " << Records.getInputFilename() << "\n"; + OS << R"cpp(

[PATCH] D90531: [clangd] Add clang-tidy options to config

njames93 marked 13 inline comments as done. njames93 added inline comments. Comment at: clang-tools-extra/clangd/ConfigFragment.h:183 + struct ClangTidyBlock { +llvm::Optional> Enable; +/// List of checks to enable or disable, can use wildcards. sammccal

[PATCH] D90531: [clangd] Add clang-tidy options to config

njames93 updated this revision to Diff 302935. njames93 added a comment. Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90531/new/ https://reviews.llvm.org/D90531 Files: clang-tools-extra/clangd/Config.h clang-tools-extra/clang

[clang] ab0ddbc - Reland [NewPM] Add OptimizationLevel param to registerPipelineStartEPCallback

Author: Arthur Eubanks Date: 2020-11-04T13:11:40-08:00 New Revision: ab0ddbc38af9508ee0f753fb897d9050025a0763 URL: https://github.com/llvm/llvm-project/commit/ab0ddbc38af9508ee0f753fb897d9050025a0763 DIFF: https://github.com/llvm/llvm-project/commit/ab0ddbc38af9508ee0f753fb897d9050025a0763.diff

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

gulfem added a comment. In D90275#2371343 , @jdoerfert wrote: > The more I think about it, the more I think we should never create a > `leaf`/`nocallback` definition. Only declarations should carry that attribute. Leaf attribute is specifically intended

[clang] 6e73cfa - Basic: Change Module::Umbrella to a PointerUnion, NFC

Author: Duncan P. N. Exon Smith Date: 2020-11-04T16:19:39-05:00 New Revision: 6e73cfa8363d43689f77b0e4e4c2787ae6ae3fb6 URL: https://github.com/llvm/llvm-project/commit/6e73cfa8363d43689f77b0e4e4c2787ae6ae3fb6 DIFF: https://github.com/llvm/llvm-project/commit/6e73cfa8363d43689f77b0e4e4c2787ae6ae

[PATCH] D90481: Basic: Change Module::Umbrella to a PointerUnion, NFC

This revision was automatically updated to reflect the committed changes. Closed by commit rG6e73cfa8363d: Basic: Change Module::Umbrella to a PointerUnion, NFC (authored by dexonsmith). Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revi

[PATCH] D90799: [PowerPC] Add paired vector load and store builtins and intrinsics

bsaleil created this revision. bsaleil added reviewers: nemanjai, amyk, saghir, lei. bsaleil added projects: LLVM, PowerPC. Herald added subscribers: llvm-commits, cfe-commits, shchenz, kbarton, hiraditya. Herald added a project: clang. bsaleil requested review of this revision. This patch adds t

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

nickdesaulniers added a comment. In D90180#2357247 , @aaron.ballman wrote: > When you pass `-fix` to clang-tidy, it will apply fix-its from the compiler > as well. See https://godbolt.org/z/7vzM4b and try adding a semicolon to the > end of the `switch`

[PATCH] D86105: [darwin] Disable the -Wpsabi warning

arphaman updated this revision to Diff 302977. arphaman added a comment. Address Ahmed's comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86105/new/ https://reviews.llvm.org/D86105 Files: clang/lib/CodeGen/TargetInfo.cpp clang/test/CodeGen/target-avx-abi-diag.c clang/test/

[clang] f976ba6 - [PowerPC] Add Sema checks for MMA types

Author: Baptiste Saleil Date: 2020-11-04T17:01:47-06:00 New Revision: f976ba61395811732b4605b6fb9b6ba5cd489372 URL: https://github.com/llvm/llvm-project/commit/f976ba61395811732b4605b6fb9b6ba5cd489372 DIFF: https://github.com/llvm/llvm-project/commit/f976ba61395811732b4605b6fb9b6ba5cd489372.dif

[PATCH] D86105: [darwin] Disable the -Wpsabi warning

arphaman marked 2 inline comments as done. arphaman added inline comments. Comment at: clang/lib/CodeGen/TargetInfo.cpp:2561-2565 // The "avx" feature changes how vectors >128 in size are passed. "avx512f" // additionally changes how vectors >256 in size are passed. Lik

[PATCH] D82035: [PowerPC] Add Sema checks for MMA types

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGf976ba613958: [PowerPC] Add Sema checks for MMA types (authored by bsaleil). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION http

[PATCH] D82035: [PowerPC] Add Sema checks for MMA types

NeHuang added inline comments. Comment at: clang/lib/Sema/SemaChecking.cpp:10339 + if (Context.getTargetInfo().getTriple().isPPC64()) +CheckPPCMMAType(RetValExp->getType(), ReturnLoc); } CheckPPCMMAType returns bool value. Do we need add it in the if condit

[PATCH] D90802: [OpenMP] [DOCS] Update OMP5.1 feature status table [NFC]

dreachem created this revision. dreachem added reviewers: kkwli0, jdoerfert, ABataev, RaviNarayanaswamy. Herald added subscribers: cfe-commits, jfb, guansong, yaxunl. Herald added a project: clang. dreachem requested review of this revision. Herald added a subscriber: sstefan1. Adding features in

[PATCH] D90799: [PowerPC] Add paired vector load and store builtins and intrinsics

bsaleil updated this revision to Diff 302983. bsaleil added a comment. Add builtin tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90799/new/ https://reviews.llvm.org/D90799 Files: clang/include/clang/Basic/BuiltinsPPC.def clang/lib/CodeGe

[PATCH] D82035: [PowerPC] Add Sema checks for MMA types

bsaleil added inline comments. Comment at: clang/lib/Sema/SemaChecking.cpp:10339 + if (Context.getTargetInfo().getTriple().isPPC64()) +CheckPPCMMAType(RetValExp->getType(), ReturnLoc); } NeHuang wrote: > CheckPPCMMAType returns bool value. Do we need add it

[PATCH] D90802: [OpenMP] [DOCS] Update OMP5.1 feature status table [NFC]

jdoerfert added inline comments. Comment at: clang/docs/OpenMPSupport.rst:343 ++--+--+--+---+ +| mis

[PATCH] D90714: [clang]Fix length threshold for MicrosoftMangle md5 hash

dblaikie added a comment. Since the same code is used to mangle all these things, probably just test one of them? Could use macros to stamp out longer names without having to write them out manually? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[PATCH] D90348: [Driver] specify -stack-protector 0 for -fno-stack-protector

nickdesaulniers added a comment. Bumping for review Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90348/new/ https://reviews.llvm.org/D90348 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:

[PATCH] D90194: [Driver] split LangOptions::SSPOff into SSPOFF and SSPUnspecified

nickdesaulniers added a comment. Bumping for review Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90194/new/ https://reviews.llvm.org/D90194 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:

[PATCH] D87956: [IR] add fn attr for no_stack_protector; prevent inlining on mismatch

MaskRay added inline comments. Herald added a subscriber: pengfei. Comment at: clang/test/Frontend/optimization-remark-missed-inline-stack-protectors.c:1 +// RUN: %clang_cc1 -stack-protector 2 -Rpass-missed=inline -O2 -verify %s -emit-llvm-only + Emm. I am a bi

[PATCH] D90441: [X86] Add support for vex, vex2, vex3, and evex for MASM

LiuChen3 added inline comments. Comment at: llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp:2851 +// Parse MASM style pseudo prefixes. +// FIXME: This prefix should only be used for MASM, not for intel-syntax. +if (isParsingIntelSyntax()) { craig.toppe

[PATCH] D90194: [Driver] split LangOptions::SSPOff into SSPOFF and SSPUnspecified

void added inline comments. Comment at: clang/include/clang/Basic/LangOptions.h:60 enum GCMode { NonGC, GCOnly, HybridGC }; - enum StackProtectorMode { SSPOff, SSPOn, SSPStrong, SSPReq }; + enum StackProtectorMode { SSPOff, SSPOn, SSPStrong, SSPReq, SSPUnspecified }; -

[PATCH] D79279: Add overloaded versions of builtin mem* functions

rsmith requested changes to this revision. rsmith added a comment. This revision now requires changes to proceed. I think the documentation isn't quite right yet, but otherwise I think I'm happy. (With a couple of code change suggestions.) In D79279#2240487

[PATCH] D90194: [Driver] split LangOptions::SSPOff into SSPOFF and SSPUnspecified

nickdesaulniers added inline comments. Comment at: clang/include/clang/Basic/LangOptions.h:60 enum GCMode { NonGC, GCOnly, HybridGC }; - enum StackProtectorMode { SSPOff, SSPOn, SSPStrong, SSPReq }; + enum StackProtectorMode { SSPOff, SSPOn, SSPStrong, SSPReq, SSPUnspecified

[PATCH] D90329: [PowerPC] Fix va_arg in Objective-C on 32-bit ELF targets

kernigh added inline comments. Comment at: clang/lib/CodeGen/TargetInfo.cpp:4723 + bool isInt = Ty->isIntegerType() || Ty->hasPointerRepresentation() || + Ty->isAggregateType(); bool isF64 = Ty->isFloatingType() && getContext().getTypeSize(Ty) == 64; ---

[PATCH] D90809: [amdgpu] Add `llvm.amdgcn.endpgm` support.

hliao created this revision. hliao added reviewers: b-sumner, tony-tye, rampitec, yaxunl. Herald added subscribers: llvm-commits, cfe-commits, kerbowa, hiraditya, t-tye, tpr, dstuttard, nhaehnle, jvesely, kzhuravl, arsenm. Herald added projects: clang, LLVM. hliao requested review of this revision

[clang] ac73b73 - [clang] Add mustprogress and llvm.loop.mustprogress attribute deduction

Author: Atmn Patel Date: 2020-11-04T22:03:14-05:00 New Revision: ac73b73c16526c9e51943759ea6cab285a57e33f URL: https://github.com/llvm/llvm-project/commit/ac73b73c16526c9e51943759ea6cab285a57e33f DIFF: https://github.com/llvm/llvm-project/commit/ac73b73c16526c9e51943759ea6cab285a57e33f.diff LO

[PATCH] D86841: [clang] Add mustprogress and llvm.loop.mustprogress attribute deduction

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGac73b73c1652: [clang] Add mustprogress and llvm.loop.mustprogress attribute deduction (authored by atmnpatel). Repository: rG LLVM Github Monorepo

[PATCH] D90448: [clang] Add type check for explicit instantiation of static data members

nomanous added a comment. Ping @rsmith @chandlerc @majnemer @lvoufo Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90448/new/ https://reviews.llvm.org/D90448 ___ cfe-commits mailing list cfe-commits@lists

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

On Tue, Oct 27, 2020 at 2:21 PM Sriraman Tallam via cfe-commits < cfe-commits@lists.llvm.org> wrote: > > > 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 i

[PATCH] D90815: -fbasic-block-sections=list=: Suppress output if failed to open the file

MaskRay created this revision. MaskRay added reviewers: dblaikie, tmsriram. Herald added a project: clang. Herald added a subscriber: cfe-commits. MaskRay requested review of this revision. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D90815 Files: clang/lib/CodeGen/BackendUt

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

I checked chmod -w a.dwo; clang -cc1 -debug-info-kind=limited -dwarf-version=4 -split-dwarf-file a.dwo -split-dwarf-output a.dwo -emit-obj -o - split-debug-output.c which suppresses the output, so -fbasic-block-sections=list= should follow the convention as well. Sent https://reviews.llvm.org/D908

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

On Wed, Nov 4, 2020 at 8:08 PM Fāng-ruì Sòng wrote: > I checked chmod -w a.dwo; clang -cc1 -debug-info-kind=limited > -dwarf-version=4 -split-dwarf-file a.dwo -split-dwarf-output a.dwo > -emit-obj -o - split-debug-output.c > which suppresses the output, so -fbasic-block-sections=list= should > fo

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

aqjune added a comment. In D90275#2374648 , @gulfem wrote: > Leaf attribute is specifically intended for library functions and I think all > the existing usage of leaf attribute is in the library function declarations. > For ex, it is only used in syscall

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

It is rare to report an error in BackendUtil.cpp . So I checked the other Diags.Report instance and noticed that -split-dwarf-file a.dwo -split-dwarf-output a.dwo (when a.dwo is not writable) suppresses the output. So there is no reason that -fbasic-block-sections=list= should not follow the conven

[clang] 20b4f4f - [Driver] Add callback to Command execution

Author: Serge Pavlov Date: 2020-11-05T12:21:40+07:00 New Revision: 20b4f4f76030f8712824c91b5f0b0a611476f747 URL: https://github.com/llvm/llvm-project/commit/20b4f4f76030f8712824c91b5f0b0a611476f747 DIFF: https://github.com/llvm/llvm-project/commit/20b4f4f76030f8712824c91b5f0b0a611476f747.diff

[PATCH] D78899: [Driver] Add callback to Command execution

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG20b4f4f76030: [Driver] Add callback to Command execution (authored by sepavloff). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

FreddyYe created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. FreddyYe requested review of this revision. Tremont microarchitecture only has GFNI(SSE) version, not AVX and AVX512 version. This patch is to avoid compiling fail on Windows when using -march=tr

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

craig.topper added inline comments. Comment at: clang/lib/Headers/gfniintrin.h:23 +#ifdef __AVX__ +#ifdef __AVX512BW__ This needs to be more like #if !(defined(_MSC_VER) || defined(__SCE__)) || __has_feature(modules) || \

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

craig.topper added a comment. What compilation failure do you get without this? The feature checks are compile time optimization for MSVC because windows.h includes intrin.h. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90822/new/ https://reviews

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

FreddyYe added a comment. The fails are all unknown type errors on Windows, since those typedefs are declared in other header files. The error message goes like: $ clang -march=tremont gfni.c .. ...\lib\clang\12.0.0\include\gfniintrin.h:129:37: error: unknown type name '__mmask

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

craig.topper added a comment. In D90822#2375372 , @FreddyYe wrote: > The fails are all unknown type errors on Windows, since those typedefs are > declared in other header files. > The error message goes like: > > $ clang -march=tremont gfni.c > ..

[clang] ae38540 - [NewPM] Provide method to run all pipeline callbacks, used for -O0

Author: Arthur Eubanks Date: 2020-11-04T22:27:16-08:00 New Revision: ae38540042668675dd16c642d850115f217ea59f URL: https://github.com/llvm/llvm-project/commit/ae38540042668675dd16c642d850115f217ea59f DIFF: https://github.com/llvm/llvm-project/commit/ae38540042668675dd16c642d850115f217ea59f.diff

[PATCH] D89158: [NewPM] Provide method to run all pipeline callbacks, used for -O0

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGae3854004266: [NewPM] Provide method to run all pipeline callbacks, used for -O0 (authored by aeubanks). Changed prior to commit: https://reviews.

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

pengfei added a comment. Craig's method sounds good. @FreddyYe , Why we check AVX512BW instead of AVX512F. I saw SDM says it depends on AVX512F. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90822/new/ https://reviews.llvm.org/D90822

[clang] 5fd3193 - [test] Add 'REQUIRES: bpf-registered-target' to bpf-O0.c

Author: Arthur Eubanks Date: 2020-11-04T23:19:14-08:00 New Revision: 5fd3193c88d82f04571897a9b9d8a49a9f491d88 URL: https://github.com/llvm/llvm-project/commit/5fd3193c88d82f04571897a9b9d8a49a9f491d88 DIFF: https://github.com/llvm/llvm-project/commit/5fd3193c88d82f04571897a9b9d8a49a9f491d88.diff

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

pengfei added a comment. In D90822#2375423 , @pengfei wrote: > Craig's method sounds good. I think it may result in potential problem once we change the order of them in immintrin.h, it should be better to use the first method. #if !(defined(_MSC_VER)

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

craig.topper added a comment. In D90822#2375425 , @pengfei wrote: > In D90822#2375423 , @pengfei wrote: > >> Craig's method sounds good. > > I think it may result in potential problem once we change the order of the

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

FreddyYe added a comment. In D90822#2375423 , @pengfei wrote: > Craig's method sounds good. > @FreddyYe , Why we check AVX512BW instead of AVX512F. I saw SDM says it > depends on AVX512F. I was referring to the old implement and test case. Seems like a e

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

pengfei added a comment. In D90822#2375445 , @craig.topper wrote: > In D90822#2375425 , @pengfei wrote: > >> In D90822#2375423 , @pengfei wrote: >> >>> Craig's method sounds

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

craig.topper added a comment. In D90822#2375463 , @FreddyYe wrote: > In D90822#2375423 , @pengfei wrote: > >> Craig's method sounds good. >> @FreddyYe , Why we check AVX512BW instead of AVX512F. I saw SDM says it >

[PATCH] D90828: [clangd] Trivial: Log missing completion signals.

usaxena95 created this revision. usaxena95 added a reviewer: adamcz. Herald added subscribers: cfe-commits, kadircet, arphaman. Herald added a project: clang. usaxena95 requested review of this revision. Herald added subscribers: MaskRay, ilya-biryukov. Repository: rG LLVM Github Monorepo https

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

pengfei added a comment. In D90822#2375469 , @craig.topper wrote: > In D90822#2375463 , @FreddyYe wrote: > >> In D90822#2375423 , @pengfei wrote: >> >>> Craig's method sound

[PATCH] D90822: [X86] use macros to split GFNI intrinsics into different kinds

FreddyYe updated this revision to Diff 303047. FreddyYe added a comment. Use header file macros instead. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90822/new/ https://reviews.llvm.org/D90822 Files: clang/lib/Headers/gfniintrin.h Index: clang

<    1   2