[PATCH] D149666: [OpenMP][OMPIRBuilder] Migrate MapCombinedInfoTy from Clang to OpenMPIRBuilder

This revision was automatically updated to reflect the committed changes. TIFitis marked 2 inline comments as done. Closed by commit rG35309db7dcef: [OpenMP][OMPIRBuilder] Migrate MapCombinedInfoTy from Clang to OpenMPIRBuilder (authored by TIFitis). Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D149796: [clang][Sema][NFC] Move `EnterExpressionEvaluationContext` to its own file

davidstone updated this revision to Diff 519514. davidstone added a comment. Address review comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149796/new/ https://reviews.llvm.org/D149796 Files: clang/include/clang/Sema/EnterExpressionEvalua

[PATCH] D149862: Remove `Sema::getFloat128Identifier`

davidstone created this revision. Herald added a project: All. davidstone requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This function was added prior to clang supporting `__float128` to ensure clang could compile a libstdc++ header. This

[PATCH] D149800: [WIP][PGO] Add ability to mark cold functions as optsize/minsize/optnone

aeubanks added a comment. In D149800#4317275 , @mtrofin wrote: > high level question, why not have it as a pass that runs after profiles (of > whatever kind - instrumented or sample-based) are ingested. The pass would > attribute functions as described.

[PATCH] D149867: [M68k] Add Clang support for the new M68k_RTD CC

myhsu created this revision. myhsu added reviewers: 0x59616e, RKSimon, craig.topper. Herald added a subscriber: pengfei. Herald added a project: All. myhsu requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. We basically piggyback most of imple

[PATCH] D149796: [clang][Sema][NFC] Move `EnterExpressionEvaluationContext` to its own file

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM! Do you need me to commit this on your behalf? If so, what name and email address would you like me to use for patch attribution? Repository: rG LLVM Github Monorepo CHA

[clang] 053bf86 - MS inline asm: remove obsolete code adding AOK_SizeDirective (e.g. dword ptr)

Author: Fangrui Song Date: 2023-05-04T09:42:25-07:00 New Revision: 053bf8640aa85911823c2fc98538474c42d33b7a URL: https://github.com/llvm/llvm-project/commit/053bf8640aa85911823c2fc98538474c42d33b7a DIFF: https://github.com/llvm/llvm-project/commit/053bf8640aa85911823c2fc98538474c42d33b7a.diff

[PATCH] D149695: MS inline asm: remove obsolete code adding AOK_SizeDirective (e.g. dword ptr)

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG053bf8640aa8: MS inline asm: remove obsolete code adding AOK_SizeDirective (e.g. dword ptr) (authored by MaskRay). Repository: rG LLVM Github Mono

[PATCH] D149869: [clang][dataflow] Remove deprecated pass-through APIs for DataflowAnalysisContext.

bazuzi created this revision. bazuzi added reviewers: ymandel, gribozavr2, xazax.hun. Herald added subscribers: martong, rnkovacs. Herald added a reviewer: NoQ. Herald added a project: All. bazuzi requested review of this revision. Herald added a project: clang. These were recently deprecated in h

[PATCH] D148689: [clang][Interp] Handle PredefinedExprs

aaron.ballman added inline comments. Comment at: clang/test/AST/Interp/literals.cpp:849 +static_assert(strings_match(__func__, "foo")); +static_assert(strings_match(__PRETTY_FUNCTION__, "void PredefinedExprs::foo()")); + } tbaeder wrote: > tbaeder wrote

[PATCH] D149796: [clang][Sema][NFC] Move `EnterExpressionEvaluationContext` to its own file

davidstone added a comment. In D149796#4319183 , @aaron.ballman wrote: > LGTM! Do you need me to commit this on your behalf? If so, what name and > email address would you like me to use for patch attribution? Yes please. David Stone, davidfromonl...@g

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

Ris-Bali updated this revision to Diff 519544. Ris-Bali marked 5 inline comments as done. Ris-Bali added a comment. Requested Changes CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/docs/ReleaseNotes.rst clang/lib/CodeGen/CGBu

[PATCH] D149872: [OpenMP][OMPIRBuilder] Migrate emitOffloadingArrays and EmitNonContiguousDescriptor from Clang

TIFitis created this revision. TIFitis added reviewers: jdoerfert, jsjodin. Herald added subscribers: sunshaoce, guansong, hiraditya, yaxunl. Herald added a project: All. TIFitis requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits, jplehr, sstefan1. Herald added p

[PATCH] D149550: [clang][Interp] Fix compound assign operator evaluation order

aaron.ballman added inline comments. Comment at: clang/lib/AST/Interp/ByteCodeExprGen.cpp:683-685 + // C++17 onwards require that we evaluate the RHS first. + // Compute RHS and save it in a temporary variable so we can + // load it again later. tbaeder wrote:

[clang] 6d68805 - [clang][Sema][NFC] Move `EnterExpressionEvaluationContext` to its own file

Author: David Stone Date: 2023-05-04T13:06:53-04:00 New Revision: 6d6880554c2cd8d4d83db56c74eda24c22e6323a URL: https://github.com/llvm/llvm-project/commit/6d6880554c2cd8d4d83db56c74eda24c22e6323a DIFF: https://github.com/llvm/llvm-project/commit/6d6880554c2cd8d4d83db56c74eda24c22e6323a.diff L

[PATCH] D149796: [clang][Sema][NFC] Move `EnterExpressionEvaluationContext` to its own file

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6d6880554c2c: [clang][Sema][NFC] Move `EnterExpressionEvaluationContext` to its own file (authored by davidstone, committed by aaron.ballman). Repo

[PATCH] D146090: [Clang] Updating handling of defaulted comparison operators to reflect changes from P2448R2

shafik updated this revision to Diff 519555. shafik marked 4 inline comments as done. shafik added a comment. - Removed diagnostic group - fixed cxx_status indentation CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146090/new/ https://reviews.llvm.org/D146090 Files: clang/docs/Release

[PATCH] D149824: [clang][Interp] Don't call getSource() on functions without a body

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM Comment at: clang/lib/AST/Interp/Function.cpp:35 assert(PC <= getCodeEnd() && "PC Does not belong to this function"); + assert(hasBody()); unsigned

[PATCH] D149828: [clang][Interp] Evaluate Base when discarding a MemberExpr

aaron.ballman accepted this revision. aaron.ballman 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/D149828/new/ https://reviews.llvm.org/D149828 _

[PATCH] D149144: [clang][dataflow] Eliminate intermediate `ReferenceValue`s from `Environment::DeclToLoc`.

xazax.hun accepted this revision. xazax.hun added inline comments. Comment at: clang/lib/Analysis/FlowSensitive/TypeErasedDataflowAnalysis.cpp:329 +static void +builtinTransferScopeEnd(const CFGScopeEnd &Elt, +TypeErasedDataflowAnalysisState &InputState) {

[PATCH] D149834: [clang][Interp] Fix ignoring TypeTraitExprs

aaron.ballman added inline comments. Comment at: clang/test/AST/Interp/literals.cpp:875 1 ? 0 : 1; +sizeof(A); +alignof(A); Let's make sure we still reject this: ``` constexpr int oh_my() { int x = 0; sizeof(int[x++]); // This would usually be ev

[PATCH] D149650: Give NullabilityKind a printing operator<

sammccall added a comment. In D149650#4316138 , @aaron.ballman wrote: > In D149650#4315211 , @sammccall > wrote: > >> In D149650#4312389 , >> @aaron.ballman wrote: >> >

[PATCH] D149850: [Clang][Modules] Support `requires cplusplus20` in a modulemap

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

[PATCH] D103930: [clang][HeaderSearch] Fix implicit module when using header maps

bruno added reviewers: Bigcheese, ChuanqiXu, jansvoboda11. bruno added a comment. Adding code owners and more relevant folks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103930/new/ https://reviews.llvm.org/D103930 ___

[PATCH] D149650: Give NullabilityKind a printing operator<

aaron.ballman accepted this revision. aaron.ballman added a comment. In D149650#4319461 , @sammccall wrote: > In D149650#4316138 , @aaron.ballman > wrote: > >> In D149650#4315211

[PATCH] D103930: [clang][HeaderSearch] Fix implicit module when using header maps

bruno added a comment. Thanks Duncan. > Given that header maps are somewhat Apple-specific Some non-obvious background: It began Apple specific, but Meta uses them at scale as well, pretty important for us to get this right. > and unit test coverage is a bit lacking for these sorts of interact

[PATCH] D149837: [clang][Interp] Fix ignoring CompoundLiteralExprs

aaron.ballman added inline comments. Comment at: clang/test/AST/Interp/literals.cpp:878 +(int){1}; +(int[]){1,2,3}; Can you also add a test like: ``` constexpr int oh_my(int x) { (int){ x++ }; return x; } static_assert(oh_my(0) == 1); ``` and ``` co

[PATCH] D146090: [Clang] Updating handling of defaulted comparison operators to reflect changes from P2448R2

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGb4692f292630: [Clang] Updating handling of defaulted comparison operators to reflect changes… (authored by shafik). Herald added a project: clang. R

[clang] b4692f2 - [Clang] Updating handling of defaulted comparison operators to reflect changes from P2448R2

Author: Shafik Yaghmour Date: 2023-05-04T11:07:16-07:00 New Revision: b4692f29263006c7ea519c7b11c9082384f0af53 URL: https://github.com/llvm/llvm-project/commit/b4692f29263006c7ea519c7b11c9082384f0af53 DIFF: https://github.com/llvm/llvm-project/commit/b4692f29263006c7ea519c7b11c9082384f0af53.dif

[PATCH] D146090: [Clang] Updating handling of defaulted comparison operators to reflect changes from P2448R2

chaitanyav added a comment. @shafik @aaron.ballman I see some errors [1/2496] Building DiagnosticAnalysisKinds.inc... FAILED: tools/clang/include/clang/Basic/DiagnosticAnalysisKinds.inc /home/nvellanki/scratch/llvm-project/tools/clang/include/clang/Basic/DiagnosticAnalysisKinds.inc cd /h

[PATCH] D146090: [Clang] Updating handling of defaulted comparison operators to reflect changes from P2448R2

clementval added a comment. This commit is failing couple of buildbots. Can you revert or fix? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146090/new/ https://reviews.llvm.org/D146090 ___ cfe-commits m

[PATCH] D149831: [clang][Interp] Fix ignoring SubstNonTypeTemplateParmExpr

aaron.ballman accepted this revision. aaron.ballman 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/D149831/new/ https://reviews.llvm.org/D149831 _

[PATCH] D146090: [Clang] Updating handling of defaulted comparison operators to reflect changes from P2448R2

shafik added a comment. I am about to fix it Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146090/new/ https://reviews.llvm.org/D146090 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lis

[PATCH] D146090: [Clang] Updating handling of defaulted comparison operators to reflect changes from P2448R2

erichkeane added a comment. In D146090#4319561 , @clementval wrote: > This commit is failing couple of buildbots. Can you revert or fix? Shafik is validating a fix for it right now, it should be fixed momentarily. Repository: rG LLVM Github Monorepo

[clang] b323b40 - [clang] Fix build after https://reviews.llvm.org/D149553

Author: Michael Liao Date: 2023-05-04T14:23:46-04:00 New Revision: b323b407f76d22bfc08b1430f7952c03eb504288 URL: https://github.com/llvm/llvm-project/commit/b323b407f76d22bfc08b1430f7952c03eb504288 DIFF: https://github.com/llvm/llvm-project/commit/b323b407f76d22bfc08b1430f7952c03eb504288.diff

[PATCH] D146764: [clang] Make predefined expressions string literals under -fms-extensions

aaron.ballman added a comment. Precommit CI found some related failures that need to be addressed, but the functional changes are all looking good to me. Be sure to also add a release note for the fix as well. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.l

[clang] 058f04e - [clang] Fix another case where CPlusPlus2b is still used.

Author: Michael Liao Date: 2023-05-04T14:27:33-04:00 New Revision: 058f04ea7dcbafbeed271fa75ee65e41409b4479 URL: https://github.com/llvm/llvm-project/commit/058f04ea7dcbafbeed271fa75ee65e41409b4479 DIFF: https://github.com/llvm/llvm-project/commit/058f04ea7dcbafbeed271fa75ee65e41409b4479.diff

[clang] 7887af0 - Fix build bots due to missing a commit thay change 2b to 23

Author: Shafik Yaghmour Date: 2023-05-04T11:33:58-07:00 New Revision: 7887af027ee5eff27bbc953074726ab8d9d9f592 URL: https://github.com/llvm/llvm-project/commit/7887af027ee5eff27bbc953074726ab8d9d9f592 DIFF: https://github.com/llvm/llvm-project/commit/7887af027ee5eff27bbc953074726ab8d9d9f592.dif

[PATCH] D149677: [clang][TypePrinter] Add option to skip over elaborated types

aaron.ballman added a comment. Will this new printing policy be used in tree? Comment at: clang/include/clang/AST/PrettyPrinter.h:143-145 + /// Ignore qualifiers as specified by elaborated type sugar, instead letting + /// the underlying type handle printing the qualifiers. +

[PATCH] D149884: [clang][deps] Teach dep directive scanner about _Pragma

benlangmuir created this revision. benlangmuir added reviewers: akyrtzi, Bigcheese, jansvoboda11. Herald added a project: All. benlangmuir requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. While we cannot handle `_Pragma` used inside macros,

[PATCH] D146090: [Clang] Updating handling of defaulted comparison operators to reflect changes from P2448R2

shafik added a comment. Landed fix 7887af027ee5eff27bbc953074726ab8d9d9f592 There was also 058f04ea7dcbafbeed271fa75ee65e41409b4479 and b323b407f76d22bfc0

[clang] 14805dc - [clang][ExtractAPI] Add semicolon to function declaration fragments

Author: NagaChaitanya Vellanki Date: 2023-05-04T11:46:43-07:00 New Revision: 14805dcb0d8abfd5cd015656313f3f98a22e0a1b URL: https://github.com/llvm/llvm-project/commit/14805dcb0d8abfd5cd015656313f3f98a22e0a1b DIFF: https://github.com/llvm/llvm-project/commit/14805dcb0d8abfd5cd015656313f3f98a22e0

[PATCH] D149737: [clang][ExtractAPI] Add semicolon to function declaration fragments

This revision was automatically updated to reflect the committed changes. Closed by commit rG14805dcb0d8a: [clang][ExtractAPI] Add semicolon to function declaration fragments (authored by chaitanyav). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149

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

jrtc27 added inline comments. Comment at: llvm/test/MC/RISCV/rv32zcmt-invalid.s:9 + +# CHECK-ERROR: error: immediate must be an integer in the range [0, 255] +cm.jalt 256 This is wrong; the immediate must be in the range [32, 255]. This needs to be enforced in t

[PATCH] D147823: [clang-repl] Reduce dynamic-library.cpp test to only load shared library

zhuhan0 abandoned this revision. zhuhan0 added a comment. Superseded by https://reviews.llvm.org/D148992. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147823/new/ https://reviews.llvm.org/D147823 ___ cf

[PATCH] D146090: [Clang] Updating handling of defaulted comparison operators to reflect changes from P2448R2

Mordante added a comment. In D146090#4319685 , @shafik wrote: > Landed fix 7887af027ee5eff27bbc953074726ab8d9d9f592 > > > There was also 058f04ea7dcbafbeed271fa75ee65e41409b4479 >

[PATCH] D148381: [WIP][Clang] Add element_count attribute

void updated this revision to Diff 519605. void marked an inline comment as done. void added a comment. Use "Expected" for the SourceRange imports. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148381/new/ https://reviews.llvm.org/D148381 Files:

[PATCH] D141907: [CMake] Ensure `CLANG_RESOURCE_DIR` is respected

tstellar added inline comments. Herald added subscribers: ekilmer, jplehr. Comment at: cmake/Modules/GetClangResourceDir.cmake:13 + if(DEFINED CLANG_RESOURCE_DIR AND NOT CLANG_RESOURCE_DIR STREQUAL "") +set(ret_dir bin/${CLANG_RESOURCE_DIR}) + else() Why is

[PATCH] D146764: [clang] Make predefined expressions string literals under -fms-extensions

aeubanks updated this revision to Diff 519613. aeubanks added a comment. only diagnose if we're initializing an array Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146764/new/ https://reviews.llvm.org/D146764 Files: clang/include/clang/AST/Expr.

[PATCH] D149550: [clang][Interp] Fix compound assign operator evaluation order

shafik added a subscriber: rsmith. shafik added inline comments. Comment at: clang/lib/AST/Interp/ByteCodeExprGen.cpp:683-685 + // C++17 onwards require that we evaluate the RHS first. + // Compute RHS and save it in a temporary variable so we can + // load it again later. ---

[PATCH] D149809: [Clang][Docs] Fix man page build

aidengrossman updated this revision to Diff 519622. aidengrossman marked 6 inline comments as done. aidengrossman added a comment. Adjust based on reviewer suggestions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149809/new/ https://reviews.llvm.

[PATCH] D149809: [Clang][Docs] Fix man page build

aidengrossman added a comment. Thank you very much for your patience and writing everything out. That makes a lot more sense. The resulting code definitely has a lot more desirable properties. I've updated the diff in accordance with your suggestions. Repository: rG LLVM Github Monorepo CHA

[PATCH] D144999: [RFC][MC][MachO]Only emits compact-unwind format for "canonical" personality symbols. For the rest, use DWARFs.

jyknight added a comment. I wonder if we actually need to define a clang frontend flag for this; I suspect nobody will ever want to specify it, since the only non-canonical personality clang will ever generate that changes behavior is the pure-C destructor-only personality, `__gnu_personality_v

[PATCH] D149516: [Sema] `setInvalidDecl` for error deduction declaration

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

[PATCH] D149612: [Sema] avoid merge error type

erichkeane added a comment. In D149612#4314536 , @shafik wrote: > In D149612#4314209 , @HerrCai0907 > wrote: > >> in SemaType.cpp#BuildArrayType, It will generate `DependentSizedArrayType` >> which cannot be mer

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

Ris-Bali updated this revision to Diff 519628. Ris-Bali added a comment. Rebase CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 Files: clang/docs/ReleaseNotes.rst clang/lib/CodeGen/CGBuiltin.cpp clang/lib/Sema/SemaChecking.cpp clang/te

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

owenpan added a subscriber: rymiel. owenpan added a comment. Herald added a reviewer: rymiel. I'll revert this patch due to the regression https://github.com/llvm/llvm-project/issues/61498 unless any of you (and @rymiel ) objects. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTIO

[PATCH] D149899: [clang-tidy] Support SystemHeaders in .clang-tidy

carlosgalvezp created this revision. Herald added subscribers: PiotrZSL, xazax.hun. Herald added a reviewer: njames93. Herald added a project: All. carlosgalvezp requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. A previous patch u

[PATCH] D149899: [clang-tidy] Support SystemHeaders in .clang-tidy

carlosgalvezp updated this revision to Diff 519633. carlosgalvezp edited the summary of this revision. carlosgalvezp added a comment. Update commit message with Github issue. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149899/new/ https://reviews

[PATCH] D149144: [clang][dataflow] Eliminate intermediate `ReferenceValue`s from `Environment::DeclToLoc`.

mboehme added inline comments. Comment at: clang/lib/Analysis/FlowSensitive/TypeErasedDataflowAnalysis.cpp:329 +static void +builtinTransferScopeEnd(const CFGScopeEnd &Elt, +TypeErasedDataflowAnalysisState &InputState) { xazax.hun wrote: >

[PATCH] D149182: Remove -Wpacked false positive for non-pod types where the layout isn't directly changed

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM aside from a nit with the release notes. Thank you for the fix! Comment at: clang/docs/ReleaseNotes.rst:109-113 +Warnings + +- Address a false posi

[PATCH] D147875: [clang][Diagnostics] Show line numbers when printing code snippets

aaron.ballman added a subscriber: clang-vendors. aaron.ballman added a comment. This should definitely have explicit documentation added to https://github.com/llvm/llvm-project/blob/main/clang/docs/UsersManual.rst?plain=1#L156 and a release note. Adding clang-vendors because of the potential th

[PATCH] D132819: [RISCV] Add MC support of RISCV zcmp Extension

craig.topper requested changes to this revision. craig.topper added a comment. This revision now requires changes to proceed. The instructions need a DecoderNamespace to separate them from c.fsdsp. See D149891 for how I've done it for Zcmt. Repository: rG LL

[PATCH] D149677: [clang][TypePrinter] Add option to skip over elaborated types

li.zhe.hua updated this revision to Diff 519643. li.zhe.hua marked an inline comment as done. li.zhe.hua added a comment. Accept suggested edits. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149677/new/ https://reviews.llvm.org/D149677 Files: c

[PATCH] D149677: [clang][TypePrinter] Add option to skip over elaborated types

li.zhe.hua added a comment. In D149677#4319646 , @aaron.ballman wrote: > Will this new printing policy be used in tree? Not initially, no. (I'm understanding "in tree" as "in the LLVM project".) Some additional context: I'm working on a refactoring too

[clang] bfbe137 - [clang][dataflow] Eliminate intermediate `ReferenceValue`s from `Environment::DeclToLoc`.

Author: Martin Braenne Date: 2023-05-04T20:57:30Z New Revision: bfbe137888151dfd506df6b3319d08c4de0e00f5 URL: https://github.com/llvm/llvm-project/commit/bfbe137888151dfd506df6b3319d08c4de0e00f5 DIFF: https://github.com/llvm/llvm-project/commit/bfbe137888151dfd506df6b3319d08c4de0e00f5.diff LOG

[PATCH] D149144: [clang][dataflow] Eliminate intermediate `ReferenceValue`s from `Environment::DeclToLoc`.

This revision was automatically updated to reflect the committed changes. Closed by commit rGbfbe13788815: [clang][dataflow] Eliminate intermediate `ReferenceValue`s from `Environment… (authored by mboehme). Changed prior to commit: https://reviews.llvm.org/D149144?vs=519373&id=519644#toc Repo

[PATCH] D149904: Generic selection expressions that accept a type operand

aaron.ballman created this revision. aaron.ballman added reviewers: clang-language-wg, efriedma, jyknight, hubert.reinterpretcast, rsmith. Herald added a subscriber: martong. Herald added a reviewer: shafik. Herald added a reviewer: NoQ. Herald added a project: All. aaron.ballman requested review

[PATCH] D149612: [Sema] avoid merge error type

HerrCai0907 added a comment. Dependent type for CPP is a common case due to template. But for `mergeTypes` in C, dependent type cannot be handled correctly. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149612/new/ https://reviews.llvm.org/D14961

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

mitchell-stellar added a comment. After reviewing the GitHub issue, it looks like the IndentAndNestingLevel is sticky over the entire scope for `#else` blocks, so the regression only occurs within `#else` blocks in the same scope. The bug I fixed affected alignment in all `#else` blocks, regard

[PATCH] D147875: [clang][Diagnostics] Show line numbers when printing code snippets

kwk added a comment. @tbaeder I have a small suggestion. Comment at: clang/lib/Frontend/TextDiagnostic.cpp:1121-1138 +static unsigned getNumDisplayWidth(unsigned N) { + if (N < 10) +return 1; + if (N < 100) +return 2; + if (N < 1'000) +return 3;

[PATCH] D149144: [clang][dataflow] Eliminate intermediate `ReferenceValue`s from `Environment::DeclToLoc`.

gribozavr2 added a comment. LGTM still. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149144/new/ https://reviews.llvm.org/D149144 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.ll

[PATCH] D149906: [Clang] Update release notes

alexander-shaposhnikov created this revision. alexander-shaposhnikov added reviewers: rsmith, erichkeane, ilya-biryukov. alexander-shaposhnikov created this object with visibility "All Users". Herald added a project: All. alexander-shaposhnikov requested review of this revision. Herald added a proj

[PATCH] D149650: Give NullabilityKind a printing operator<

sammccall added a comment. Thanks Aaron! The check is not going to be ready for upstreaming really soon - plenty of thorny issues to work out first, and checking in a half-working version would do more harm than good. But that's definitely the goal once we have some deployment experience. Rep

[clang] 0a53220 - Give NullabilityKind a printing operator<

Author: Sam McCall Date: 2023-05-04T23:24:51+02:00 New Revision: 0a532207b8696d81e46017f444bd2257347f129b URL: https://github.com/llvm/llvm-project/commit/0a532207b8696d81e46017f444bd2257347f129b DIFF: https://github.com/llvm/llvm-project/commit/0a532207b8696d81e46017f444bd2257347f129b.diff LO

[PATCH] D149650: Give NullabilityKind a printing operator<

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0a532207b869: Give NullabilityKind a printing operator<< (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D149906: [Clang] Update release notes

alexander-shaposhnikov updated this revision to Diff 519660. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149906/new/ https://reviews.llvm.org/D149906 Files: clang/docs/ReleaseNotes.rst Index: clang/docs/ReleaseNotes.rst ===

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

mitchell-stellar added a comment. I retract my objection. I forgot that from clang-format's perspective, `#else` blocks in separate functions have the same indent and nesting level, and thus the same "scope", so the alignment issues may persist there. You will see misalignment in `#else` blocks

[PATCH] D149514: Check if First argument in _builtin_assume_aligned_ is of pointer type

Ris-Bali added a comment. I have made the requested changes. Please do let me know if other changes are required. Also I would like to thank everyone for their help and guidance. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149514/new/ https://reviews.llvm.org/D149514 ___

[PATCH] D147875: [clang][Diagnostics] Show line numbers when printing code snippets

jrtc27 added inline comments. Comment at: clang/lib/Frontend/TextDiagnostic.cpp:1121-1138 +static unsigned getNumDisplayWidth(unsigned N) { + if (N < 10) +return 1; + if (N < 100) +return 2; + if (N < 1'000) +return 3; kwk wrote: > This function sc

[PATCH] D146178: [Clang][Sema] Fix comparison of constraint expressions

davidtgoldblatt added a comment. This version of the commit also introduces some breakages; as before I'm not sure if it's the code or the diff that's incorrect. Repro: enum class Enum { E1 }; template inline constexpr bool some_concept = true; template struct S { template

[PATCH] D149612: [Sema] avoid merge error type

HerrCai0907 updated this revision to Diff 519671. HerrCai0907 added a comment. getConstantArrayType instal QualType() Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149612/new/ https://reviews.llvm.org/D149612 Files: clang/docs/ReleaseNotes.rst

[PATCH] D149612: [Sema] avoid merge error type

HerrCai0907 updated this revision to Diff 519675. HerrCai0907 added a comment. use nullpt as SizeExpr Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149612/new/ https://reviews.llvm.org/D149612 Files: clang/docs/ReleaseNotes.rst clang/lib/Sema/

[PATCH] D149912: [clangd] downgrade missing-includes diagnostic to Information level

sammccall created this revision. sammccall added a reviewer: kadircet. Herald added a subscriber: arphaman. Herald added a project: All. sammccall requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra. In practic

[PATCH] D149144: [clang][dataflow] Eliminate intermediate `ReferenceValue`s from `Environment::DeclToLoc`.

xazax.hun added inline comments. Comment at: clang/lib/Analysis/FlowSensitive/TypeErasedDataflowAnalysis.cpp:329 +static void +builtinTransferScopeEnd(const CFGScopeEnd &Elt, +TypeErasedDataflowAnalysisState &InputState) { mboehme wrote: >

[PATCH] D149912: [clangd] downgrade missing-includes diagnostic to Information level

sammccall added a comment. This change is just one option to be evaluated for making these diagnostics more user-friendly. Mostly sending the patch now to attach some screenshots... Effect in VSCode (blue underline, visible in problems view, filterable) F27332814: image.png

[PATCH] D148385: [RISCV] Implement KCFI operand bundle lowering

samitolvanen planned changes to this revision. samitolvanen added a comment. Uploaded D149915 to remove the arch-specific KCFI passes. I'll update this patch once we clean that up. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revi

[PATCH] D149917: [lld][WebAssembly] Add --preserve-features flag

sbc100 created this revision. Herald added subscribers: pmatos, asb, wingo, ecnelises, sunfish, jgravelle-google, dschuff. Herald added a project: All. sbc100 requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, aheejin. Herald added a project: clang. This flag cause

[PATCH] D149917: [lld][WebAssembly] Add --preserve-features flag

sbc100 added a comment. Still needs a test.. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149917/new/ https://reviews.llvm.org/D149917 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lis

[PATCH] D149612: [Sema] avoid merge error type

erichkeane added inline comments. Comment at: clang/lib/Sema/SemaType.cpp:2582 } else if (ArraySize->isTypeDependent() || ArraySize->isValueDependent()) { -T = Context.getDependentSizedArrayType(T, ArraySize, ASM, Quals, Brackets); +if (getLangOpts().CPlusPlus) { +

[PATCH] D149904: Generic selection expressions that accept a type operand

erichkeane added inline comments. Comment at: clang/include/clang/AST/Expr.h:5686 /// if and only if the generic selection expression is result-dependent. - unsigned NumAssocs, ResultIndex; + unsigned NumAssocs, ResultIndex, IsExprPredicate; enum : unsigned {

[PATCH] D141907: [CMake] Ensure `CLANG_RESOURCE_DIR` is respected

paperchalice added inline comments. Comment at: cmake/Modules/GetClangResourceDir.cmake:13 + if(DEFINED CLANG_RESOURCE_DIR AND NOT CLANG_RESOURCE_DIR STREQUAL "") +set(ret_dir bin/${CLANG_RESOURCE_DIR}) + else() tstellar wrote: > Why is the bin prefix here?

[clang] 3b9ed6e - Revert "[Clang][Sema] Fix comparison of constraint expressions"

Author: Alexander Shaposhnikov Date: 2023-05-05T00:02:26Z New Revision: 3b9ed6e5323176550925f3b0a2c50ced1b61438d URL: https://github.com/llvm/llvm-project/commit/3b9ed6e5323176550925f3b0a2c50ced1b61438d DIFF: https://github.com/llvm/llvm-project/commit/3b9ed6e5323176550925f3b0a2c50ced1b61438d.d

[PATCH] D146178: [Clang][Sema] Fix comparison of constraint expressions

alexander-shaposhnikov added a comment. Reverted in https://github.com/llvm/llvm-project/commit/3b9ed6e5323176550925f3b0a2c50ced1b61438d, it'll take time to investigate this case. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146178/new/ https://r

[PATCH] D141907: [CMake] Ensure `CLANG_RESOURCE_DIR` is respected

tstellar added inline comments. Comment at: cmake/Modules/GetClangResourceDir.cmake:13 + if(DEFINED CLANG_RESOURCE_DIR AND NOT CLANG_RESOURCE_DIR STREQUAL "") +set(ret_dir bin/${CLANG_RESOURCE_DIR}) + else() paperchalice wrote: > tstellar wrote: > > Why is

[PATCH] D141907: [CMake] Ensure `CLANG_RESOURCE_DIR` is respected

tstellar added inline comments. Comment at: cmake/Modules/GetClangResourceDir.cmake:13 + if(DEFINED CLANG_RESOURCE_DIR AND NOT CLANG_RESOURCE_DIR STREQUAL "") +set(ret_dir bin/${CLANG_RESOURCE_DIR}) + else() tstellar wrote: > paperchalice wrote: > > tstella

[PATCH] D146090: [Clang] Updating handling of defaulted comparison operators to reflect changes from P2448R2

shafik added a comment. In D146090#4319737 , @Mordante wrote: > In D146090#4319685 , @shafik wrote: > >> Landed fix 7887af027ee5eff27bbc953074726ab8d9d9f592 >>

cfe-commits@lists.llvm.org

shafik added a comment. In D148924#4318381 , @massberg wrote: > I have checked the paper P2002R1 and as far as I can tell it is fully > implemented when this patch has landed. So I read the paper but since the paper does not have examples for each chang

[PATCH] D143624: Inlining: Run the legacy AlwaysInliner before the regular inliner.

aemerson added a comment. I have another attempt at fixing this in D149918 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143624/new/ https://reviews.llvm.org/D143624 _

[PATCH] D146178: [Clang][Sema] Fix comparison of constraint expressions

alexander-shaposhnikov updated this revision to Diff 519708. alexander-shaposhnikov added a comment. Add more tests, Fix HandlePartialClassTemplateSpec. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146178/new/ https://reviews.llvm.org/D146178 Fil

  1   2   3   >