[PATCH] D115503: [DebugInfo][Clang] record the access flag for class/struct/union types.

dblaikie added a comment. In D115503#3199489 , @shchenz wrote: > In D115503#3199459 , @Esme wrote: > >> In D115503#3195171 , @dblaikie >> wrote: >> >>> Ah, cool - could

[PATCH] D115982: [clang][AVR] Implement '__flashN' for variables on different flash banks

benshi001 created this revision. benshi001 added reviewers: aykevl, dylanmckay. Herald added a subscriber: Jim. benshi001 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo

[clang] 4e310d8 - [clang] Strip redundant lambda (NFC)

Author: Kazu Hirata Date: 2021-12-17T20:55:10-08:00 New Revision: 4e310d89f22303873c952a9ca5ca91c921407336 URL: https://github.com/llvm/llvm-project/commit/4e310d89f22303873c952a9ca5ca91c921407336 DIFF: https://github.com/llvm/llvm-project/commit/4e310d89f22303873c952a9ca5ca91c921407336.diff

[PATCH] D115910: [doc] Fix regex in ClangFormatStyleOptions for IncludeCategories

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG212e6c99776f: [doc] Fix regex in ClangFormatStyleOptions for IncludeCategories (authored by jhuels, committed by owenpan). Herald added a

[clang] 212e6c9 - [doc] Fix regex in ClangFormatStyleOptions for IncludeCategories

Author: Joshua Huels Date: 2021-12-17T18:46:12-08:00 New Revision: 212e6c99776f393e0af368ca2086eb8bae2ee1ed URL: https://github.com/llvm/llvm-project/commit/212e6c99776f393e0af368ca2086eb8bae2ee1ed DIFF: https://github.com/llvm/llvm-project/commit/212e6c99776f393e0af368ca2086eb8bae2ee1ed.diff

[clang] fee5771 - Use DenseMap::lookup (NFC)

Author: Kazu Hirata Date: 2021-12-17T18:19:25-08:00 New Revision: fee57711fe4a5578adc65bf9ebfe664905ff5309 URL: https://github.com/llvm/llvm-project/commit/fee57711fe4a5578adc65bf9ebfe664905ff5309 DIFF: https://github.com/llvm/llvm-project/commit/fee57711fe4a5578adc65bf9ebfe664905ff5309.diff

[PATCH] D115976: [clang] Fix a crash case when reporting an uninitialized field.

hliao updated this revision to Diff 395248. hliao added a comment. Revise the formatting. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115976/new/ https://reviews.llvm.org/D115976 Files: clang/include/clang/AST/Expr.h

LLVM buildmaster will be restarted tonight

Hello everyone, LLVM buildmaster will be restarted after 6 PM PST today. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D115804: [CodeGen] use saturating FP casts when compiling with "no-strict-float-cast-overflow"

craig.topper added a comment. What's the plan for constrained intrinsics versions of these intrinsics? The IRBuilder calls for CreateFPToSI and CreateFPToUI are strict FP aware, but this new code isn't. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 713ee23 - [clang] Use llvm::reverse (NFC)

Author: Kazu Hirata Date: 2021-12-17T16:51:42-08:00 New Revision: 713ee230f884651afd17ab7b910e85a992ce406b URL: https://github.com/llvm/llvm-project/commit/713ee230f884651afd17ab7b910e85a992ce406b DIFF: https://github.com/llvm/llvm-project/commit/713ee230f884651afd17ab7b910e85a992ce406b.diff

[PATCH] D115979: [InstrProf] Don't profile merge by default in lightweight mode

ellis created this revision. Herald added a subscriber: ormris. ellis added reviewers: kyulee, MaskRay, alanphipps. ellis published this revision for review. Herald added projects: clang, Sanitizers. Herald added subscribers: Sanitizers, cfe-commits. Profile merging is not supported when using

[PATCH] D115976: [clang] Fix a crash case when reporting an uninitialized field.

hliao created this revision. hliao added a reviewer: rsmith. hliao requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. - As the initializer list's getSyntacticForm() may return NULL, need to consider the current semantic initalizer list

[PATCH] D115972: [clang-format] Fix AlignConsecutiveAssignments breaking lambda formatting.

curdeius planned changes to this revision. curdeius added a comment. Actually, I've just run it on a larger codebase and found that this patch misbehaves. Need to rework and add faulty test cases. Sorry for the noise. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D115972: [clang-format] Fix AlignConsecutiveAssignments breaking lambda formatting.

MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115972/new/ https://reviews.llvm.org/D115972

[PATCH] D115972: [clang-format] Fix AlignConsecutiveAssignments breaking lambda formatting.

curdeius created this revision. curdeius added reviewers: MyDeveloperDay, HazardyKnusperkeks, owenpan. curdeius requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fixes https://github.com/llvm/llvm-project/issues/52772. This patch fixes the

[PATCH] D115968: [clang-format] Refactor common handling of attributes. NFC.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG9cf4b7266bbf: [clang-format] Refactor common handling of attributes. NFC. (authored by curdeius). Repository: rG LLVM Github Monorepo CHANGES

[clang] 9cf4b72 - [clang-format] Refactor common handling of attributes. NFC.

Author: Marek Kurdej Date: 2021-12-17T23:02:45+01:00 New Revision: 9cf4b7266bbf238e4a5bd85c8d0011aac2f9181b URL: https://github.com/llvm/llvm-project/commit/9cf4b7266bbf238e4a5bd85c8d0011aac2f9181b DIFF: https://github.com/llvm/llvm-project/commit/9cf4b7266bbf238e4a5bd85c8d0011aac2f9181b.diff

[PATCH] D115968: [clang-format] Refactor common handling of attributes. NFC.

MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115968/new/ https://reviews.llvm.org/D115968

[PATCH] D115103: Leak Sanitizer port to Windows

clemenswasser updated this revision to Diff 395219. clemenswasser added a comment. Simplify Architecture checks CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115103/new/ https://reviews.llvm.org/D115103 Files: clang/lib/Driver/ToolChains/MSVC.cpp compiler-rt/cmake/config-ix.cmake

[PATCH] D115967: [clang-format][NFC] Handle wrapping after => in mustBreakBefore()

MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land. nice. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115967/new/ https://reviews.llvm.org/D115967

[PATCH] D114622: [clang-tidy][analyzer] Fix false-positive in IdenticalExprChecker and misc-redundant-expression

NoQ added a comment. P.S. `bugprone-branch-clone` seems to have attempted to use `CloneDetector` in the past but now it's no more than a dead #include. I wonder what happened there. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114622/new/ https://reviews.llvm.org/D114622

[PATCH] D115967: [clang-format][NFC] Handle wrapping after => in mustBreakBefore()

owenpan added inline comments. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1641 case tok::equal: - // Fat arrows (=>) have tok::TokenKind tok::equal but TokenType - // TT_FatArrow. They always start an expression or a child block if - // followed by

[PATCH] D115968: [clang-format] Refactor common handling of attributes. NFC.

curdeius created this revision. curdeius added a reviewer: MyDeveloperDay. curdeius requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D115968 Files:

[PATCH] D115932: [Analyzer] Create and handle SymbolCast for pointer to integral conversion

NoQ added a comment. Oh nice, it's great to see these things moving. I have a couple observations for your specific approach: (1) Sounds like you're introducing a symbol `(long p)` that represents the exact same value as (and is used interchangeably with) `{$p} (as long)`. Ambiguity is

[PATCH] D115561: [Clang][OpenMP] Add the support for atomic compare in parser

tianshilei1992 updated this revision to Diff 395208. tianshilei1992 added a comment. remove unexpected change Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115561/new/ https://reviews.llvm.org/D115561 Files:

[PATCH] D115561: [Clang][OpenMP] Add the support for atomic compare in parser

tianshilei1992 updated this revision to Diff 395207. tianshilei1992 added a comment. Herald added a project: Flang. update one test case and flang fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115561/new/ https://reviews.llvm.org/D115561

[PATCH] D114622: [clang-tidy][analyzer] Fix false-positive in IdenticalExprChecker and misc-redundant-expression

NoQ added a comment. These checks are almost 2000 lines of code each and it looks like all they do is figure out if two statements are the same and we have a very flexible reusable solution for this sort of stuff - `CloneDetector`, but none of them use it. Your patch demonstrates that they all

[PATCH] D115967: [clang-format][NFC] Handle wrapping after => in mustBreakBefore()

HazardyKnusperkeks added inline comments. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1641 case tok::equal: - // Fat arrows (=>) have tok::TokenKind tok::equal but TokenType - // TT_FatArrow. They always start an expression or a child block if - //

[PATCH] D115967: [clang-format][NFC] Handle wrapping after => in mustBreakBefore()

owenpan created this revision. owenpan added reviewers: MyDeveloperDay, curdeius, HazardyKnusperkeks, peterstys. owenpan added a project: clang-format. owenpan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Move the handling of brace

[PATCH] D114995: clang-tidy: improve the 'modernize-use-default-member-init'

oleg.smolsky added a comment. In D114995#3198033 , @aaron.ballman wrote: > In D114995#3183240 , > @malcolm.parsons wrote: > >> In D114995#3180475 , >> @aaron.ballman

[PATCH] D114995: clang-tidy: improve the 'modernize-use-default-member-init'

oleg.smolsky updated this revision to Diff 395190. oleg.smolsky added a comment. Added a "two constructors" test case along with the support for that. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114995/new/ https://reviews.llvm.org/D114995

[PATCH] D115888: [Attributor][Fix] Add alignment return attribute to HeapToStack

jdoerfert accepted this revision. jdoerfert added a comment. LG, don't forget to update the commit message. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115888/new/ https://reviews.llvm.org/D115888 ___

[PATCH] D115888: [Attributor][Fix] Add alignment return attribute to HeapToStack

jhuber6 updated this revision to Diff 395187. jhuber6 added a comment. Removing else if, we should be able to check for all allocations. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115888/new/ https://reviews.llvm.org/D115888 Files:

[PATCH] D115959: [clangd] Fix undefined behavior when generating error message at rename with an invalid name

ArcsinX created this revision. ArcsinX added reviewers: sammccall, kadircet, hokein. Herald added subscribers: usaxena95, arphaman. ArcsinX requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra. `Message()`

[PATCH] D115888: [Attributor][Fix] Add alignment return attribute to HeapToStack

jhuber6 updated this revision to Diff 395182. jhuber6 added a comment. Removing OpenMP code, only adding support for return alignments. Fixing OpenMP will occur in a following patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115888/new/

[PATCH] D114483: [SYCL] Add support for sycl_special_class attribute

zahiraam added inline comments. Comment at: clang/lib/Sema/SemaDecl.cpp:9118 + NewFD->getKind() == Decl::Kind::CXXMethod && + NewFD->getName() == "__init" && D.isFunctionDefinition()) { +if (auto *def = Parent->getDefinition())

[PATCH] D114483: [SYCL] Add support for sycl_special_class attribute

zahiraam updated this revision to Diff 395176. zahiraam marked 5 inline comments as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114483/new/ https://reviews.llvm.org/D114483 Files: clang/include/clang/AST/CXXRecordDeclDefinitionBits.def clang/include/clang/AST/DeclCXX.h

[PATCH] D104830: AST: Create __va_list in the std namespace even in C.

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

[PATCH] D115879: [clang-format] extern with new line brace without indentation

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

[PATCH] D115888: [Attributor][Fix] Add alignment return attribute to HeapToStack

jhuber6 added a comment. I will split this into two revisions, one handling the return alignment attribute in the Attributor, and one adding alignment information to the `__kmpc_alloc_shared` OpenMP runtime call, turning it into an aligned allocation. Comment at:

[PATCH] D115738: [clang-format] Fix formatting of the code that follows C# Lambda Expressions

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG163c13fed9f6: [clang-format] Fix formatting of the code that follows C# Lambda Expressions (authored by peterstys, committed by owenpan).

[clang] 163c13f - [clang-format] Fix formatting of the code that follows C# Lambda Expressions

Author: Peter Stys Date: 2021-12-17T10:42:15-08:00 New Revision: 163c13fed9f68c31a14b3d2409b994909f0600bb URL: https://github.com/llvm/llvm-project/commit/163c13fed9f68c31a14b3d2409b994909f0600bb DIFF: https://github.com/llvm/llvm-project/commit/163c13fed9f68c31a14b3d2409b994909f0600bb.diff

[PATCH] D115879: [clang-format] extern with new line brace without indentation

MyDeveloperDay updated this revision to Diff 395166. MyDeveloperDay marked 4 inline comments as done. MyDeveloperDay added a comment. Address nits and review comments (remove unnecessary lines) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115879/new/ https://reviews.llvm.org/D115879

[PATCH] D115903: [clang-format] Extra spaces surrounding arrow in templated member call in variable decl

MyDeveloperDay updated this revision to Diff 395165. MyDeveloperDay added a comment. Add additional test to show lambda's should be ok CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115903/new/ https://reviews.llvm.org/D115903 Files: clang/lib/Format/TokenAnnotator.cpp

[PATCH] D115938: [clang-format] Formatter does not handle c++11 string literal prefix with stringize #

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG62ead3654795: [clang-format] Formatter does not handle c++11 string literal prefix with… (authored by MyDeveloperDay). Repository: rG LLVM Github

[clang] 62ead36 - [clang-format] Formatter does not handle c++11 string literal prefix with stringize #

Author: mydeveloperday Date: 2021-12-17T18:29:32Z New Revision: 62ead3654795bfb09bf7296ef5db8deea0f758cb URL: https://github.com/llvm/llvm-project/commit/62ead3654795bfb09bf7296ef5db8deea0f758cb DIFF: https://github.com/llvm/llvm-project/commit/62ead3654795bfb09bf7296ef5db8deea0f758cb.diff

[PATCH] D115902: [OPENMP]Look through member function call base during implicit DSA analysis.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG667485413fd3: [OPENMP]Look through member function call base during implicit DSA analysis. (authored by ABataev). Repository: rG LLVM Github

[clang] 6674854 - [OPENMP]Look through member function call base during implicit DSA analysis.

Author: Alexey Bataev Date: 2021-12-17T10:18:51-08:00 New Revision: 667485413fd32cf470d7cd6260b50e486fedaf5e URL: https://github.com/llvm/llvm-project/commit/667485413fd32cf470d7cd6260b50e486fedaf5e DIFF: https://github.com/llvm/llvm-project/commit/667485413fd32cf470d7cd6260b50e486fedaf5e.diff

[PATCH] D115709: [RISCV] Remove Zvamo Extention

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

[PATCH] D115901: [OpenMP][NFC] update status for 5.1 'fail' atomic extension

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd976fb020428: [OpenMP][NFC] update status for 5.1 fail atomic extension (authored by dreachem, committed by Chi-Chun, Chen

[clang] d976fb0 - [OpenMP][NFC] update status for 5.1 'fail' atomic extension

Author: Deepak Eachempati Date: 2021-12-17T11:46:37-06:00 New Revision: d976fb0204283b5aabf40d805e59ce6d1502d1d9 URL: https://github.com/llvm/llvm-project/commit/d976fb0204283b5aabf40d805e59ce6d1502d1d9 DIFF:

[PATCH] D115888: [Attributor][Fix] Add alignment return attribute to HeapToStack

jdoerfert added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp:1411 +CGM.getLLVMContext(), llvm::Attribute::Alignment, +CGM.getContext().getTypeAlignInChars(VarTy).getQuantity())); This doesn't work. If the type alignment

[PATCH] D115902: [OPENMP]Look through member function call base during implicit DSA analysis.

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

[PATCH] D115879: [clang-format] extern with new line brace without indentation

owenpan accepted this revision. owenpan added inline comments. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1291-1293 +((Style.IndentExternBlock == FormatStyle::IEBS_Indent) || + (Style.BraceWrapping.AfterExternBlock && +

[PATCH] D114622: [clang-tidy][analyzer] Fix false-positive in IdenticalExprChecker and misc-redundant-expression

steakhal updated this revision to Diff 395150. steakhal added a comment. Sorry for the late update, but here we are: Now, I properly handle all kinds of `NestedNameSpecifiers`. Basically, I verify if the decls are matching, then I use the nested name specifiers for finding mismatches. I look

[PATCH] D115916: [funcattrs] Consistently treat calling a function pointer as a non-capturing read

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG33cbaab1416b: [funcattrs] Consistently treat calling a function pointer as a non-capturing… (authored by reames). Herald added a project: clang.

[clang] 33cbaab - [funcattrs] Consistently treat calling a function pointer as a non-capturing read

Author: Philip Reames Date: 2021-12-17T09:02:03-08:00 New Revision: 33cbaab1416b234d5a08b41e3110d64a00b0651c URL: https://github.com/llvm/llvm-project/commit/33cbaab1416b234d5a08b41e3110d64a00b0651c DIFF: https://github.com/llvm/llvm-project/commit/33cbaab1416b234d5a08b41e3110d64a00b0651c.diff

[PATCH] D115738: [clang-format] Fix formatting of the code that follows C# Lambda Expressions

peterstys added a comment. In D115738#3199988 , @MyDeveloperDay wrote: > Thank you for the patch We'll need your name and email address for that, but > yes we'll be happy to commit it for you. > > If you think you'd like to play some more, then you can

[PATCH] D115938: [clang-format] Formatter does not handle c++11 string literal prefix with stringize #

curdeius accepted this revision. curdeius added a comment. This revision is now accepted and ready to land. LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115938/new/ https://reviews.llvm.org/D115938 ___ cfe-commits mailing list

[PATCH] D115924: [ConstantFolding] Unify handling of load from uniform value

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG9fd4f80e33a4: [ConstantFolding] Unify handling of load from

[clang] 9fd4f80 - [ConstantFolding] Unify handling of load from uniform value

Author: Nikita Popov Date: 2021-12-17T17:05:06+01:00 New Revision: 9fd4f80e33a4ae4567483819646650f5735286e2 URL: https://github.com/llvm/llvm-project/commit/9fd4f80e33a4ae4567483819646650f5735286e2 DIFF: https://github.com/llvm/llvm-project/commit/9fd4f80e33a4ae4567483819646650f5735286e2.diff

[PATCH] D114966: [clang][deps] Split stat and file content caches

jansvoboda11 added inline comments. Comment at: clang/lib/Tooling/DependencyScanning/DependencyScanningFilesystem.cpp:275 + const auto = getOrEmplaceSharedEntryForUID(std::move(TEntry)); + const auto = getOrInsertSharedEntryForFilename(Filename, Entry1); + return

[PATCH] D115938: [clang-format] Formatter does not handle c++11 string literal prefix with stringize #

MyDeveloperDay updated this revision to Diff 395135. MyDeveloperDay added a comment. Add the raw string literal cases CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115938/new/ https://reviews.llvm.org/D115938 Files: clang/lib/Format/TokenAnnotator.cpp

[PATCH] D115738: [clang-format] Fix formatting of the code that follows C# Lambda Expressions

MyDeveloperDay added a comment. Thank you for the patch We'll need your name and email address for that, but yes we'll be happy to commit it for you. If you think you'd like to play some more, then you can apply for commit access. (not sure of the process for that now, but its probably in the

[PATCH] D110622: [HIPSPV][3/4] Enable SPIR-V emission for HIP

yaxunl added a comment. In D110622#3199233 , @linjamaki wrote: > Assuming that this patch is ready to land. @tra or @yaxunl, could you please > commit this patch to the LLVM for us? Thanks. I can help commit this patch. Repository: rG LLVM Github

[PATCH] D115738: [clang-format] Fix formatting of the code that follows C# Lambda Expressions

peterstys added a comment. In D115738#3199896 , @MyDeveloperDay wrote: > In D115738#3199879 , @peterstys > wrote: > >> This is my first PR into this repo. I'd like to learn more about the process >> of

[PATCH] D115942: [X86][MS] Change the alignment of f80 to 16 bytes on Windows 32bits to match with ICC

pengfei added a comment. This is split from D115441 . Add a targeted LLVM test. Thanks @rnk for the advise. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115942/new/ https://reviews.llvm.org/D115942

[PATCH] D115942: [X86][MS] Change the alignment of f80 to 16 bytes on Windows 32bits to match with ICC

pengfei created this revision. pengfei added reviewers: rnk, andrew.w.kaylor, erichkeane, craig.topper. Herald added a subscriber: hiraditya. pengfei requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. MSVC currently doesn't

[PATCH] D115441: [X86][MS] Add 80bit long double support for Windows

pengfei updated this revision to Diff 395129. pengfei added a comment. Split the LLVM datalayout to a different patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115441/new/ https://reviews.llvm.org/D115441 Files:

[PATCH] D115441: [X86][MS] Add 80bit long double support for Windows

pengfei updated this revision to Diff 395128. pengfei added a comment. Split the LLVM datalayout to a different patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115441/new/ https://reviews.llvm.org/D115441 Files:

[PATCH] D115790: [Coroutines] Set presplit attribute in Clang

ezhulenev added a subscriber: mehdi_amini. ezhulenev added a comment. There are two places where in MLIR you can put an attribute to coroutine functions: 1. https://github.com/llvm/llvm-project/blob/main/mlir/lib/Dialect/Async/Transforms/AsyncToAsyncRuntime.cpp#L126 This is the point when

[PATCH] D115738: [clang-format] Fix formatting of the code that follows C# Lambda Expressions

MyDeveloperDay added a comment. In D115738#3199879 , @peterstys wrote: > This is my first PR into this repo. I'd like to learn more about the process > of submitting patches, specifically about: > > - Do I need to get LGTM from all the reviewers before

[PATCH] D115936: [Clang] Add isInNamespace() to check if a Decl in a specific namespace

junaire updated this revision to Diff 395125. junaire added a comment. Format patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115936/new/ https://reviews.llvm.org/D115936 Files: clang/include/clang/AST/DeclBase.h

[clang] 9e45146 - [CodeGen] Fix element type for sret argument

Author: Nikita Popov Date: 2021-12-17T16:13:28+01:00 New Revision: 9e451467217be1325e95eca60bad7924d23e4fe4 URL: https://github.com/llvm/llvm-project/commit/9e451467217be1325e95eca60bad7924d23e4fe4 DIFF: https://github.com/llvm/llvm-project/commit/9e451467217be1325e95eca60bad7924d23e4fe4.diff

[PATCH] D115738: [clang-format] Fix formatting of the code that follows C# Lambda Expressions

peterstys added a comment. This is my first PR into this repo. I'd like to learn more about the process of submitting patches, specifically about: - Do I need to get LGTM from all the reviewers before I can submit it? - There are some build failures on the CI, but look unrelated, do I force

[PATCH] D115932: [Analyzer] Create and handle SymbolCast for pointer to integral conversion

steakhal added a comment. Many thanks for digging into this @martong. I really enjoyed it! I also believe that this is the fix for the underlying issue. I also think the `getAsSymbol()` should be somewhere where we can create new symbols. Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D115938: [clang-format] Formatter does not handle c++11 string literal prefix with stringize #

curdeius added a comment. You shouldn't have added the outer quotes as `#str` adds them. This works: // clang-format off #define MyRawString(str) R#str void foo() { const auto * s1 = MyRawString((" Hello \ world ")); const auto * s2 = MyRawString(abc(" Hello \ world

[PATCH] D115931: [analyzer] Enable move semantics for CallDescriptionMap

steakhal accepted this revision. steakhal added a comment. This revision is now accepted and ready to land. LGTM Comment at: clang/unittests/StaticAnalyzer/CallDescriptionTest.cpp:22 +static_assert(std::is_move_constructible>() && +

[PATCH] D115934: [analyzer] Add range constructor to CallDescriptionMap

steakhal accepted this revision. steakhal added a comment. This revision is now accepted and ready to land. LGTM Comment at: clang/unittests/StaticAnalyzer/CallDescriptionTest.cpp:24-28 +static_assert( +std::is_constructible, +

[PATCH] D115938: [clang-format] Formatter does not handle c++11 string literal prefix with stringize #

MyDeveloperDay added a comment. In D115938#3199832 , @curdeius wrote: > When at it, should we also take care of `LR"(string)"`, `R`, `uR`, `u8R` and > `UR`? Cf. https://en.cppreference.com/w/cpp/language/string_literal > From MS doc: > > // Raw string

[PATCH] D115938: [clang-format] Formatter does not handle c++11 string literal prefix with stringize #

curdeius added a comment. When at it, should we also take care of `LR"(string)"`, `uR...`, `u8R` and `UR`? Cf. https://en.cppreference.com/w/cpp/language/string_literal From MS doc: // Raw string literals containing unescaped \ and " auto R0 = R"("Hello \ world")"; // const char* auto

[PATCH] D115879: [clang-format] extern with new line brace without indentation

curdeius accepted this revision. curdeius added a comment. This revision is now accepted and ready to land. LGTM! Comment at: clang/lib/Format/UnwrappedLineParser.cpp:1285-1287 +if (Style.BraceWrapping.AfterExternBlock) { + addUnwrappedLine(); }

[PATCH] D115938: [clang-format] Formatter does not handle c++11 string literal prefix with stringize #

MyDeveloperDay created this revision. MyDeveloperDay added reviewers: HazardyKnusperkeks, curdeius, owenpan. MyDeveloperDay added projects: clang, clang-format. MyDeveloperDay requested review of this revision. https://github.com/llvm/llvm-project/issues/27740 Ensure #define _u(str) u#str

[PATCH] D115670: Implement some constexpr vector unary operators, fix boolean-ops

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa94f68a2bd31: Implement some constexpr vector unary operators, fix boolean-ops (authored by erichkeane). Herald added a project: clang. Repository:

[clang] a94f68a - Implement some constexpr vector unary operators, fix boolean-ops

Author: Erich Keane Date: 2021-12-17T06:08:36-08:00 New Revision: a94f68a2bd31db9712582b973e78746cee9a4e50 URL: https://github.com/llvm/llvm-project/commit/a94f68a2bd31db9712582b973e78746cee9a4e50 DIFF: https://github.com/llvm/llvm-project/commit/a94f68a2bd31db9712582b973e78746cee9a4e50.diff

[PATCH] D115936: [Clang] Add isInNamespace() to check if a Decl in a specific namespace

junaire created this revision. junaire added reviewers: rtrieu, ddunbar, CornedBee, gribozavr. junaire requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Currently we can only check whether a Decl in namespace `std` or not. However, it will

[PATCH] D113837: Sema: Let InitListExpr have dependent type instead of 'void'

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

[PATCH] D114971: [clang][deps] Handle symlinks in minimizing FS

jansvoboda11 updated this revision to Diff 395104. jansvoboda11 added a comment. Rebase on top of new version of D114966 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114971/new/ https://reviews.llvm.org/D114971

[PATCH] D114966: [clang][deps] Split stat and file content caches

jansvoboda11 updated this revision to Diff 395103. jansvoboda11 added a comment. Re-run CI. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114966/new/ https://reviews.llvm.org/D114966 Files:

[PATCH] D114966: [clang][deps] Split stat and file content caches

jansvoboda11 updated this revision to Diff 395102. jansvoboda11 added a comment. Implement new version that ensures the stat and contents of a file are always in sync. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114966/new/

[PATCH] D115935: [clang][deps] NFC: Simplify handling of cached FS errors

jansvoboda11 updated this revision to Diff 395101. jansvoboda11 added a comment. Add more assertions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115935/new/ https://reviews.llvm.org/D115935 Files:

[PATCH] D115935: [clang][deps] NFC: Simplify handling of cached FS errors

jansvoboda11 created this revision. jansvoboda11 added reviewers: Bigcheese, dexonsmith, arphaman. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The return types of some `CachedFileSystemEntry` member function are needlessly

[PATCH] D115934: [analyzer] Add range constructor to CallDescriptionMap

gamesh411 created this revision. gamesh411 added reviewers: steakhal, martong. Herald added subscribers: manas, ASDenysPetrov, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware, xazax.hun. Herald added a reviewer: Szelethus. gamesh411 requested review

[clang] 3f3b5c3 - [clang][deps] NFC: Unify ErrorOr patterns

Author: Jan Svoboda Date: 2021-12-17T14:00:20+01:00 New Revision: 3f3b5c3ec0da8f4982a8645fdb403de582295f0b URL: https://github.com/llvm/llvm-project/commit/3f3b5c3ec0da8f4982a8645fdb403de582295f0b DIFF: https://github.com/llvm/llvm-project/commit/3f3b5c3ec0da8f4982a8645fdb403de582295f0b.diff

[clang] bcdf7f5 - [clang][deps] NFC: Take and store entry as reference

Author: Jan Svoboda Date: 2021-12-17T14:00:20+01:00 New Revision: bcdf7f5e9104cb87d06969d74f4feb0cc7aba08f URL: https://github.com/llvm/llvm-project/commit/bcdf7f5e9104cb87d06969d74f4feb0cc7aba08f DIFF: https://github.com/llvm/llvm-project/commit/bcdf7f5e9104cb87d06969d74f4feb0cc7aba08f.diff

[clang] af7a421 - [clang][deps] NFC: Remove explicit call to implicit constructor

Author: Jan Svoboda Date: 2021-12-17T14:00:20+01:00 New Revision: af7a421ef4aaf3d098ba78c1dda8ac35d31136cd URL: https://github.com/llvm/llvm-project/commit/af7a421ef4aaf3d098ba78c1dda8ac35d31136cd DIFF: https://github.com/llvm/llvm-project/commit/af7a421ef4aaf3d098ba78c1dda8ac35d31136cd.diff

[clang] 195a529 - [clang][deps] NFC: Rename member variable

Author: Jan Svoboda Date: 2021-12-17T14:00:20+01:00 New Revision: 195a5294c28e982607cc11bc63c3be1c0e3c312b URL: https://github.com/llvm/llvm-project/commit/195a5294c28e982607cc11bc63c3be1c0e3c312b DIFF: https://github.com/llvm/llvm-project/commit/195a5294c28e982607cc11bc63c3be1c0e3c312b.diff

[clang] 4170ea9 - [clang][deps] NFC: Fix whitespace formatting

Author: Jan Svoboda Date: 2021-12-17T14:00:20+01:00 New Revision: 4170ea9445cc407ec1bf8cfa5bc5ce2d5295fa36 URL: https://github.com/llvm/llvm-project/commit/4170ea9445cc407ec1bf8cfa5bc5ce2d5295fa36 DIFF: https://github.com/llvm/llvm-project/commit/4170ea9445cc407ec1bf8cfa5bc5ce2d5295fa36.diff

[PATCH] D115932: [Analyzer] Create and handle SymbolCast for pointer to integral conversion

martong added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/SValBuilder.cpp:417 + // the member function of SValBuilder (?) + if (symRHS) +if (auto RLocAsInt = RHS.getAs()) { We should handle LHS as well. Repository: rG LLVM Github

[PATCH] D115149: [analyzer][solver] Fix assertion on (NonLoc, Op, Loc) expressions

martong added a comment. In D115149#3181580 , @NoQ wrote: > Like, that's the whole reason why `nonloc::LocAsInteger` exists: so that we > could cast a pointer to an integer and actually have a way to represent the > resulting value as `NonLoc`. > I'm

  1   2   >