[PATCH] D123808: [clang] Fix the bogus diagnostic introduced by the newly-added UsingTemplate Kind.

hokein created this revision. hokein added a reviewer: sammccall. Herald added a project: All. hokein requested review of this revision. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D123808 Files: clang/lib/Sema/SemaDeclCXX.cpp clang/test/CXX/

[PATCH] D123352: [analyzer] Add FixItHint to `nullability.NullReturnedFromNonnull` and `nullability.NullableReturnedFromNonnull`

MosheBerman updated this revision to Diff 422922. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123352/new/ https://reviews.llvm.org/D123352 Files: clang/include/clang/StaticAnalyzer/Checkers/Checkers.td clang/lib/StaticAnalyzer/Checkers/Nullabi

[PATCH] D119136: [clang] Implement Change scope of lambda trailing-return-type

cor3ntin added inline comments. Comment at: clang/lib/Parse/ParseExprCXX.cpp:1397 +for (LateParsedAttribute *Attr : LateParsedAttrs) { + ParseLambdaLexedAttribute(*Attr, Attributes, D); + delete Attr; aaron.ballman wrote: > cor3ntin wrot

[clang-tools-extra] b9ca972 - [clang-tidy] Add portability-std-allocator-const check

Author: Fangrui Song Date: 2022-04-14T11:13:41-07:00 New Revision: b9ca972b1ff0548a831fe416cec8d39f7e569c94 URL: https://github.com/llvm/llvm-project/commit/b9ca972b1ff0548a831fe416cec8d39f7e569c94 DIFF: https://github.com/llvm/llvm-project/commit/b9ca972b1ff0548a831fe416cec8d39f7e569c94.diff

[PATCH] D123627: Correctly diagnose prototype redeclaration errors in C

efriedma added a comment. In D123627#3451373 , @aaron.ballman wrote: > Thank you for letting me know -- I've speculatively fixed the issue in > 726901d06aab2f92d684d28507711308368c29d6 >

[PATCH] D119136: [clang] Implement Change scope of lambda trailing-return-type

aaron.ballman added inline comments. Comment at: clang/lib/Parse/ParseExprCXX.cpp:1397 +for (LateParsedAttribute *Attr : LateParsedAttrs) { + ParseLambdaLexedAttribute(*Attr, Attributes, D); + delete Attr; cor3ntin wrote: > aaron.ballman

[PATCH] D97121: [clang-tidy] Add a Standalone diagnostics mode to clang-tidy

njames93 added a comment. In D97121#3448467 , @aaron.ballman wrote: > I'd like to hear from @sammccall as well, but I think the changes here LGTM. > Can you please add a release note for the fix? Is this a clang-tidy, clangd or both release notes. The c

[clang] 3efad61 - [HLSL] Pointers are unsupported in HLSL

Author: Chris Bieneman Date: 2022-04-14T13:32:51-05:00 New Revision: 3efad612d234f938d5012317cfb827e2bf8a1ee7 URL: https://github.com/llvm/llvm-project/commit/3efad612d234f938d5012317cfb827e2bf8a1ee7 DIFF: https://github.com/llvm/llvm-project/commit/3efad612d234f938d5012317cfb827e2bf8a1ee7.diff

[PATCH] D123167: [HLSL] Pointers are unsupported in HLSL

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG3efad612d234: [HLSL] Pointers are unsupported in HLSL (authored by beanz). Changed prior to commit: https://reviews.llvm.org/D123167?vs=422860&id=

[PATCH] D123627: Correctly diagnose prototype redeclaration errors in C

aaron.ballman added a comment. In D123627#3452424 , @efriedma wrote: > In D123627#3451373 , @aaron.ballman > wrote: > >> Thank you for letting me know -- I've speculatively fixed the issue in >> 726901d06aab2f92

[PATCH] D97121: [clang-tidy] Add a Standalone diagnostics mode to clang-tidy

aaron.ballman added a comment. In D97121#3452435 , @njames93 wrote: > In D97121#3448467 , @aaron.ballman > wrote: > >> I'd like to hear from @sammccall as well, but I think the changes here LGTM. >> Can you please

[PATCH] D119136: [clang] Implement Change scope of lambda trailing-return-type

aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM, thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119136/new/ https://reviews.llvm.org/D119136 ___ cfe-commits mailing li

[PATCH] D123627: Correctly diagnose prototype redeclaration errors in C

efriedma added a comment. > The reason you get the weird behavior with the note pointing to the same line > as the declaration is because rintf() is a predefined builtin: > https://godbolt.org/z/j3W759M7a (note the same lovely diagnostic note > behavior). It points at the same location, but at

[PATCH] D123345: Treat `std::move`, `forward`, and `move_if_noexcept` as builtins.

aaron.ballman added a comment. In D123345#3446495 , @rsmith wrote: > In D123345#3440935 , @aaron.ballman > wrote: > >> Any chance you'd be interested in submitting this patch to >> http://llvm-compile-time-track

[PATCH] D123627: Correctly diagnose prototype redeclaration errors in C

aaron.ballman added a comment. In D123627#3452491 , @efriedma wrote: >> The reason you get the weird behavior with the note pointing to the same >> line as the declaration is because rintf() is a predefined builtin: >> https://godbolt.org/z/j3W759M7a (n

[PATCH] D123649: Allow flexible array initialization in C++.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG5955a0f9375a: Allow flexible array initialization in C++. (authored by efriedma). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 5955a0f - Allow flexible array initialization in C++.

Author: Eli Friedman Date: 2022-04-14T11:56:40-07:00 New Revision: 5955a0f9375a8c0b134eeb4a8de5155dcce7c94f URL: https://github.com/llvm/llvm-project/commit/5955a0f9375a8c0b134eeb4a8de5155dcce7c94f DIFF: https://github.com/llvm/llvm-project/commit/5955a0f9375a8c0b134eeb4a8de5155dcce7c94f.diff

[PATCH] D123763: [randstruct] Enforce using a designated init for a randomized struct

void updated this revision to Diff 422934. void marked an inline comment as done. void added a comment. Accept initialzers of the forms: struct foo f = {}; struct foo g = {0}; Move tests to lit tester. Improve error message. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D123763: [randstruct] Enforce using a designated init for a randomized struct

void updated this revision to Diff 422936. void marked an inline comment as done. void added a comment. Add some smoke tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123763/new/ https://reviews.llvm.org/D123763 Files: clang/include/clang/B

[PATCH] D123763: [randstruct] Enforce using a designated init for a randomized struct

void added a comment. Updated with those changes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123763/new/ https://reviews.llvm.org/D123763 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:

[PATCH] D123810: [Cuda] Add initial support for wrapping CUDA images in the new driver.

jhuber6 created this revision. jhuber6 added reviewers: jdoerfert, JonChesterfield, tra, yaxunl. Herald added a subscriber: carlosgalvezp. Herald added a project: All. jhuber6 requested review of this revision. Herald added subscribers: cfe-commits, sstefan1. Herald added a project: clang. This pa

[PATCH] D123812: [CUDA] Add wrapper code generation for registering CUDA images

jhuber6 created this revision. jhuber6 added reviewers: jdoerfert, JonChesterfield, tra, yaxunl. Herald added a subscriber: carlosgalvezp. Herald added a project: All. jhuber6 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch adds

[PATCH] D123627: Correctly diagnose prototype redeclaration errors in C

uweigand added a comment. In D123627#3451373 , @aaron.ballman wrote: > Thank you for letting me know -- I've speculatively fixed the issue in > 726901d06aab2f92d684d28507711308368c29d6 >

[PATCH] D123405: [dllexport] odr-use constexpr default args for constructor closures

rnk added a comment. Looks good to me after the fact. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123405/new/ https://reviews.llvm.org/D123405 ___ cfe-commits mailing list cfe-commits@lists.llvm.org ht

[PATCH] D123627: Correctly diagnose prototype redeclaration errors in C

aaron.ballman added a comment. In D123627#3452572 , @uweigand wrote: > In D123627#3451373 , @aaron.ballman > wrote: > >> Thank you for letting me know -- I've speculatively fixed the issue in >> 726901d06aab2f92

[PATCH] D123636: [randstruct] Add test for "-frandomize-layout-seed-file" flag

MaskRay added inline comments. Comment at: clang/unittests/AST/RandstructTest.cpp:40 -static std::unique_ptr makeAST(const std::string &SourceCode) { +static std::string Seed = "1234567890abcdef"; + `constexpr llvm::StringLiteral Seed = ...` or `constexpr char

[clang] 7726ad0 - [PS5] Add basic PS5 driver behavior

Author: Paul Robinson Date: 2022-04-14T12:45:33-07:00 New Revision: 7726ad04e2633a843fb9743e5adfbb404446e657 URL: https://github.com/llvm/llvm-project/commit/7726ad04e2633a843fb9743e5adfbb404446e657 DIFF: https://github.com/llvm/llvm-project/commit/7726ad04e2633a843fb9743e5adfbb404446e657.diff

[PATCH] D123601: [clang][AArch64] Split neon tests into 2 files

tstellar added a comment. Ok, so maybe my concerns about testing overhead are not that legitimate. That's just something I've heard mentioned in the past. However, I still think this should be fixed in buildkite. It's seems much more sustainable to just bump the timeout in buildkite rather

[PATCH] D123812: [CUDA] Add wrapper code generation for registering CUDA images

jhuber6 added a comment. For reference, this is how I used this driver to offload a CUDA and OpenMP offload kernel that both called an external "hello world" function in `device.cu`. $ clang++ hello.cu device.cu -foffload-new-driver --offload-arch=sm_70 -c $ clang++ openmp.cpp -fopenmp-new-

[PATCH] D123812: [CUDA] Add wrapper code generation for registering CUDA images

tra added inline comments. Comment at: clang/tools/clang-linker-wrapper/OffloadWrapper.cpp:351 +/// required for texture / surface / managed variables. +Function *createRegisterGlobalsFunction(Module &M) { + LLVMContext &C = M.getContext(); Do you think ge

[PATCH] D123812: [CUDA] Add wrapper code generation for registering CUDA images

jhuber6 added inline comments. Comment at: clang/tools/clang-linker-wrapper/OffloadWrapper.cpp:351 +/// required for texture / surface / managed variables. +Function *createRegisterGlobalsFunction(Module &M) { + LLVMContext &C = M.getContext(); tra wrote:

[clang] 6cf0b1b - Comment out assertions about initializer size added in D123649.

Author: Eli Friedman Date: 2022-04-14T13:58:17-07:00 New Revision: 6cf0b1b3da3e8662baf030a2d741e3becaaab2b0 URL: https://github.com/llvm/llvm-project/commit/6cf0b1b3da3e8662baf030a2d741e3becaaab2b0 DIFF: https://github.com/llvm/llvm-project/commit/6cf0b1b3da3e8662baf030a2d741e3becaaab2b0.diff

[PATCH] D123649: Allow flexible array initialization in C++.

efriedma added a comment. Apparently the assertions I added are triggering in the LLVM test-suite. Commented them out in 6cf0b1b3da3e8662baf030a2d741e3becaaab2b0 ; I'll come up with a proper fix soon. Repository: rG LLVM

[PATCH] D123808: [clang] Fix the bogus diagnostic introduced by the newly-added UsingTemplate Kind.

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Sema/SemaDeclCXX.cpp:11014 // Check that the return type is written as a specialization of // the template specified as the deductio

[PATCH] D121176: [ASTStructuralEquivalence] Add support for comparing ObjCCategoryDecl.

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

[PATCH] D123812: [CUDA] Add wrapper code generation for registering CUDA images

tra added inline comments. Comment at: clang/tools/clang-linker-wrapper/OffloadWrapper.cpp:351 +/// required for texture / surface / managed variables. +Function *createRegisterGlobalsFunction(Module &M) { + LLVMContext &C = M.getContext(); jhuber6 wrote:

[PATCH] D123642: [clang codegen] Assume arguments of __atomic_* are aligned.

rjmccall added a comment. The GCC documentation really ought to specify what alignment assumptions these functions make, if any. But they control the specification here, so if they're making alignment assumptions, then I agre

[PATCH] D123636: [randstruct] Add test for "-frandomize-layout-seed-file" flag

void marked an inline comment as done. void added inline comments. Comment at: clang/unittests/AST/RandstructTest.cpp:72 + + return std::tuple(AST.release(), ASTFileSeed.release()); +}; MaskRay wrote: > aaron.ballman wrote: > > void wrote: > > > aaron.ballman w

[PATCH] D123636: [randstruct] Add test for "-frandomize-layout-seed-file" flag

MaskRay added inline comments. Comment at: clang/unittests/AST/RandstructTest.cpp:72 + + return std::tuple(AST.release(), ASTFileSeed.release()); +}; void wrote: > MaskRay wrote: > > aaron.ballman wrote: > > > void wrote: > > > > aaron.ballman wrote: > > > > >

[PATCH] D123498: [clang] Adding Platform/Architecture Specific Resource Header Installation Targets

qiongsiwu1 updated this revision to Diff 422959. qiongsiwu1 added a comment. Thanks @DavidSpickett !! The patch is updated. The `arm` headers are now split between `arm` and `aarch64`. > Also, what's the feedback mechanism for issues with these listings? Say we > add a new Arm header but forget

[PATCH] D123812: [CUDA] Add wrapper code generation for registering CUDA images

jhuber6 added inline comments. Comment at: clang/tools/clang-linker-wrapper/OffloadWrapper.cpp:351 +/// required for texture / surface / managed variables. +Function *createRegisterGlobalsFunction(Module &M) { + LLVMContext &C = M.getContext(); tra wrote:

[PATCH] D123636: [randstruct] Add test for "-frandomize-layout-seed-file" flag

void updated this revision to Diff 422965. void added a comment. Use "unique_ptr" when possible. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123636/new/ https://reviews.llvm.org/D123636 Files: clang/unittests/AST/RandstructTest.cpp Index: cla

[PATCH] D123636: [randstruct] Add test for "-frandomize-layout-seed-file" flag

void added inline comments. Comment at: clang/unittests/AST/RandstructTest.cpp:72 + + return std::tuple(AST.release(), ASTFileSeed.release()); +}; MaskRay wrote: > void wrote: > > MaskRay wrote: > > > aaron.ballman wrote: > > > > void wrote: > > > > > aaron.bal

[PATCH] D123345: Treat `std::move`, `forward`, and `move_if_noexcept` as builtins.

rsmith added a comment. In D123345#3452496 , @aaron.ballman wrote: > Do you have ideas on how we can improve the debugging checkpoint behavior (if > at all)? I think we just live with it, like we do for other builtin functions. (There might be things

[clang] f20968e - [Driver] Remove unneeded -f[no-]pascal-strings translation. NFC

Author: Fangrui Song Date: 2022-04-14T15:20:58-07:00 New Revision: f20968e00667955f9911864ea922d5a24f548d0d URL: https://github.com/llvm/llvm-project/commit/f20968e00667955f9911864ea922d5a24f548d0d DIFF: https://github.com/llvm/llvm-project/commit/f20968e00667955f9911864ea922d5a24f548d0d.diff

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

aprantl added inline comments. Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:1684 + +if (AT->isDeduced() && ThisPtr->getPointeeCXXRecordDecl()->isLambda()) + Elts.push_back(getOrCreateType(AT->getDeducedType(),Unit)); aprantl wrote: > Can you add a comme

[PATCH] D123787: [clang][OpenMP][DebugInfo] Debug support for TLS variables when present in OpenMP consructs

aprantl added a comment. > For an example, if thread local variable is present in copyin clause > (testcase attached with the patch), parameter with same name is generated as parameter to artificial function. When user inquires the thread Local variable, its debug info is hidden by the paramete

[PATCH] D123636: [randstruct] Add test for "-frandomize-layout-seed-file" flag

void updated this revision to Diff 422976. void added a comment. Rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123636/new/ https://reviews.llvm.org/D123636 Files: clang/unittests/AST/RandstructTest.cpp Index: clang/unittests/AST/Randstr

[clang] 31ea479 - [randstruct] Add test for "-frandomize-layout-seed-file" flag

Author: Bill Wendling Date: 2022-04-14T15:41:25-07:00 New Revision: 31ea4798ad0990838ccd27f80ca112f177ce91d9 URL: https://github.com/llvm/llvm-project/commit/31ea4798ad0990838ccd27f80ca112f177ce91d9 DIFF: https://github.com/llvm/llvm-project/commit/31ea4798ad0990838ccd27f80ca112f177ce91d9.diff

[PATCH] D123636: [randstruct] Add test for "-frandomize-layout-seed-file" flag

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG31ea4798ad09: [randstruct] Add test for "-frandomize-layout-seed-file" flag (authored by void). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D122983: [C11/C2x] Change the behavior of the implicit function declaration warning

rsmith added a comment. In D122983#3451920 , @erichkeane wrote: > I think Aaron's approach provides a proper 'depreciation' period in our > compiler, as best as we have the ability to do. We've been warning by default for a decade that this code is not

[PATCH] D123825: clang/AMDGPU: Define macro for -munsafe-fp-atomics

arsenm created this revision. arsenm added reviewers: yaxunl, rampitec. Herald added subscribers: kerbowa, t-tye, tpr, dstuttard, jvesely, kzhuravl. Herald added a project: All. arsenm requested review of this revision. Herald added a subscriber: wdng. The HIP headers want to use this to swap the

[PATCH] D123825: clang/AMDGPU: Define macro for -munsafe-fp-atomics

rampitec accepted this revision. rampitec added a comment. This revision is now accepted and ready to land. Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123825/new/ https://reviews.llvm.org/D123825 ___ cfe-commits mailing list cfe-co

[PATCH] D123826: Fix size of flexible array initializers, and re-enable assertions.

efriedma created this revision. efriedma added a reviewer: erichkeane. Herald added a project: All. efriedma requested review of this revision. Herald added a project: clang. In D123649 , I got the formula for getFlexibleArrayInitChars slightly wrong: the flexibl

[clang] 27dead3 - Revert "[randstruct] Add test for "-frandomize-layout-seed-file" flag"

Author: Bill Wendling Date: 2022-04-14T16:07:00-07:00 New Revision: 27dead3e3a532625704654a0b66d77f8654d1af2 URL: https://github.com/llvm/llvm-project/commit/27dead3e3a532625704654a0b66d77f8654d1af2 DIFF: https://github.com/llvm/llvm-project/commit/27dead3e3a532625704654a0b66d77f8654d1af2.diff

[PATCH] D80344: [Windows SEH]: HARDWARE EXCEPTION HANDLING (MSVC -EHa) - Part 1

tentzen added a comment. In D80344#3450940 , @bowang wrote: > Any updates on this patch? Would like to see it moving forward. > > I was trying to use Google Breakpad with Clang. Breakpad is a crash handling > library that generates a minidump at crash. Th

[PATCH] D123345: Treat `std::move`, `forward`, and `move_if_noexcept` as builtins.

rnk added a comment. Generally speaking, this sounds like a good idea to me. One time in 2019 I used -ftime-trace+ClangBuildAnalyzer and it told me that std::unique_ptr was the most expensive template because it is instantiated so m

[clang] 8c77a75 - [randstruct] Add test for "-frandomize-layout-seed-file" flag

Author: Bill Wendling Date: 2022-04-14T16:35:41-07:00 New Revision: 8c77a75fb6a82a4cc2182bca89e007e7190a83de URL: https://github.com/llvm/llvm-project/commit/8c77a75fb6a82a4cc2182bca89e007e7190a83de DIFF: https://github.com/llvm/llvm-project/commit/8c77a75fb6a82a4cc2182bca89e007e7190a83de.diff

[PATCH] D120639: [RISCV] Pass -mno-relax to assembler when -fno-integrated-as specified

MaskRay accepted this revision. MaskRay added a comment. LGTM if you use `addOptOutFlag` Comment at: clang/lib/Driver/ToolChains/Gnu.cpp:764 CmdArgs.push_back(MArchName.data()); +if (!Args.hasFlag(options::OPT_mrelax, options::OPT_mno_relax, true)) + CmdArgs.push_

[PATCH] D112921: [clang] Enable sized deallocation by default in C++14 onwards

MaskRay added a comment. Beside the concern raised by platform maintainers: the cc1 default switch part should be made separately from the patch. This makes revert easy and leaves fewer churn to the test suite. If -fno-sized-deallocation is a better cc1 default (but not Driver's), I can make su

[PATCH] D123300: [Clang] Enable opaque pointers by default

manojgupta added a comment. We noticed a new crash that still repros at head. clang -target armv7a-linux-gnueabihf -march=armv8a -mthumb -c -O2 file.cc llvm-project/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:10135: llvm::Value *llvm::VPTransformState::get(llvm::VPValue *, unsigned int): As

[PATCH] D123403: [OpenMP] Refactor OMPScheduleType enum.

peixin accepted this revision. peixin added inline comments. This revision is now accepted and ready to land. Comment at: llvm/include/llvm/Frontend/OpenMP/OMPConstants.h:135 + UnorderedGuidedSimd = BaseGuidedSimd | ModifierUnordered, // (46) + UnorderedRuntimeSimd = BaseRunt

[clang] a1303b2 - clang/AMDGPU: Define macro for -munsafe-fp-atomics

Author: Matt Arsenault Date: 2022-04-14T22:04:59-04:00 New Revision: a1303b23c9de6ef6d667aa923ec266ca4a0334e7 URL: https://github.com/llvm/llvm-project/commit/a1303b23c9de6ef6d667aa923ec266ca4a0334e7 DIFF: https://github.com/llvm/llvm-project/commit/a1303b23c9de6ef6d667aa923ec266ca4a0334e7.diff

[PATCH] D123825: clang/AMDGPU: Define macro for -munsafe-fp-atomics

arsenm closed this revision. arsenm added a comment. a1303b23c9de6ef6d667aa923ec266ca4a0334e7 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123825/new/ https://reviews.llvm.org/D123825 __

[PATCH] D123498: [clang] Adding Platform/Architecture Specific Resource Header Installation Targets

qiongsiwu1 updated this revision to Diff 422996. qiongsiwu1 added a comment. > Also, what's the feedback mechanism for issues with these listings? Say we > add a new Arm header but forget to put it on these lists, who would be the > first to notice? (I admit I'm not very familiar with the proces

[PATCH] D123831: [POC][WIP] Use relative include in extract-api

zixuw created this revision. zixuw added reviewers: ributzka, dang, cishida. Herald added a project: All. zixuw requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Proof-of-concept patch. Needs more work. Transform input headers to relative (an

[PATCH] D123831: [POC][WIP] Use relative include in extract-api

zixuw added inline comments. Comment at: clang/lib/ExtractAPI/ExtractAPIConsumer.cpp:87 +Dir = DirPath; +for (auto NI = path::begin(File), NE = path::end(File), + DI = path::begin(Dir), DE = path::end(Dir); I still need to dig into this logic

[PATCH] D123831: [POC][WIP] Use relative include in extract-api

zixuw updated this revision to Diff 423011. zixuw added a comment. Add test case to demonstrate the framework case. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123831/new/ https://reviews.llvm.org/D123831 Files: clang/lib/ExtractAPI/ExtractAPI

[PATCH] D122409: [libclang] Add missing CursorKind enums defined in Index.h.

sighingnow added a comment. Ping~ Could anyone take a look at this trivial patch and get it landed? Thanks in advance! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122409/new/ https://reviews.llvm.org/D122409 ___

[PATCH] D122409: [libclang] Add missing CursorKind enums defined in Index.h.

sighingnow added a comment. /cc @arphaman @jdoerfert Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122409/new/ https://reviews.llvm.org/D122409 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[PATCH] D123836: [Driver] Move Lanai IAS enabling to Generic_GCC::IsIntegratedAssemblerDefault, NFC

brad created this revision. brad added a reviewer: MaskRay. brad added a project: clang. Herald added a subscriber: StephenFan. Herald added a project: All. brad requested review of this revision. Move enabling of IAS back to Generic_GCC::IsIntegratedAssemblerDefault(). Repository: rG LLVM Git

[PATCH] D123837: [C++20] [Modules] Judge isInCurrentModule currently

ChuanqiXu created this revision. ChuanqiXu added reviewers: iains, rsmith, clang-language-wg. ChuanqiXu added a project: clang. Herald added a subscriber: dexonsmith. Herald added a project: All. ChuanqiXu requested review of this revision. Herald added a subscriber: cfe-commits. Now the implement

<    1   2