[PATCH] D155239: [clang-format] Add SpacesInParens with SpacesInParensOptions

MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land. This looks good can you wait for one of the others to accept too Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155239/new/

[PATCH] D155539: [CUDA][HIP] Use the same default language std as C++

yaxunl updated this revision to Diff 541836. yaxunl added a comment. update release note CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155539/new/ https://reviews.llvm.org/D155539 Files: clang/docs/ReleaseNotes.rst clang/include/clang/Basic/LangStandards.def

[PATCH] D155670: [c-index-test] Suppress -Wcast-qual after D153911

MaskRay abandoned this revision. MaskRay added a comment. commit 0f0c3d45d7d75ba82a955246da654146a7d57a0d did this first Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D155545: [clang][Interp] Pass CallExpr to builtin functions

tbaeder updated this revision to Diff 541834. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155545/new/ https://reviews.llvm.org/D155545 Files: clang/lib/AST/Interp/ByteCodeExprGen.cpp clang/lib/AST/Interp/Interp.h clang/lib/AST/Interp/InterpBuiltin.cpp

[PATCH] D155356: [clang][Interp] Implement __builtin_nan family of functions

tbaeder added a comment. One thing I'm wondering though is: constexpr char A[] = {'1', '2', '3'}; constexpr double d = __builtin_nans(A); what's expected here? The given char array is not null-terminated. Should the interpreter ignore the null byte at the end if it's there?

[clang] 12f35b3 - [tools] Use "#pragma GCC" instead of "#pragma clang" to ignore -Wcast-qual in c-index-test.c (NFC)

Author: Jie Fu Date: 2023-07-19T12:05:05+08:00 New Revision: 12f35b39ee63e932d4bdf6b28cce2e3d07713c4d URL: https://github.com/llvm/llvm-project/commit/12f35b39ee63e932d4bdf6b28cce2e3d07713c4d DIFF: https://github.com/llvm/llvm-project/commit/12f35b39ee63e932d4bdf6b28cce2e3d07713c4d.diff LOG:

[PATCH] D155356: [clang][Interp] Implement __builtin_nan family of functions

tbaeder updated this revision to Diff 541833. tbaeder marked an inline comment as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155356/new/ https://reviews.llvm.org/D155356 Files: clang/lib/AST/Interp/InterpBuiltin.cpp clang/test/AST/Interp/builtin-functions.cpp Index:

[PATCH] D155356: [clang][Interp] Implement __builtin_nan family of functions

tbaeder marked 2 inline comments as done. tbaeder added inline comments. Comment at: clang/lib/AST/Interp/InterpBuiltin.cpp:106 +// 2008 revisions, MIPS interpreted sNaN-2008 as qNan and qNaN-2008 as +// sNaN. This is now known as "legacy NaN" encoding. +if

[PATCH] D155356: [clang][Interp] Implement __builtin_nan family of functions

tbaeder updated this revision to Diff 541832. tbaeder marked 3 inline comments as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155356/new/ https://reviews.llvm.org/D155356 Files: clang/lib/AST/Interp/InterpBuiltin.cpp clang/test/AST/Interp/builtin-functions.cpp Index:

[PATCH] D155647: [RISCV] Add C intrinsics for scalar crypto

wangpc added a comment. Can we run mem2reg pass in RUNs just like RVV tests (maybe another cleanup patch)? I think there are a lot of noises in CHECKS. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155647/new/ https://reviews.llvm.org/D155647

[clang] 16d5078 - [tools] Fix buildbot build failure

Author: Jie Fu Date: 2023-07-19T11:39:00+08:00 New Revision: 16d50781e16374f5bf930ba0b9aa8cbbb034ee6c URL: https://github.com/llvm/llvm-project/commit/16d50781e16374f5bf930ba0b9aa8cbbb034ee6c DIFF: https://github.com/llvm/llvm-project/commit/16d50781e16374f5bf930ba0b9aa8cbbb034ee6c.diff LOG:

[PATCH] D155147: [X86] Add SM3 instructions.

FreddyYe updated this revision to Diff 541824. FreddyYe marked an inline comment as done. FreddyYe added a comment. Address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155147/new/ https://reviews.llvm.org/D155147 Files:

[clang] 0f0c3d4 - [tools] Ignore -Wcast-qual in c-index-test.c after D153911 (NFC)

Author: Jie Fu Date: 2023-07-19T10:55:12+08:00 New Revision: 0f0c3d45d7d75ba82a955246da654146a7d57a0d URL: https://github.com/llvm/llvm-project/commit/0f0c3d45d7d75ba82a955246da654146a7d57a0d DIFF: https://github.com/llvm/llvm-project/commit/0f0c3d45d7d75ba82a955246da654146a7d57a0d.diff LOG:

[PATCH] D155671: [include-cleaner] allow spelling strategies to customize verbatim/system headers

sammccall created this revision. sammccall added a reviewer: kadircet. Herald added a project: All. sammccall requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. Our use case is wanting to apply a spelling strategy to rewrite the

[PATCH] D155147: [X86] Add SM3 instructions.

pengfei added inline comments. Comment at: clang/lib/Headers/sm3intrin.h:162 +/// \param imm8 +///A 128-bit vector of [4 x int]. +/// \returns This is `int` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D155670: [c-index-test] Suppress -Wcast-qual after D153911

MaskRay created this revision. MaskRay added reviewers: AlexM, dblaikie. Herald added a subscriber: arphaman. Herald added a project: All. MaskRay requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. free_remapped_files needs to discard the

[PATCH] D155416: [Clang][RISCV] Improve diagnostic message for full multiply intrinsics

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG28c3a74a5c6c: [Clang][RISCV] Improve diagnostic message for full multiply intrinsics (authored by eopXD). Repository: rG LLVM Github Monorepo

[clang] 28c3a74 - [Clang][RISCV] Improve diagnostic message for full multiply intrinsics

Author: eopXD Date: 2023-07-18T19:39:46-07:00 New Revision: 28c3a74a5c6c7ed1ac97a010ca080eaacfd7f324 URL: https://github.com/llvm/llvm-project/commit/28c3a74a5c6c7ed1ac97a010ca080eaacfd7f324 DIFF: https://github.com/llvm/llvm-project/commit/28c3a74a5c6c7ed1ac97a010ca080eaacfd7f324.diff LOG:

[PATCH] D155147: [X86] Add SM3 instructions.

FreddyYe updated this revision to Diff 541815. FreddyYe marked an inline comment as done. FreddyYe added a comment. Address comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155147/new/ https://reviews.llvm.org/D155147 Files:

[PATCH] D155147: [X86] Add SM3 instructions.

pengfei added inline comments. Comment at: clang/lib/Headers/sm3intrin.h:161-164 +/// \param imm8 +///A 128-bit vector of [4 x int]. +/// \returns +///A 32-bit int. The description should invert Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D155147: [X86] Add SM3 instructions.

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

[PATCH] D154755: [clang-format] Fix formatting of if statements with BlockIndent

gedare added inline comments. Comment at: clang/unittests/Format/FormatTest.cpp:25486 "comment\n" - " return;\n" "}", MyDeveloperDay wrote: > Why remove? oops. that's a spurious change. I have reverted it.

[PATCH] D154755: [clang-format] Fix formatting of if statements with BlockIndent

gedare updated this revision to Diff 541813. gedare marked an inline comment as done. gedare added a comment. Revert deleted line in test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154755/new/ https://reviews.llvm.org/D154755 Files:

[PATCH] D155148: [X86] Add SM4 instructions.

skan accepted this revision. skan added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155148/new/ https://reviews.llvm.org/D155148 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D154366: [clang][ExprConstant] Print template arguments when describing stack frame

shafik added a comment. I think this looks fine, I just wonder if we should be adding more tests to make sure we cover the a fuller set of types and non-type template parameters. I feel like this is always what bites us when bugs come up, if we had just test more carefully we would have caught

[PATCH] D155414: [Clang][RISCV] Guard RVV intrinsics types that is not available when ELEN < 64

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc4a5b5849767: [Clang][RISCV] Guard RVV intrinsics types that is not available when ELEN 64 (authored by eopXD). Repository: rG LLVM Github

[clang] c4a5b58 - [Clang][RISCV] Guard RVV intrinsics types that is not available when ELEN < 64

Author: eopXD Date: 2023-07-18T18:57:33-07:00 New Revision: c4a5b58497677f6be2618765f89b08462e820337 URL: https://github.com/llvm/llvm-project/commit/c4a5b58497677f6be2618765f89b08462e820337 DIFF: https://github.com/llvm/llvm-project/commit/c4a5b58497677f6be2618765f89b08462e820337.diff LOG:

[PATCH] D155668: [RISCV] Upgrade Zvfh version to 1.0 and move out of experimental state.

craig.topper created this revision. craig.topper added reviewers: asb, reames, kito-cheng. Herald added subscribers: jobnoorman, luke, VincentWu, vkmr, frasercrmck, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, arphaman, the_o, brucehoult, MartinMosbeck,

[PATCH] D155667: [-Wunsafe-buffer-usage] Check source location validity before using `TypeLoc`s

ziqingluo-90 created this revision. ziqingluo-90 added reviewers: NoQ, jkorous, t-rasmud, malavikasamak. Herald added a project: All. ziqingluo-90 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The safe-buffer analysis analyzes `TypeLoc`s

[clang] 8f54b83 - [clang-format][doc] Replace single back quotes with double ones

Author: Owen Pan Date: 2023-07-18T18:27:15-07:00 New Revision: 8f54b8331f9f083220eab06fe903b47115441dbe URL: https://github.com/llvm/llvm-project/commit/8f54b8331f9f083220eab06fe903b47115441dbe DIFF: https://github.com/llvm/llvm-project/commit/8f54b8331f9f083220eab06fe903b47115441dbe.diff

[PATCH] D155012: Fix types of arm64 MSVC __readx18/__writex18 intrinsics

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG4ae87b3f8adc: Fix types of arm64 MSVC __readx18/__writex18 intrinsics (authored by ahatanak). Repository: rG LLVM Github Monorepo CHANGES SINCE

[clang] 4ae87b3 - Fix types of arm64 MSVC __readx18/__writex18 intrinsics

Author: Akira Hatanaka Date: 2023-07-18T18:16:20-07:00 New Revision: 4ae87b3f8adc35faa3769f2929e51940dd397803 URL: https://github.com/llvm/llvm-project/commit/4ae87b3f8adc35faa3769f2929e51940dd397803 DIFF:

[PATCH] D151587: [clang][ConstantEmitter] have tryEmitPrivate[ForVarInit] try ConstExprEmitter fast-path first

efriedma added inline comments. Comment at: clang/lib/AST/ExprConstant.cpp:8360 + // Do not constant fold an R-value. + if (Info.EvalMode == EvalInfo::EM_ConstantFold && !E->isLValue()) +return false; nickdesaulniers wrote: > efriedma wrote: > > Checking

[PATCH] D155146: [X86] Add SHA512 instructions.

FreddyYe marked 3 inline comments as done. FreddyYe added a comment. I think we can discuss this issue first in https://reviews.llvm.org/D155662 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155146/new/ https://reviews.llvm.org/D155146

[PATCH] D138810: [RISCV] Support vector crypto extension C intrinsics

craig.topper added inline comments. Comment at: clang/lib/Sema/SemaChecking.cpp:4576 + case RISCVVector::BI__builtin_rvv_vaeskf1_vi_ta: + case RISCVVector::BI__builtin_rvv_vsm4k_vi_ta: +return SemaBuiltinConstantArgRange(TheCall, 1, 0, 31); craig.topper

[PATCH] D155146: [X86] Add SHA512 instructions.

FreddyYe updated this revision to Diff 541793. FreddyYe added a comment. Add missing doxygen Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155146/new/ https://reviews.llvm.org/D155146 Files: clang/docs/ReleaseNotes.rst

[PATCH] D153059: [-Wunsafe-buffer-usage] Group parameter fix-its

t-rasmud added inline comments. Comment at: clang/test/SemaCXX/warn-unsafe-buffer-usage-fixits-multi-parm-span.cpp:306 + b[5] = 5; // expected-note{{used in buffer access here}} +} Can we have a test case with qualifiers on parameters and maybe another with a

[PATCH] D155147: [X86] Add SM3 instructions.

FreddyYe added inline comments. Comment at: clang/lib/Headers/sm3intrin.h:230 +/// \endcode +#define _mm_sm3rnds2_epi32(A, B, C, D) \ + (__m128i) __builtin_ia32_vsm3rnds2((__v4su)A, (__v4su)B, (__v4su)C, (int)D) pengfei

[PATCH] D155147: [X86] Add SM3 instructions.

FreddyYe updated this revision to Diff 541792. FreddyYe marked 5 inline comments as done. FreddyYe added a comment. Address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155147/new/ https://reviews.llvm.org/D155147 Files:

[PATCH] D155661: [ASTImporter] Fix recursive friend class template with non-type parm

danix800 created this revision. Herald added subscribers: martong, kristof.beyls. Herald added a reviewer: a.sidorin. Herald added a reviewer: shafik. Herald added a project: All. danix800 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits.

[PATCH] D155148: [X86] Add SM4 instructions.

FreddyYe updated this revision to Diff 541787. FreddyYe marked an inline comment as done. FreddyYe added a comment. Address comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155148/new/ https://reviews.llvm.org/D155148 Files:

[clang] b43df5b - PseudoObjectExpr: Prefer ArrayRef over iterator_range when iterating with pointers

Author: David Blaikie Date: 2023-07-18T23:43:41Z New Revision: b43df5bfe7e7ef358e135b515b0651ec51f635d8 URL: https://github.com/llvm/llvm-project/commit/b43df5bfe7e7ef358e135b515b0651ec51f635d8 DIFF: https://github.com/llvm/llvm-project/commit/b43df5bfe7e7ef358e135b515b0651ec51f635d8.diff

[PATCH] D153536: [Clang] Implement P2169 A nice placeholder with no name

dblaikie added a comment. In D153536#4490918 , @cor3ntin wrote: > @dblaikie Would you be willing to look at the debugger side of things in a > subsequent patch? I'm not familiar with debug symbol code gen so I'm not sure > I'd be able to improve thing

[PATCH] D155012: Fix types of arm64 MSVC __readx18/__writex18 intrinsics

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

[PATCH] D154366: [clang][ExprConstant] Print template arguments when describing stack frame

dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land. Sounds good to me CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154366/new/ https://reviews.llvm.org/D154366 ___ cfe-commits mailing

[PATCH] D152632: [Clang] Add warnings for CWG2521

This revision was automatically updated to reflect the committed changes. Closed by commit rG5ce5e983f82c: [Clang] Add warnings for CWG2521 (authored by rZhBoYao). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D152632/new/

[clang] 5ce5e98 - [Clang] Add warnings for CWG2521

Author: Po-yao Chang Date: 2023-07-19T07:23:34+08:00 New Revision: 5ce5e983f82c802e44faa8ed42d605d70c045ba9 URL: https://github.com/llvm/llvm-project/commit/5ce5e983f82c802e44faa8ed42d605d70c045ba9 DIFF: https://github.com/llvm/llvm-project/commit/5ce5e983f82c802e44faa8ed42d605d70c045ba9.diff

[PATCH] D155524: [-Wunsafe-buffer-usage] Ignore the FixableGadgets that will not be fixed at an earlier stage

ziqingluo-90 added inline comments. Comment at: clang/lib/Analysis/UnsafeBufferUsage.cpp:1095-1102 // Gadgets "claim" variables they're responsible for. Once this loop finishes, // the tracker will only track DREs that weren't claimed by any gadgets, // i.e. not

[PATCH] D154880: [-Wunsafe-buffer-usage][WIP] Add a facility for debugging low fixit coverage.

NoQ added inline comments. Comment at: clang/lib/Analysis/UnsafeBufferUsage.cpp:2248-2249 +#ifndef NDEBUG +// FIXME: F->getBaseStmt() should never be null! +// (Or we should build a better interface for this.) +Handler.addDebugNoteForVar(

[PATCH] D154880: [-Wunsafe-buffer-usage][WIP] Add a facility for debugging low fixit coverage.

NoQ added a comment. Awesome!! Did you try running it on some real code? Does this actually cover most cases? (I suspect that (1.) is going to be the most popular case, but that's also the easiest case to diagnose visually. We might still want a note if we wanted to prioritize among

[PATCH] D151587: [clang][ConstantEmitter] have tryEmitPrivate[ForVarInit] try ConstExprEmitter fast-path first

nickdesaulniers added inline comments. Comment at: clang/lib/AST/ExprConstant.cpp:8360 + // Do not constant fold an R-value. + if (Info.EvalMode == EvalInfo::EM_ConstantFold && !E->isLValue()) +return false; efriedma wrote: > Checking isLValue() doesn't

[PATCH] D155529: [clang-format] Add SpaceInParensOption for __attribute__ keyword

gedare planned changes to this revision. gedare added a comment. I need to fix this to reflect the changes in the parent rev. Comment at: clang/include/clang/Format/Format.h:4208-4213 +/// Put a space in parentheses inside attribute specifier lists. +/// \code +///

[PATCH] D155239: [clang-format] Add SpacesInParens with SpacesInParensOptions

gedare added a comment. In D155239#4509921 , @HazardyKnusperkeks wrote: > If you limit it to `Never` I don't see any value in the differentiation. You > could just always use `Custom` (by dropping the custom and only having the > nested options). > >

[PATCH] D155239: [clang-format] Add SpacesInParens with SpacesInParensOptions

gedare updated this revision to Diff 541770. gedare added a comment. Parse deprecated options and map to new ones. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155239/new/ https://reviews.llvm.org/D155239 Files:

[PATCH] D155134: [clang][docs] Defensively turn off exception behavior in dump_ast_matchers.py

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1e62587a48a3: [clang][docs] Defensively turn off exception behavior in dump_ast_matchers.py (authored by t-rasmud). Herald added a project: clang.

[clang] 1e62587 - [clang][docs] Defensively turn off exception behavior in dump_ast_matchers.py

Author: Rashmi Mudduluru Date: 2023-07-18T15:36:09-07:00 New Revision: 1e62587a48a33b3bf5939e1eef2fd4e41b7e75f6 URL: https://github.com/llvm/llvm-project/commit/1e62587a48a33b3bf5939e1eef2fd4e41b7e75f6 DIFF:

[PATCH] D155647: [RISCV] Add C intrinsics for scalar crypto

craig.topper updated this revision to Diff 541759. craig.topper added a comment. Fix name of zip/unzip builtin Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155647/new/ https://reviews.llvm.org/D155647 Files: clang/lib/Headers/CMakeLists.txt

[PATCH] D155304: [clang][docs] Update LibASTMatchersReference.html

NoQ added a subscriber: ccotter. NoQ added a comment. @ccotter: now your matcher is properly documented in https://clang.llvm.org/docs/LibASTMatchersReference.html Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155304/new/

[PATCH] D155273: [clang-format] Add TypeNames option to disambiguate types/objects

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

[PATCH] D155304: [clang][docs] Update LibASTMatchersReference.html

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG3ea673a97b05: [clang][docs] Update LibASTMatchersReference.html (authored by t-rasmud). Herald added a project: clang. Herald added a subscriber:

[clang] 3ea673a - [clang][docs] Update LibASTMatchersReference.html

Author: Rashmi Mudduluru Date: 2023-07-18T15:27:59-07:00 New Revision: 3ea673a97b0583affc22345b9d62e863ba36b3d8 URL: https://github.com/llvm/llvm-project/commit/3ea673a97b0583affc22345b9d62e863ba36b3d8 DIFF:

[PATCH] D155651: [SystemZ][z/OS] Add OpenFlags to CreateMissingDirectories path when creating temp files

tltao created this revision. Herald added a project: All. tltao requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Additional patch to https://reviews.llvm.org/D103806 to add the same flags in the path where the CreateMissingDirectories

[PATCH] D155641: [-Wunsafe-buffer-usage] Do not assert that function parameters have names.

NoQ added a comment. I nitpicked to comments, but other than that LGTM, thanks! Comment at: clang/lib/Analysis/UnsafeBufferUsage.cpp:1890-1891 + // A parameter of a function definition has no name. + // FIXME: We should create a name for the parameter as part of the

[PATCH] D155524: [-Wunsafe-buffer-usage] Ignore the FixableGadgets that will not be fixed at an earlier stage

NoQ added inline comments. Comment at: clang/lib/Analysis/UnsafeBufferUsage.cpp:1095-1102 // Gadgets "claim" variables they're responsible for. Once this loop finishes, // the tracker will only track DREs that weren't claimed by any gadgets, // i.e. not understood by

[PATCH] D154382: [ClangRepl] support code completion at a REPL

capfredf updated this revision to Diff 541753. capfredf added a comment. Remove unnecessary braces Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154382/new/ https://reviews.llvm.org/D154382 Files:

[PATCH] D155647: [RISCV] Add C intrinsics for scalar crypto

craig.topper updated this revision to Diff 541752. craig.topper added a comment. Fix file description Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155647/new/ https://reviews.llvm.org/D155647 Files: clang/lib/Headers/CMakeLists.txt

[PATCH] D155647: [RISCV] Add C intrinsics for scalar crypto

craig.topper updated this revision to Diff 541751. craig.topper added a comment. git add the header Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155647/new/ https://reviews.llvm.org/D155647 Files: clang/lib/Headers/CMakeLists.txt

[PATCH] D155647: [RISCV] Add C intrinsics for scalar crypto

craig.topper created this revision. craig.topper added reviewers: asb, VincentWu, wangpc, kito-cheng. Herald added subscribers: jobnoorman, luke, vkmr, frasercrmck, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01,

[PATCH] D154880: [-Wunsafe-buffer-usage][WIP] Add a facility for debugging low fixit coverage.

t-rasmud updated this revision to Diff 541745. t-rasmud added a comment. This patch addresses cases 2, 3, and 4 described in the summary (i.e) adds debug notes for unclaimed uses of variables and for failed fixit generation of variable declarations. CHANGES SINCE LAST ACTION

[PATCH] D155636: [clang][dataflow] Add function to `WatchedLiteralsSolver` that reports whether the iteration limit has been reached.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1defa781243f: [clang][dataflow] Add function to `WatchedLiteralsSolver` that reports whether… (authored by ymandel). Repository: rG LLVM Github

[clang] 1defa78 - [clang][dataflow] Add function to `WatchedLiteralsSolver` that reports whether the iteration limit has been reached.

Author: Yitzhak Mandelbaum Date: 2023-07-18T21:43:55Z New Revision: 1defa781243f9d0bc66719465e4de33e9fb7a243 URL: https://github.com/llvm/llvm-project/commit/1defa781243f9d0bc66719465e4de33e9fb7a243 DIFF: https://github.com/llvm/llvm-project/commit/1defa781243f9d0bc66719465e4de33e9fb7a243.diff

[PATCH] D155414: [Clang][RISCV] Guard RVV intrinsics types that is not available when ELEN < 64

craig.topper accepted this revision. craig.topper 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/D155414/new/ https://reviews.llvm.org/D155414

[PATCH] D134475: [clang-cl] Add support for [[msvc::constexpr]] C++11 attribute

rsmith added inline comments. Comment at: clang/lib/AST/ExprConstant.cpp:9591-9594 + bool IsMSConstexpr = Info.CurrentCall->CanEvalMSConstexpr && + OperatorNew->hasAttr(); if (OperatorNew->isReservedGlobalPlacementOperator() && +

[PATCH] D134475: [clang-cl] Add support for [[msvc::constexpr]] C++11 attribute

rsmith added inline comments. Comment at: clang/lib/AST/ExprConstant.cpp:5615-5627 +if (canEvalMSConstexpr || isMSConstexpr) { + // Diagnose invalid usage of [[msvc::constexpr]] function + bool isConstructor = isa(Definition); + if (canEvalMSConstexpr) { //

[PATCH] D155642: Unconditionally include intrinsic headers in MSVC mode

Neumann-A created this revision. Herald added a project: All. Neumann-A requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Even though there is a significant compile time overhead for all users being a true drop in replacement for cl

[PATCH] D155273: [clang-format] Add TypeNames option to disambiguate types/objects

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG5c106f7b947e: [clang-format] Add TypeNames option to disambiguate types/objects (authored by owenpan). Repository: rG LLVM Github Monorepo

[clang] 5c106f7 - [clang-format] Add TypeNames option to disambiguate types/objects

Author: Owen Pan Date: 2023-07-18T14:18:40-07:00 New Revision: 5c106f7b947e514852402ad5678c0ebf70ce91b1 URL: https://github.com/llvm/llvm-project/commit/5c106f7b947e514852402ad5678c0ebf70ce91b1 DIFF: https://github.com/llvm/llvm-project/commit/5c106f7b947e514852402ad5678c0ebf70ce91b1.diff

[PATCH] D134588: [clang-tidy] Fix bugprone-exception-escape warn on noexcept calls

PiotrZSL abandoned this revision. PiotrZSL added a comment. Already fixed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134588/new/ https://reviews.llvm.org/D134588 ___ cfe-commits mailing list

[PATCH] D155641: [-Wunsafe-buffer-usage] Do not assert that function parameters have names.

ziqingluo-90 created this revision. ziqingluo-90 added reviewers: jkorous, NoQ, t-rasmud, malavikasamak. Herald added a project: All. ziqingluo-90 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. It is possible that a function parameter

[PATCH] D153423: [clang-tidy] Allow explicit throwing in bugprone-exception-escape for special functions

This revision was automatically updated to reflect the committed changes. Closed by commit rGff40c3488177: [clang-tidy] Allow explicit throwing in bugprone-exception-escape for special… (authored by PiotrZSL). Changed prior to commit: https://reviews.llvm.org/D153423?vs=534059=541727#toc

[PATCH] D151047: [clang-format] Fix indent for selective formatting.

This revision was automatically updated to reflect the committed changes. Closed by commit rG0570cc568f5c: [clang-format] Fix indent for selective formatting (authored by Sedeniono, committed by owenpan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] ff40c34 - [clang-tidy] Allow explicit throwing in bugprone-exception-escape for special functions

Author: Piotr Zegar Date: 2023-07-18T21:11:08Z New Revision: ff40c34881772e0641027327cd1791f4acacf6ff URL: https://github.com/llvm/llvm-project/commit/ff40c34881772e0641027327cd1791f4acacf6ff DIFF: https://github.com/llvm/llvm-project/commit/ff40c34881772e0641027327cd1791f4acacf6ff.diff LOG:

[clang] 0570cc5 - [clang-format] Fix indent for selective formatting

Author: Sedenion Date: 2023-07-18T14:10:53-07:00 New Revision: 0570cc568f5c8047267be7ab5056f8c4f45349b5 URL: https://github.com/llvm/llvm-project/commit/0570cc568f5c8047267be7ab5056f8c4f45349b5 DIFF: https://github.com/llvm/llvm-project/commit/0570cc568f5c8047267be7ab5056f8c4f45349b5.diff

[PATCH] D155342: [clang][JumpDiagnostics] ignore non-asm goto target scopes

rjmccall added a comment. Yes, this diagnostic implements a core semantic rule and must be emitted. There are some situations where Clang will actually generate malformed IR (violating dominance rules) if you fail to diagnose jump violations correctly. The IR you get when there's a jump over

[PATCH] D155580: [trivial-auto-var-init] Do not emit initialization code for empty class

jfb added a comment. (sorry for sending twice, looks like email reply failed) This is the same as padding, and is initialized on purpose. If it’s truly unused then the optimizer will eliminate it… unless it can’t prove whether the store is dead. Does this show up in any meaningless

[PATCH] D153310: Add codegen for llvm pow builtin

bob80905 updated this revision to Diff 541721. bob80905 added a comment. - add newline EOF Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153310/new/ https://reviews.llvm.org/D153310 Files: clang/docs/LanguageExtensions.rst

[PATCH] D153310: Add codegen for llvm pow builtin

bob80905 updated this revision to Diff 541717. bob80905 added a comment. - add strictfp test, use emitbinarybuiltin, fix some whitespace Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153310/new/ https://reviews.llvm.org/D153310 Files:

[PATCH] D155342: [clang][JumpDiagnostics] ignore non-asm goto target scopes

jyknight added inline comments. Comment at: clang/lib/Sema/JumpDiagnostics.cpp:658 +if (auto *G = dyn_cast(Jump)) { + for (AddrLabelExpr *L : G->labels()) { efriedma wrote: > nickdesaulniers wrote: > > efriedma wrote: > > > nickdesaulniers wrote: > >

[PATCH] D155342: [clang][JumpDiagnostics] ignore non-asm goto target scopes

efriedma added inline comments. Comment at: clang/lib/Sema/JumpDiagnostics.cpp:658 +if (auto *G = dyn_cast(Jump)) { + for (AddrLabelExpr *L : G->labels()) { nickdesaulniers wrote: > efriedma wrote: > > nickdesaulniers wrote: > > > rjmccall wrote: > >

[PATCH] D155635: [OpenMP] [Reduction] Allow PLUS (+) operator on reduction clauses in OMP > 52

ABataev added a comment. In D155635#4512188 , @mdfazlay wrote: > In D155635#4512183 , @ABataev wrote: > >> Tests? > > I think I need to use `-fopenmp-version=60` to test this revision. As OMP 6.0 > is not

[PATCH] D155636: [clang][dataflow] Add function to `WatchedLiteralsSolver` that reports whether the iteration limit has been reached.

ymandel created this revision. ymandel added reviewers: xazax.hun, mboehme. Herald added a subscriber: martong. Herald added a reviewer: NoQ. Herald added a project: All. ymandel requested review of this revision. Herald added a project: clang. This change provides a centralized record of whether

[PATCH] D155635: [OpenMP] [Reduction] Allow PLUS (+) operator on reduction clauses in OMP > 52

mdfazlay added a comment. In D155635#4512183 , @ABataev wrote: > Tests? I think I need to use `-fopenmp-version=60` to test this revision. As OMP 6.0 is not official yet, am I allowed to use `-fopenmp-version=60` in my LIT test? Repository: rG LLVM

[PATCH] D155094: [clang-format] Refactoring and asserts in LevelIndentTracker. (NFC)

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGe4aa1428a28d: [clang-format] Refactoring and asserts in LevelIndentTracker. (NFC) (authored by Sedeniono, committed by owenpan). Repository: rG

[clang] e4aa142 - [clang-format] Refactoring and asserts in LevelIndentTracker. (NFC)

Author: Sedenion Date: 2023-07-18T13:32:07-07:00 New Revision: e4aa1428a28dce38cd4712b5720baa333cbdd25b URL: https://github.com/llvm/llvm-project/commit/e4aa1428a28dce38cd4712b5720baa333cbdd25b DIFF: https://github.com/llvm/llvm-project/commit/e4aa1428a28dce38cd4712b5720baa333cbdd25b.diff

[PATCH] D155635: [OpenMP] [Reduction] Allow PLUS (+) operator on reduction clauses in OMP > 52

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

[PATCH] D134475: [clang-cl] Add support for [[msvc::constexpr]] C++11 attribute

RIscRIpt added a comment. According to git blame @rsmith has done lots of changes in `clang/lib/AST/ExprConstant.cpp`; added as reviewer. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134475/new/ https://reviews.llvm.org/D134475

[PATCH] D155342: [clang][JumpDiagnostics] ignore non-asm goto target scopes

nickdesaulniers added a comment. In D155342#4511879 , @rjmccall wrote: >> We need to check the scopes like we would for direct goto, because we don't >> want to bypass non-trivial destructors. > > I think this is the basic misunderstanding here. Direct

[PATCH] D155342: [clang][JumpDiagnostics] ignore non-asm goto target scopes

nickdesaulniers updated this revision to Diff 541705. nickdesaulniers added a comment. - rewrite comment based on feedback from @rjmccall and @efriedma Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155342/new/ https://reviews.llvm.org/D155342

[PATCH] D155635: [OpenMP] [Reduction] Allow PLUS (+) operator on reduction clauses in OMP > 52

mdfazlay created this revision. Herald added subscribers: sunshaoce, guansong, yaxunl. Herald added a project: All. mdfazlay requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, jplehr, sstefan1. Herald added a project: clang. Currently,

[PATCH] D155625: [clang-tidy] Warn only for copyable/movable classes in cppcoreguidelines-avoid-const-or-ref-members

PiotrZSL accepted this revision. PiotrZSL added a comment. This revision is now accepted and ready to land. LGTM, but I'm not sure if isCopyableOrMovable will always work correctly, for a user defined special members it looks ok, but for some implicit ones I worry it may not always work.

  1   2   3   4   >