[PATCH] D144967: [PowerPC] Recognize long CPU name for -mtune in Clang

MaskRay added a comment. In D144967#4164858 , @nathanchance wrote: > Could this be merged into `main` and backported to `release/16.x`? If this > makes 16.0.0 final, I think the kernel can avoid working around this issue > altogether, as `-mtune` was o

[PATCH] D144967: [PowerPC] Recognize long CPU name for -mtune in Clang

nemanjai added a comment. I'll provide an updated patch in a few min... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144967/new/ https://reviews.llvm.org/D144967 ___ cfe-commits mailing list cfe-commits

[PATCH] D144967: [PowerPC] Recognize long CPU name for -mtune in Clang

nemanjai added a comment. In D144967#4165140 , @nemanjai wrote: > I'll provide an updated patch in a few min... Ah, what I mean is I will update the patch and push in a few min. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https:/

[PATCH] D122258: [MC] Omit DWARF unwind info if compact unwind is present where eligible

jyknight added a comment. > no-compact-unwind is particularly useful for newer x86_64 platforms: we don't > want to omit DWARF unwind for x86_64 in general due to possible backwards > compat issues, but we should make it possible for people to opt into this > behavior if they are only targeting

[PATCH] D145173: Make section attribute and -ffunction-sections play nicely

probinson created this revision. probinson added a reviewer: MaskRay. Herald added a project: All. probinson requested review of this revision. People use -ffunction-sections to put each function into its own object-file section; this makes linker garbage-collection simpler. However, if there's an

[PATCH] D145093: Add map info for dereference pointer.

ABataev added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:7487-7497 + auto TNext = Next; + bool IsVarDerefAssoWithArray = false; + if (UO && UO->getOpcode() == UO_Deref) +for (; TNext != CE; TNext = std::next(TNext)) + if (is

[PATCH] D145088: [RISCV] Add attribute(riscv_rvv_vector_bits(N)) based on AArch64 arm_sve_vector_bits.

tschuett added a comment. Any binary that uses this feature is not forward portable to hardware with a larger vector size. That's true for SVE as well. I did not understood this sentence. AFAIK, SVE uses the ptrue instruction to generate a mask to only activate the necessary lanes. If I do

[PATCH] D145021: [Clang][AIX][p] Claim -p in front end

francii updated this revision to Diff 501929. francii added a comment. Update based on review Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145021/new/ https://reviews.llvm.org/D145021 Files: clang/lib/Driver/ToolChains/AIX.cpp clang/lib/Drive

[PATCH] D144884: [clang-format] Only add pragma continuation indentation for 'omp' clauses

MyDeveloperDay added a comment. without this change what does this look like? EXPECT_EQ( "#pragma omp target \\\n" "reduction(+ : var) \\\n" "map(to : A[0 : N]) \\\n" "map(to : B[0 : N]) \\\n" "map(from : C[0 : N]) \\\n" "first

[PATCH] D145088: [RISCV] Add attribute(riscv_rvv_vector_bits(N)) based on AArch64 arm_sve_vector_bits.

craig.topper added a comment. In D145088#4165180 , @tschuett wrote: > Any binary that uses this feature is not forward portable to hardware > with a larger vector size. That's true for SVE as well. > > I did not understood this sentence. AFAIK, SVE us

[PATCH] D145164: [clang][RISCV] Enable -fasynchronous-unwind-tables by default on Linux

MaskRay accepted this revision. MaskRay added a comment. This revision is now accepted and ready to land. LG since GCC made a similar change and this matches several major architectures on Linux/other ELF based operating systems. (At heart I think `-fasynchronous-unwind-tables` is not a good def

[PATCH] D144884: [clang-format] Only add pragma continuation indentation for 'omp' clauses

jhuber6 added a comment. In D144884#4165192 , @MyDeveloperDay wrote: > without this change what does this look like? > > EXPECT_EQ( > "#pragma omp target \\\n" > "reduction(+ : var) \\\n" > "map(to : A[0 : N]) \\\n"

[PATCH] D144309: [HLSL] add max/min library functions

bob80905 updated this revision to Diff 501933. bob80905 added a comment. - format Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144309/new/ https://reviews.llvm.org/D144309 Files: clang/lib/Headers/hlsl/hlsl_intrinsics.h clang/test/CodeGenHLSL

[clang] 59cd692 - [PowerPC] Recognize long CPU name for -mtune in Clang

Author: Nemanja Ivanovic Date: 2023-03-02T14:29:40-05:00 New Revision: 59cd692454c9430f0fb77ca14b65cb9afcfe2776 URL: https://github.com/llvm/llvm-project/commit/59cd692454c9430f0fb77ca14b65cb9afcfe2776 DIFF: https://github.com/llvm/llvm-project/commit/59cd692454c9430f0fb77ca14b65cb9afcfe2776.di

[PATCH] D144967: [PowerPC] Recognize long CPU name for -mtune in Clang

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG59cd692454c9: [PowerPC] Recognize long CPU name for -mtune in Clang (authored by nemanjai). Changed prior to commit: https://reviews.llvm.org/D144

[PATCH] D145093: Add map info for dereference pointer.

jyu2 added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:7487-7497 + auto TNext = Next; + bool IsVarDerefAssoWithArray = false; + if (UO && UO->getOpcode() == UO_Deref) +for (; TNext != CE; TNext = std::next(TNext)) + if (isa(T

[clang] 5680b75 - [C2x] Add tests for WG14 N3035 and update the C status page

Author: Aaron Ballman Date: 2023-03-02T14:36:50-05:00 New Revision: 5680b7570342c3457b80d3129fe60e53ef7ddfd5 URL: https://github.com/llvm/llvm-project/commit/5680b7570342c3457b80d3129fe60e53ef7ddfd5 DIFF: https://github.com/llvm/llvm-project/commit/5680b7570342c3457b80d3129fe60e53ef7ddfd5.diff

[PATCH] D144309: [HLSL] add max/min library functions

python3kgae added a comment. In D144309#4159431 , @bob80905 wrote: > - ushort is now unsigned Use uint16_t/int16_t/uint64_t/int64_t for 16/64 bit integers. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144

[PATCH] D143418: [libclang] Add API to override preamble storage path

aaron.ballman added inline comments. Comment at: clang/include/clang-c/Index.h:329 + * CXIndexOptions Opts = { sizeof(CXIndexOptions), + * clang_getDefaultGlobalOptions() }; + * \endcode vedgy wrote: > vedgy wrote: > > aaron.ballman wrote:

[PATCH] D145093: Add map info for dereference pointer.

ABataev added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:7487-7497 + auto TNext = Next; + bool IsVarDerefAssoWithArray = false; + if (UO && UO->getOpcode() == UO_Deref) +for (; TNext != CE; TNext = std::next(TNext)) + if (is

[clang] d14e7ee - [clang-format] Add -j to clang-format-diff to speed up formatting

Author: Sean Maher Date: 2023-03-02T11:59:11-08:00 New Revision: d14e7ee3d17cfa60d44256d742c10e9949a6048f URL: https://github.com/llvm/llvm-project/commit/d14e7ee3d17cfa60d44256d742c10e9949a6048f DIFF: https://github.com/llvm/llvm-project/commit/d14e7ee3d17cfa60d44256d742c10e9949a6048f.diff LO

[PATCH] D141230: [clang-format-diff.py] give clang-format-diff a job pool (10x speed)

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd14e7ee3d17c: [clang-format] Add -j to clang-format-diff to speed up formatting (authored by seanptmaher, committed by owenpan). Repository: rG LL

[clang] 554ba99 - Revert "[Clang] Refactor "Designators" into a unified implementation [NFC]"

Author: Bill Wendling Date: 2023-03-02T12:09:29-08:00 New Revision: 554ba996956559acdab777d18999b9985d95be4c URL: https://github.com/llvm/llvm-project/commit/554ba996956559acdab777d18999b9985d95be4c DIFF: https://github.com/llvm/llvm-project/commit/554ba996956559acdab777d18999b9985d95be4c.diff

[PATCH] D138539: Use std::nullopt_t instead of NoneType (NFC)

MaskRay added a comment. In D138539#4164313 , @steakhal wrote: > Oh, I forgot to mention why this change breaks the equality operator of > `llvm::StringSet`. It's because the `StringMap::operator==` will try to > compare the `value` as well, which is of

[PATCH] D145093: Add map info for dereference pointer.

jyu2 added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:7487-7497 + auto TNext = Next; + bool IsVarDerefAssoWithArray = false; + if (UO && UO->getOpcode() == UO_Deref) +for (; TNext != CE; TNext = std::next(TNext)) + if (isa(T

[PATCH] D145093: Add map info for dereference pointer.

ABataev added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:7487-7497 + auto TNext = Next; + bool IsVarDerefAssoWithArray = false; + if (UO && UO->getOpcode() == UO_Deref) +for (; TNext != CE; TNext = std::next(TNext)) + if (is

[PATCH] D145178: [clang][NFC] reformats the SARIF diagnostic test so it's human readable

cjdb created this revision. cjdb added reviewers: aaron.ballman, shafik, erichkeane. Herald added a project: All. cjdb requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The current FileCheck test output is very difficult to read, which makes

[PATCH] D145178: [clang][NFC] reformats the SARIF diagnostic test so it's human readable

cjdb added inline comments. Comment at: clang/test/Frontend/sarif-diagnostics.cpp:31 +// RUN: %clang -fsyntax-only -Wall -Wextra -fdiagnostics-format=sarif-stderr %s > %t.txt 2>&1 || true +// RUN: FileCheck -dump-input=always %s --input-file=%t.txt --check-prefixes=CHECK,COMMON

[PATCH] D145178: [clang][NFC] reformats the SARIF diagnostic test so it's human readable

cjdb added inline comments. Comment at: clang/test/Frontend/sarif-diagnostics.cpp:31 +// RUN: %clang -fsyntax-only -Wall -Wextra -fdiagnostics-format=sarif-stderr %s > %t.txt 2>&1 || true +// RUN: FileCheck -dump-input=always %s --input-file=%t.txt --check-prefixes=CHECK,COMMON

[PATCH] D145178: [clang][NFC] reformats the SARIF diagnostic test so it's human readable

cjdb updated this revision to Diff 501955. cjdb added a comment. renames check identifiers Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145178/new/ https://reviews.llvm.org/D145178 Files: clang/test/Frontend/sarif-diagnostics.cpp Index: clang/

[PATCH] D144454: Add builtin for llvm set rounding

rjmccall added a comment. `__builtin_set_flt_rounds` seems better. We should add the portability checking, yeah. Look at the checking we do for certain builtins with `CheckBuiltinTargetInSupported` in `SemaChecking.cpp`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https

[PATCH] D144309: [HLSL] add max/min library functions

bob80905 updated this revision to Diff 501963. bob80905 added a comment. - use already defined hlsl types Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144309/new/ https://reviews.llvm.org/D144309 Files: clang/lib/Headers/hlsl/hlsl_intrinsics.h

[PATCH] D144309: [HLSL] add max/min library functions

python3kgae added inline comments. Comment at: clang/test/CodeGenHLSL/builtins/max.hlsl:12 +// NO_HALF: call i16 @llvm.smax.i16( +int16_t test_max_short ( int16_t p0, int16_t p1 ) { + return max ( p0, p1 ); Just guard 16bit integer tests with #ifdef __HLSL_ENABL

[clang] 8640545 - [clang][deps] Preserve input ordering in the full output

Author: Jan Svoboda Date: 2023-03-02T13:42:11-08:00 New Revision: 864054500830d708cc9f7d8a53195a95488e4e0b URL: https://github.com/llvm/llvm-project/commit/864054500830d708cc9f7d8a53195a95488e4e0b DIFF: https://github.com/llvm/llvm-project/commit/864054500830d708cc9f7d8a53195a95488e4e0b.diff L

[PATCH] D145098: [clang][deps] Preserve input ordering in the full output

This revision was automatically updated to reflect the committed changes. Closed by commit rG864054500830: [clang][deps] Preserve input ordering in the full output (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145098/new/

[PATCH] D144977: [analyzer] Fix of the initialization list parsing.

earnol planned changes to this revision. earnol added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/RegionStore.cpp:564-565 - SVal getBindingForElement(RegionBindingsConstRef B, const ElementRegion *R); + SVal getBindingForElement(RegionBindingsConstRef B, const

[clang] 9679075 - [clang][deps] NFC: Simplify worker loop

Author: Jan Svoboda Date: 2023-03-02T13:50:08-08:00 New Revision: 9679075a1ae6a20ed102597bf166b3f3adc95499 URL: https://github.com/llvm/llvm-project/commit/9679075a1ae6a20ed102597bf166b3f3adc95499 DIFF: https://github.com/llvm/llvm-project/commit/9679075a1ae6a20ed102597bf166b3f3adc95499.diff L

[PATCH] D144603: Add option to disable compiler launcher on external projects

fangism added a comment. In D144603#4163751 , @phosek wrote: > In D144603#4162974 , @haowei wrote: > >> I think remove the compiler launcher from default pass through flags are >> fine. What about the using compi

[PATCH] D145101: [clang][deps] NFC: Simplify worker loop

This revision was automatically updated to reflect the committed changes. Closed by commit rG9679075a1ae6: [clang][deps] NFC: Simplify worker loop (authored by jansvoboda11). Changed prior to commit: https://reviews.llvm.org/D145101?vs=501616&id=501969#toc Repository: rG LLVM Github Monorepo

[PATCH] D145187: [Fuchsia] Add other necessary components to LLDB install.

mysterymath created this revision. mysterymath added reviewers: phosek, haowei. Herald added a subscriber: abrachet. Herald added a project: All. mysterymath requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monor

[PATCH] D145178: [clang][NFC] reformats the SARIF diagnostic test so it's human readable

cjdb updated this revision to Diff 501982. cjdb added a comment. removes redundant line Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145178/new/ https://reviews.llvm.org/D145178 Files: clang/test/Frontend/sarif-diagnostics.cpp Index: clang/tes

[PATCH] D144977: [analyzer] Fix of the initialization list parsing.

earnol requested review of this revision. earnol added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/RegionStore.cpp:1787 + if (V && + (!targetType->isStructureOrClassType() && !targetType->isUnionType())) return *V; isuckatcs wrote: > I a

[PATCH] D145197: [clang][deps] NFC: Refactor and comment ModuleDeps sorting

jansvoboda11 created this revision. jansvoboda11 added a reviewer: Bigcheese. Herald added subscribers: ributzka, mgrang. Herald added a project: All. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. I once again stumbled across

[PATCH] D144672: [Sanitizers] Error when attempting to use `static-lsan` with `TSan` or `Asan` on darwin

yln added inline comments. Comment at: compiler-rt/test/asan/TestCases/replaceable_new_delete_static.cpp:10-11 + +// darwin only supports shared-libsan, so this should fail. +// XFAIL: darwin + This should work, right? Repository: rG LLVM Github Monorepo C

[PATCH] D145021: [Clang][AIX][p] Claim -p in front end

francii updated this revision to Diff 502006. francii added a comment. Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145021/new/ https://reviews.llvm.org/D145021 Files: clang/lib/Driver/ToolChains/AIX.cpp clang/lib/Driver/ToolChains/Cla

[PATCH] D144309: [HLSL] add max/min library functions

bob80905 updated this revision to Diff 502009. bob80905 added a comment. - only run tests if 16 bit is enabled Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144309/new/ https://reviews.llvm.org/D144309 Files: clang/lib/Headers/hlsl/hlsl_intrinsi

[PATCH] D145201: [clang] fixes header processing for `-fdiagnostics-format=sarif`

cjdb created this revision. cjdb added reviewers: aaron.ballman, shafik, erichkeane. Herald added a project: All. cjdb requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Including headers used to fire an assertion; now they report a diagnostic

[PATCH] D144709: [clang-format] Improve QualifierAlignment

MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land. Firstly let me say thank you for this review, your perseverance demonstrates to me that there is a desire to make this capability better, which tells me that a feature that ult

[PATCH] D144672: [Sanitizers] Error when attempting to use `static-lsan` with `TSan` or `Asan` on darwin

dmaclach added inline comments. Comment at: compiler-rt/test/asan/TestCases/replaceable_new_delete_static.cpp:10-11 + +// darwin only supports shared-libsan, so this should fail. +// XFAIL: darwin + yln wrote: > This should work, right? No.. darwin should fail

[PATCH] D145150: clang: Emit nofpclass(nan inf) for -ffinite-math-only

arsenm updated this revision to Diff 502015. arsenm added a comment. Drop todo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145150/new/ https://reviews.llvm.org/D145150 Files: clang/lib/CodeGen/CGCall.cpp clang/test/CodeGen/complex-math.c clang/test/CodeGen/fp-function-attrs.cpp

[PATCH] D144672: [Sanitizers] Error when attempting to use `static-lsan` with `TSan` or `Asan` on darwin

zixuw added inline comments. Comment at: compiler-rt/test/asan/TestCases/replaceable_new_delete_static.cpp:10-11 + +// darwin only supports shared-libsan, so this should fail. +// XFAIL: darwin + dmaclach wrote: > yln wrote: > > This should work, right? > No..

[PATCH] D144672: [Sanitizers] Error when attempting to use `static-lsan` with `TSan` or `Asan` on darwin

yln added inline comments. Comment at: compiler-rt/test/asan/TestCases/replaceable_new_delete_static.cpp:10-11 + +// darwin only supports shared-libsan, so this should fail. +// XFAIL: darwin + zixuw wrote: > dmaclach wrote: > > yln wrote: > > > This should wor

[PATCH] D145178: [clang][NFC] reformats the SARIF diagnostic test so it's human readable

cjdb updated this revision to Diff 502019. cjdb added a comment. removes something from a future commit This should be the final alteration pre-review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145178/new/ https://reviews.llvm.org/D145178 Fil

[PATCH] D145201: [clang] fixes header processing for `-fdiagnostics-format=sarif`

cjdb updated this revision to Diff 502020. cjdb added a comment. actually commits the files Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145201/new/ https://reviews.llvm.org/D145201 Files: clang/include/clang/Frontend/SARIFDiagnostic.h clang/

[PATCH] D145201: [clang] fixes header processing for `-fdiagnostics-format=sarif`

cjdb updated this revision to Diff 502021. cjdb added a comment. fixes something that wasn't supposed to be changed in the rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145201/new/ https://reviews.llvm.org/D145201 Files: clang/include/cla

[PATCH] D145047: Fix broken link on Clang documentation page

tupaschoal added a comment. In D145047#4164976 , @royjacobson wrote: > Thanks! Do you need me to land this for you? If so, could you please provide > name + email to sign the commit with? Yes, please! Tulio Leao Repository: rG LLVM Github Monorepo

[PATCH] D144977: [analyzer] Fix of the initialization list parsing.

isuckatcs added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/CStringChecker.cpp:432 + const LocationContext *LCtx = C.getLocationContext(); + SVal SrcVal = State->getSVal(Buffer.Expression, LCtx); + QualType SourceValType = SrcVal.getType(C.getAST

[PATCH] D143128: [-Wunsafe-buffer-usage] Fix-Its transforming `&DRE[any]` to `(DRE.data() + any)`

NoQ added a comment. In D143128#4108502 , @ziqingluo-90 wrote: > In D143128#4108375 , @NoQ wrote: > >> Why do we prefer `DRE.data() + any` to `&DRE.data()[any]`? It could be much >> less intrusive this way, and

[clang] bf52ead - [C++20] [Modules] Support to export declarations in language linkage

Author: Chuanqi Xu Date: 2023-03-03T10:31:48+08:00 New Revision: bf52ead24ca4fe1b73bceec7bba3abfe15541649 URL: https://github.com/llvm/llvm-project/commit/bf52ead24ca4fe1b73bceec7bba3abfe15541649 DIFF: https://github.com/llvm/llvm-project/commit/bf52ead24ca4fe1b73bceec7bba3abfe15541649.diff LO

[PATCH] D144367: [C++20] [Modules] Support to export declarations in the language linkage

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGbf52ead24ca4: [C++20] [Modules] Support to export declarations in language linkage (authored by ChuanqiXu). Herald added a project: clang. Herald add

[clang] 5828692 - [C++20] [Modules] Make TheImplicitGlobalModuleFragment and TheExportedImplicitGlobalModuleFragment to be useable modules

Author: Chuanqi Xu Date: 2023-03-03T11:15:29+08:00 New Revision: 5828692f70bc5f92fad9ed21a1844044924680aa URL: https://github.com/llvm/llvm-project/commit/5828692f70bc5f92fad9ed21a1844044924680aa DIFF: https://github.com/llvm/llvm-project/commit/5828692f70bc5f92fad9ed21a1844044924680aa.diff LO

[PATCH] D145138: [clang-tidy] Implement FixIts for C arrays

ccotter updated this revision to Diff 502040. ccotter added a comment. - Add option doc, remove auto Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145138/new/ https://reviews.llvm.org/D145138 Files: clang-tools-extra/clang-tidy/modernize/AvoidCA

[PATCH] D145138: [clang-tidy] Implement FixIts for C arrays

ccotter marked 2 inline comments as done. ccotter added inline comments. Comment at: clang-tools-extra/clang-tidy/modernize/AvoidCArraysCheck.cpp:588 +: ClangTidyCheck(Name, Context), + IncludeInserter(Options.getLocalOrGlobal("IncludeStyle", +

[PATCH] D145138: [clang-tidy] Implement FixIts for C arrays

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/clang-tidy/modernize/AvoidCArraysCheck.cpp:588 +: ClangTidyCheck(Name, Context), + IncludeInserter(Options.getLocalOrGlobal("IncludeStyle", + utils::IncludeS

[PATCH] D144454: Add builtin for llvm set rounding

sepavloff added a comment. In D144454#4163688 , @rjmccall wrote: > I see. If we're going to take the target-independent values specified by > `FLT_ROUNDS`, then the original builtin name is more appropriate. Of course, > this has the disadvantage of n

[clang] 5783363 - [C++20] [Modules] Deprecate to load C++20 Modules eagerly

Author: Chuanqi Xu Date: 2023-03-03T14:25:33+08:00 New Revision: 57833636816a13ccda53714413c532dc81e3b5ff URL: https://github.com/llvm/llvm-project/commit/57833636816a13ccda53714413c532dc81e3b5ff DIFF: https://github.com/llvm/llvm-project/commit/57833636816a13ccda53714413c532dc81e3b5ff.diff LO

[PATCH] D144707: [C++20] [Modules] Deprecate to load C++20 Named Modules eagerly

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG57833636816a: [C++20] [Modules] Deprecate to load C++20 Modules eagerly (authored by ChuanqiXu). Repository: rG LLVM Github Monorepo CHANGES SINC

[PATCH] D144709: [clang-format] Improve QualifierAlignment

AlexanderHederstaf added a comment. Thanks for the comments and help on this review. There are a lot of special cases I had not considered before, and while it works on the clang/ and llvm/ subfolders as well as tests and the other repository I tested on, I suppose there could still be some hid

[PATCH] D143418: [libclang] Add API to override preamble storage path

vedgy added inline comments. Comment at: clang/include/clang-c/Index.h:329 + * CXIndexOptions Opts = { sizeof(CXIndexOptions), + * clang_getDefaultGlobalOptions() }; + * \endcode aaron.ballman wrote: > vedgy wrote: > > vedgy wrote: > > > a

<    1   2