[clang] b646f09 - [clang-format] Fix misplacement of `*` in declaration of pointer to struct

Author: Huang Zhen-Hong Date: 2022-06-29T15:21:02+08:00 New Revision: b646f0955574c6ad4c156c9db522e46f597cfda9 URL: https://github.com/llvm/llvm-project/commit/b646f0955574c6ad4c156c9db522e46f597cfda9 DIFF:

[PATCH] D127873: [clang-format] Fix misplacement of `*` in declaration of pointer to struct

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGb646f0955574: [clang-format] Fix misplacement of `*` in declaration of pointer to struct (authored by Huang Zhen-Hong jackhunag1...@gmail.com,

[PATCH] D128645: Update developer policy.

vext01 added inline comments. Comment at: llvm/docs/DeveloperPolicy.rst:88 +#. Patches should be unified diffs with "infinite context" (i.e. using something + like `git diff -U99 main`). + hubert.reinterpretcast wrote: > Using `git diff` like this, there

[PATCH] D128652: [PowerPC] Finished kill_canary implementation and debugging

nemanjai added a comment. "Made a new phabricator review because of git issues" is not an appropriate description of a review/revision. Hopefully the description you add will describe what this intrinsic is supposed to do. It seems to me that this is a poorly designed feature if it is meant to

[PATCH] D127898: [clang][dataflow] Add API to separate analysis from diagnosis

sgatev accepted this revision. sgatev added inline comments. Comment at: clang/lib/Analysis/FlowSensitive/TypeErasedDataflowAnalysis.cpp:401 + const TypeErasedDataflowAnalysisState ) { +PostVisitStmt(Stmt.getStmt(), State); + });

[PATCH] D127187: [C++20] [Modules] Implement AllAdditionalTUReachable

ChuanqiXu updated this revision to Diff 440872. ChuanqiXu added a reviewer: MaskRay. ChuanqiXu added a comment. Herald added a subscriber: StephenFan. Rebasing. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127187/new/ https://reviews.llvm.org/D127187 Files:

[PATCH] D128352: [clang][dataflow] Use diagnosis API in optional checker

sgatev accepted this revision. sgatev added inline comments. Comment at: clang/include/clang/Analysis/FlowSensitive/DataflowAnalysis.h:126-129 +auto *Lattice = +llvm::any_cast(); +PostVisitStmt(Stmt, DataflowAnalysisState{ +

[PATCH] D126189: [C++20][Modules] Build module static initializers per P1874R1.

ChuanqiXu added a comment. @iains may I ask what's the issue to not land this? It looks like you're waiting for the behavior to be consistency with GCC? Since this patch could fix https://github.com/llvm/llvm-project/issues/51873, which breaks the users to compile a hello world example.

[PATCH] D126189: [C++20][Modules] Build module static initializers per P1874R1.

iains added a comment. In D126189#3617850 , @ChuanqiXu wrote: > @iains may I ask what's the issue to not land this? It looks like you're > waiting for the behavior to be consistency with GCC? > > Since this patch could fix

[clang] b405407 - [clang][flang] Disable defaulting to `-fpie` for LLVM Flang

Author: Andrzej Warzynski Date: 2022-06-29T07:53:06Z New Revision: b405407a489902c0acfcf936bfda9821a1deb170 URL: https://github.com/llvm/llvm-project/commit/b405407a489902c0acfcf936bfda9821a1deb170 DIFF: https://github.com/llvm/llvm-project/commit/b405407a489902c0acfcf936bfda9821a1deb170.diff

[PATCH] D128333: [clang][flang] Disable defaulting to `-fpie` for LLVM Flang

This revision was automatically updated to reflect the committed changes. Closed by commit rGb405407a4899: [clang][flang] Disable defaulting to `-fpie` for LLVM Flang (authored by awarzynski). Changed prior to commit: https://reviews.llvm.org/D128333?vs=440874=440884#toc Repository: rG LLVM

[PATCH] D128783: Check for more -fsanitize=array-bounds regressions

MaskRay added a comment. Do we need a C test (just add a `-x c` RUN line)? @serge-sans-paille Do you think we may likely make C++ stricter than C? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128783/new/ https://reviews.llvm.org/D128783

[PATCH] D105584: [MLIR][OpenMP] Distribute Construct Operation

abidmalikwaterloo added a comment. In D105584#3617666 , @clementval wrote: > In D105584#3617456 , > @abidmalikwaterloo wrote: > >> // CHECK-LABEL: omp_DistributeOp >> func.func @omp_DistributeOp(%lb :

[PATCH] D128612: RISC-V big-endian support implementation

gbenyei added a comment. In D128612#3617906 , @jhenderson wrote: > Objcopy aspects look good, thanks. Thanks Comment at: llvm/lib/ExecutionEngine/JITLink/ELF_riscv.cpp:554 .buildGraph(); - } else { -

[PATCH] D124748: [clang-format] Fix whitespace counting stuff

kevcadieux added a comment. @sstwcw @HazardyKnusperkeks : this change introduced a regression in one of the clang format unit tests. I fixed it in the revision below. Could anyone please take a look / approve this revision to unblock Windows debug builds? Thanks!

[PATCH] D128612: RISC-V big-endian support implementation

gbenyei added a comment. In D128612#3617955 , @MaskRay wrote: > lld/ELF change should be dropped from this change. Don't use > `config->endianness`. > I feel sad that for little-endian users who don't use big-endian, every write > now is slightly

[PATCH] D126694: [C++20][Modules] Implementation of GMF decl elision.

iains updated this revision to Diff 440915. iains marked an inline comment as done. iains added a comment. rebased after D113545 was landed and removed that as a parent. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D126694: [C++20][Modules] Implementation of GMF decl elision.

iains added inline comments. Comment at: clang/include/clang/AST/DeclBase.h:624 bool isModulePrivate() const { return getModuleOwnershipKind() == ModuleOwnershipKind::ModulePrivate; } ChuanqiXu wrote: > According to the opinion from @rsmith, the

[clang] 4ee6b78 - [test][RISCV][Driver] Precommit tests for D128625

Author: Anton Afanasyev Date: 2022-06-29T13:25:56+03:00 New Revision: 4ee6b7806bc04e3d037c0679260a54828ce7ad4c URL: https://github.com/llvm/llvm-project/commit/4ee6b7806bc04e3d037c0679260a54828ce7ad4c DIFF:

[PATCH] D126189: [C++20][Modules] Build module static initializers per P1874R1.

iains updated this revision to Diff 440924. iains added a comment. rebased, corrected some spellings. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126189/new/ https://reviews.llvm.org/D126189 Files: clang/include/clang/AST/ASTContext.h

[PATCH] D128333: [clang][flang] Disable defaulting to `-fpie` for LLVM Flang

awarzynski updated this revision to Diff 440874. awarzynski added a comment. Update the test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128333/new/ https://reviews.llvm.org/D128333 Files: clang/lib/Driver/ToolChains/Linux.cpp

[PATCH] D128612: RISC-V big-endian support implementation

jhenderson added a comment. Objcopy aspects look good, thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128612/new/ https://reviews.llvm.org/D128612 ___ cfe-commits mailing list

[PATCH] D127189: [clang][AIX] Add option to control quadword lock free atomics ABI on AIX

lkail added a comment. > Is there any precedent for options that start with -maix or -m for any > other OS? There is `-maix-struct-return`. > Is quadword the best word to use? There is no type information and this is > restricted to integers. Would something like -maix-i128-atomics be a

[PATCH] D128329: [clangd] Also mark output arguments of operator call expressions

ckandeler added a comment. In D128329#3617172 , @nridge wrote: > I'm going to take the liberty of approving this, as it seems straightforward > and unlikely to be contentious in any way. Thanks! > Please let me know if you need me to commit it. Yes,

[PATCH] D124690: [clangd] add inlay hints for std::forward-ed parameter packs

sammccall added inline comments. Comment at: clang-tools-extra/clangd/InlayHints.cpp:478 bool shouldHintReference(const ParmVarDecl *Param) { -// If the parameter is a non-const reference type, print an inlay hint +// If the parameter is of non-const l-value reference

[PATCH] D124690: [clangd] add inlay hints for std::forward-ed parameter packs

upsj updated this revision to Diff 440921. upsj marked 14 inline comments as done. upsj added a comment. - simplify parameter pack detection - improve function naming - make handling of unexpanded packs and varargs more visible - add tests involving template specializations - make documentation

[PATCH] D124690: [clangd] add inlay hints for std::forward-ed parameter packs

upsj added inline comments. Comment at: clang-tools-extra/clangd/AST.cpp:690 +getPackTemplateParameter(const FunctionDecl *Callee) { + if (const auto *TemplateDecl = Callee->getPrimaryTemplate()) { +auto TemplateParams = TemplateDecl->getTemplateParameters()->asArray();

[PATCH] D128679: [pseudo] Define a clangPseudoCLI library.

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/pseudo/benchmarks/Benchmark.cpp:51 const std::string *SourceText = nullptr; -const Grammar *G = nullptr; +const Language *PLang =

[PATCH] D122768: [Clang][C++20] Support capturing structured bindings in lambdas

cor3ntin updated this revision to Diff 440912. cor3ntin added a comment. - fix typo - add decompositions tests for the array and tuuple cases Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122768/new/ https://reviews.llvm.org/D122768 Files:

[clang] 32aac7b - [NFC] Switch FloatModeKind enum class to use bitmask enums

Author: Jolanta Jensen Date: 2022-06-29T11:02:02+01:00 New Revision: 32aac7babfdd2a7e6d40cb186d676bef93bfc6bb URL: https://github.com/llvm/llvm-project/commit/32aac7babfdd2a7e6d40cb186d676bef93bfc6bb DIFF:

[PATCH] D128182: [NFC] Switch FloatModeKind enum class to use bitmask enums

This revision was automatically updated to reflect the committed changes. Closed by commit rG32aac7babfdd: [NFC] Switch FloatModeKind enum class to use bitmask enums (authored by jolanta.jensen). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D128783: Check for more -fsanitize=array-bounds regressions

sberg created this revision. sberg added a reviewer: MaskRay. sberg added a project: clang. Herald added a subscriber: StephenFan. Herald added a project: All. sberg requested review of this revision. ...that had been introduced with (since reverted)

[PATCH] D128786: [clang-format] Fix incorrect isspace input (NFC)

kevcadieux created this revision. Herald added a project: All. kevcadieux requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This change fixes a clang-format unit test failure introduced by D124748 . The

[PATCH] D126864: [clang] Introduce -fstrict-flex-arrays= for stricter handling of flexible arrays

sberg added a comment. In D126864#3614297 , @MaskRay wrote: > Oh, I did not see this when pushing a test > efd90ffbfc427ad4c4675ac1fcae9d53cc7f1322 > . > Consider adding additional

[PATCH] D127189: [clang][AIX] Add option to control quadword lock free atomics ABI on AIX

nemanjai added a comment. I am ok with this change overall, I just have a couple of questions about naming of the option. 1. Is there any precedent for options that start with `-maix` or `-m` for any other OS? 2. Is `quadword` the best word to use? There is no type information and this is

[PATCH] D128612: RISC-V big-endian support implementation

MaskRay added a comment. lld/ELF change should be dropped from this change. Don't use `config->endianness`. I feel sad that for little-endian users who don't use big-endian, every write now is slightly slower due to a check ;-) Comment at:

[PATCH] D128795: [pseudo] Reimplement hardcoded error handling as a recovery strategy. NFC

sammccall created this revision. sammccall added a reviewer: hokein. Herald added a project: All. sammccall requested review of this revision. Herald added subscribers: cfe-commits, alextsao1999. Herald added a project: clang-tools-extra. Repository: rG LLVM Github Monorepo

[PATCH] D128487: [ODRHash diagnostics] Move repetetive code at lambda calls into lambdas themselves. NFC.

ChuanqiXu added a comment. In D128487#3617422 , @vsapsai wrote: > Thanks for the review! > > In D128487#3614251 , @ChuanqiXu > wrote: > >> Is it possible to combine the several `DiagNote` into `DiagError`? So

[PATCH] D128706: [Clang] Disable clang-format entirely for clang/test tree.

mboehme added a comment. In D128706#3615623 , @aaron.ballman wrote: > The changes here LGTM and I think this is the correct way to go. However, > since you discovered that it was perhaps intentional that we did some style > checking on tests, do you

[PATCH] D128645: Update developer policy.

vext01 updated this revision to Diff 440887. vext01 added a comment. Fixed grammar bits. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128645/new/ https://reviews.llvm.org/D128645 Files: llvm/docs/DeveloperPolicy.rst Index: llvm/docs/DeveloperPolicy.rst

[PATCH] D128786: [clang-format] Fix incorrect isspace input (NFC)

kevcadieux added a comment. FYI: failure looks like the following: [ RUN ] FormatTest.IncorrectUnbalancedBracesInMacrosWithUnicode Exception Code: 0x8003 #0 0x7ffe4253fc44 (C:\WINDOWS\SYSTEM32\ucrtbased.dll+0x7fc44) #1 0x7ffe4254a379

[PATCH] D128486: [pseudo] Add error-recovery framework & brace-based recovery

hokein added a comment. the patch looks a good start to me, some initial comments (mostly around the recovery part). Comment at: clang-tools-extra/pseudo/include/clang-pseudo/GLR.h:150 +// OldHeads is the parse state at TokenIndex. +// This function consumes consumes zero or

[PATCH] D126189: [C++20][Modules] Build module static initializers per P1874R1.

iains added a comment. In D126189#3598568 , @urnathan wrote: > please sed /initialiser/initializer/, I noticed a few had crept in. should be done now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D126907: Deferred Concept Instantiation Implementation Take 2

ChuanqiXu added a comment. In D126907#3615807 , @erichkeane wrote: > All tests pass now, I was able to get the template-template checks working > correctly, and it passes all the tests I have available. @ChuanqiXu if you > could review/run what tests

[PATCH] D124690: [clangd] add inlay hints for std::forward-ed parameter packs

sammccall accepted this revision. sammccall added a comment. Thanks for the readability improvements! I've forgotten if you have commit access? This stuff is complicated and I'm definitely going to forget the reasoning, the intuitive explanations are going to help a lot if changes are needed

[PATCH] D128679: [pseudo] Define a clangPseudoCLI library.

hokein updated this revision to Diff 440929. hokein marked an inline comment as done. hokein added a comment. address review comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128679/new/ https://reviews.llvm.org/D128679 Files:

[PATCH] D128679: [pseudo] Define a clangPseudoCLI library.

hokein added inline comments. Comment at: clang-tools-extra/pseudo/benchmarks/Benchmark.cpp:51 const std::string *SourceText = nullptr; -const Grammar *G = nullptr; +const Language *PLang = nullptr; sammccall wrote: > nit: still PLang here and in a bunch of

[PATCH] D125944: Template instantiation error recovery

Purva-Chaudhari updated this revision to Diff 440959. Purva-Chaudhari added a comment. Added new file for template test CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125944/new/ https://reviews.llvm.org/D125944 Files: clang/include/clang/Sema/Sema.h

[clang] 64ab2b1 - Improve handling of static assert messages.

Author: Corentin Jabot Date: 2022-06-29T14:57:35+02:00 New Revision: 64ab2b1dcc5136a744fcac21d3d2c59e9cce040a URL: https://github.com/llvm/llvm-project/commit/64ab2b1dcc5136a744fcac21d3d2c59e9cce040a DIFF:

[PATCH] D108469: Improve handling of static assert messages.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG64ab2b1dcc51: Improve handling of static assert messages. (authored by cor3ntin). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D128814: [Clang][Preprocessor] Fix inconsistent `FLT_EVAL_METHOD` when compiling vs preprocessing

egorzhdan created this revision. egorzhdan added a reviewer: zahiraam. Herald added a project: All. egorzhdan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. When running `clang -E -Ofast` on macOS, the `__FLT_EVAL_METHOD__` macro is `0`,

[PATCH] D125944: Template instantiation error recovery

Purva-Chaudhari updated this revision to Diff 440980. Purva-Chaudhari added a comment. File name CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125944/new/ https://reviews.llvm.org/D125944 Files: clang/include/clang/Sema/Sema.h clang/lib/Interpreter/IncrementalParser.cpp

[PATCH] D128816: [OpenMP] Add loop tripcount argument to kernel launch and remove push function

jhuber6 created this revision. jhuber6 added reviewers: jdoerfert, tianshilei1992, JonChesterfield, ABataev. Herald added subscribers: mattd, asavonic, guansong, yaxunl. Herald added a project: All. jhuber6 requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits,

[PATCH] D128783: [test] Check for more -fsanitize=array-bounds regressions

serge-sans-paille added a comment. In D128783#3617924 , @MaskRay wrote: > Do we need a C test (just add a `-x c` RUN line)? @serge-sans-paille Do you > think we may likely make C++ stricter than C? I've started a similar discussion on

[PATCH] D128747: ISSUE - incorrect -Winfinite-recursion warning on potentially-unevaluated operand #21668

appmonster007 updated this revision to Diff 440997. appmonster007 marked an inline comment as done. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128747/new/ https://reviews.llvm.org/D128747 Files: clang/docs/ReleaseNotes.rst

[PATCH] D128774: [libTooling] Add a comment about comment parsing to getAssociatedRange.

ymandel accepted this revision. ymandel added a comment. This revision is now accepted and ready to land. Thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128774/new/ https://reviews.llvm.org/D128774

[PATCH] D128766: Update references to Discourse instead of the mailing lists.

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM with some minor corrections. Thank you for this! Comment at: clang/docs/ExternalClangExamples.rst:21 -If you know of (or wrote!) a tool or project using

[PATCH] D126864: [clang] Introduce -fstrict-flex-arrays= for stricter handling of flexible arrays

serge-sans-paille updated this revision to Diff 440942. serge-sans-paille added a subscriber: chandlerc. serge-sans-paille added a comment. Code updated to take into account two situations: - size resulting from macro expansion. Previous behavior was inconsistent in that situation. I chose to

[PATCH] D128762: [Clang] Rename StringLiteral::isAscii() => isOrdinary() [NFC]

aaron.ballman added a comment. Also, precommit CI pointed out that you need to make some additional changes in clang-tools-extra to avoid build errors. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128762/new/ https://reviews.llvm.org/D128762

[PATCH] D128499: [Clang] Fix: Restore warning inadvertently removed by D126061.

miyuki added a comment. Thanks for fixing the issue. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128499/new/ https://reviews.llvm.org/D128499 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] 9d2e830 - [analyzer] Fix BindingDecl evaluation for reference types

Author: isuckatcs Date: 2022-06-29T13:01:19+02:00 New Revision: 9d2e830737bcf8035cf263e4b4cb279b7b07cf24 URL: https://github.com/llvm/llvm-project/commit/9d2e830737bcf8035cf263e4b4cb279b7b07cf24 DIFF: https://github.com/llvm/llvm-project/commit/9d2e830737bcf8035cf263e4b4cb279b7b07cf24.diff

[PATCH] D128716: [analyzer] Fix BindingDecl evaluation for reference types.

This revision was automatically updated to reflect the committed changes. Closed by commit rG9d2e830737bc: [analyzer] Fix BindingDecl evaluation for reference types (authored by isuckatcs). Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github

[PATCH] D128807: [clang][transformer] Finish plumbing `Note` all the way to the output.

courbet updated this revision to Diff 440948. courbet added a comment. Format patch Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128807/new/ https://reviews.llvm.org/D128807 Files:

[PATCH] D128706: [Clang] Disable clang-format entirely for clang/test tree.

aaron.ballman added a comment. Should we also do `llvm/test` and `clang-tools-extra/test` at the same time? The LLVM tests will show up in precommit CI, but I'm not certain that precommit CI actually tests clang-tools-extra (it also notably lacks coverage for libc++ and lldb as well).

[PATCH] D128747: ISSUE - incorrect -Winfinite-recursion warning on potentially-unevaluated operand #21668

aaron.ballman added a comment. Thank you for this fix, it's coming along well! Can you also add a release note to `clang/docs/ReleaseNotes.rst` mentioning the fix? Comment at: clang/lib/Analysis/CFG.cpp:4062 + // operand. [...] + // We add only potentially evaluated

[PATCH] D128805: [pseudo] Fix bugs/inconsistencies in forest dump.

hokein accepted this revision. hokein added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/pseudo/unittests/ForestTest.cpp:138 + + const auto *B = (symbol("B"), ruleFor("B"), {Star}); + const auto *A1 = (symbol("A"),

[PATCH] D128752: [CUDA] Stop adding CUDA features twice

This revision was automatically updated to reflect the committed changes. Closed by commit rG56ab966a04dd: [CUDA] Stop adding CUDA features twice (authored by jhuber6). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128752/new/

[clang] 56ab966 - [CUDA] Stop adding CUDA features twice

Author: Joseph Huber Date: 2022-06-29T09:34:09-04:00 New Revision: 56ab966a04dd22570fcb18276e2409c94e82c571 URL: https://github.com/llvm/llvm-project/commit/56ab966a04dd22570fcb18276e2409c94e82c571 DIFF: https://github.com/llvm/llvm-project/commit/56ab966a04dd22570fcb18276e2409c94e82c571.diff

[PATCH] D128726: [RISCV][NFC] Move static global variables into static variable in function.

jrtc27 added a comment. Seems you're right, the C++11 standard does clearly say (9.7p4): > Dynamic initialization of a block-scope variable with static storage duration > (6.6.4.1) or thread storage duration (6.6.4.2) is performed the first time > control passes through its declaration; such a

[PATCH] D128821: [clangd][ObjC] Fix ObjC method definition completion

dgoldman created this revision. dgoldman added a reviewer: sammccall. Herald added subscribers: usaxena95, kadircet, arphaman. Herald added a project: All. dgoldman requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project:

[PATCH] D128747: ISSUE - incorrect -Winfinite-recursion warning on potentially-unevaluated operand #21668

appmonster007 updated this revision to Diff 440992. appmonster007 added a comment. inserted bullet point for `-Winfinite-recursion` diagnostics fix, where diagnostic will not warn for unevaluated operands of ``typeid`` expression, under "Improvements to Clang’s diagnostics" in

[PATCH] D128805: [pseudo] Fix bugs/inconsistencies in forest dump.

sammccall created this revision. sammccall added a reviewer: hokein. Herald added a project: All. sammccall requested review of this revision. Herald added subscribers: cfe-commits, alextsao1999. Herald added a project: clang-tools-extra. - when printing a shared node for the second time, don't

[PATCH] D128774: [libTooling] Add a comment about comment parsing to getAssociatedRange.

ymandel added inline comments. Comment at: clang/include/clang/Tooling/Transformer/SourceCode.h:47 +/// range containing associated comments, you may need to invoke the tool with +/// -fparse-all-comments. CharSourceRange getAssociatedRange(const Decl , ASTContext );

[PATCH] D128812: [pseudo] prototype build syntax-tree from the parse forest.

hokein created this revision. Herald added a subscriber: mgorny. Herald added a project: All. hokein requested review of this revision. Herald added subscribers: cfe-commits, alextsao1999. Herald added projects: clang, clang-tools-extra. WARNING: this is an extremely-hacked prototype.

[PATCH] D128783: [test] Check for more -fsanitize=array-bounds regressions

serge-sans-paille added a comment. And note that with current clang, the behavior I describe here is not uniform across clang code base :-/ Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128783/new/ https://reviews.llvm.org/D128783

[clang-tools-extra] 333620d - [clangd] Support multiline semantic tokens

Author: Kadir Cetinkaya Date: 2022-06-29T13:49:03+02:00 New Revision: 333620d37a26949e9f66c823425cf9a2065e3890 URL: https://github.com/llvm/llvm-project/commit/333620d37a26949e9f66c823425cf9a2065e3890 DIFF:

[PATCH] D127856: [clangd] Support multiline semantic tokens

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG333620d37a26: [clangd] Support multiline semantic tokens (authored by kadircet). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D128807: [clang][transformer] Finish plumbing `Note` all the way to the output.

courbet created this revision. courbet added a reviewer: ymandel. Herald added a subscriber: carlosgalvezp. Herald added a project: All. courbet requested review of this revision. Herald added projects: clang, clang-tools-extra. Right now we can only add a single warning, notes are not possible.

[PATCH] D124690: [clangd] add inlay hints for std::forward-ed parameter packs

upsj marked an inline comment as done. upsj added a comment. yes, I have commit access Comment at: clang-tools-extra/clangd/InlayHints.cpp:483 + !Type.getNonReferenceType().isConstQualified() && + !isExpandedParameterPack(Param); }

[clang-tools-extra] 1ba7f52 - [pseudo] Update the cxx.bnf path in comments to reflect the new

Author: Haojian Wu Date: 2022-06-29T15:10:39+02:00 New Revision: 1ba7f5218ccdc0b4fd836cb4f0d647866f793c87 URL: https://github.com/llvm/llvm-project/commit/1ba7f5218ccdc0b4fd836cb4f0d647866f793c87 DIFF: https://github.com/llvm/llvm-project/commit/1ba7f5218ccdc0b4fd836cb4f0d647866f793c87.diff

[PATCH] D123952: [FPEnv] Allow CompoundStmt to keep FP options

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM modulo the rename request. Comment at: clang/lib/AST/Stmt.cpp:370-371 setStmts(Stmts); + if (hasStoredFPFeatures()) +

[PATCH] D128726: [RISCV][NFC] Move static global variables into static variable in function.

kito-cheng added a comment. My understanding is the reason why no global variable is because 1. the initialization order and 2. might increase the launch time of programs, moving that into function scope could resolve both issue: 1. initialized in deterministic order[1], 2. Initialized that

[PATCH] D128762: [Clang] Rename StringLiteral::isAscii() => isOrdinary() [NFC]

cor3ntin added inline comments. Comment at: clang/include/clang/AST/Expr.h:1789-1886 + enum StringKind { Ordinary, Wide, UTF8, UTF16, UTF32 }; private: unsigned numTrailingObjects(OverloadToken) const { return 1; } unsigned numTrailingObjects(OverloadToken) const {

[PATCH] D128679: [pseudo] Define a clangPseudoCLI library.

sammccall accepted this revision. sammccall added a comment. Looks good! Comment at: clang-tools-extra/pseudo/include/clang-pseudo/Language.h:21 + Grammar G; + LRTable Table; + LR might be more distinguishing (and shorter!), up to you Repository: rG LLVM

[PATCH] D128762: [Clang] Rename StringLiteral::isAscii() => isOrdinary() [NFC]

aaron.ballman added inline comments. Comment at: clang/include/clang/AST/Expr.h:1789-1886 + enum StringKind { Ordinary, Wide, UTF8, UTF16, UTF32 }; private: unsigned numTrailingObjects(OverloadToken) const { return 1; } unsigned numTrailingObjects(OverloadToken) const

[PATCH] D128814: [Clang][Preprocessor] Fix inconsistent `FLT_EVAL_METHOD` when compiling vs preprocessing

egorzhdan updated this revision to Diff 440978. egorzhdan added a comment. Adjust commit message Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128814/new/ https://reviews.llvm.org/D128814 Files: clang/lib/Lex/Preprocessor.cpp

[PATCH] D128795: [pseudo] Reimplement hardcoded error handling as a recovery strategy. NFC

hokein accepted this revision. hokein added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/pseudo/include/clang-pseudo/grammar/Grammar.h:89 // FIXME: these should be provided as extensions instead. -enum class RecoveryStrategy :

[clang] f382545 - [clang-cl] Handle some pragma alloc_text corner cases handled by MSVC

Author: Stephen Long Date: 2022-06-29T06:45:59-07:00 New Revision: f382545b2ba8a39435f7efa02dadc722c429d2cd URL: https://github.com/llvm/llvm-project/commit/f382545b2ba8a39435f7efa02dadc722c429d2cd DIFF: https://github.com/llvm/llvm-project/commit/f382545b2ba8a39435f7efa02dadc722c429d2cd.diff

[PATCH] D128649: [clang-cl] Handle some pragma alloc_text corner cases handled by MSVC

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGf382545b2ba8: [clang-cl] Handle some pragma alloc_text corner cases handled by MSVC (authored by steplong). Repository: rG LLVM Github Monorepo

[PATCH] D128103: Adds AST Matcher for ObjCStringLiteral

aaron.ballman added a comment. The only things left to do are: regenerate the documentation by running clang/docs/tools/dump_ast_matchers.py and add a release note for the new matcher, but otherwise this looks good to me. In D128103#3617156 , @NoQ

[PATCH] D128762: [Clang] Rename StringLiteral::isAscii() => isOrdinary() [NFC]

cor3ntin updated this revision to Diff 440994. cor3ntin added a comment. Herald added subscribers: carlosgalvezp, usaxena95, kadircet, arphaman. Herald added a project: clang-tools-extra. - fix clang-tidy build - apply the same change in StringLiteralParser/CharLiteralParser Repository: rG

[PATCH] D128747: ISSUE - incorrect -Winfinite-recursion warning on potentially-unevaluated operand #21668

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. Thank you for the fix, the changes LGTM! From our off-review discussion, I understand you need me to commit this on your behalf. What name and email address would you like me to

[PATCH] D128814: [Clang][Preprocessor] Fix inconsistent `FLT_EVAL_METHOD` when compiling vs preprocessing

zahiraam added a comment. Took the time to test the change on a few "crooked" tests I had used for the original patch. It works! LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128814/new/ https://reviews.llvm.org/D128814

[PATCH] D128747: ISSUE - incorrect -Winfinite-recursion warning on potentially-unevaluated operand #21668

appmonster007 updated this revision to Diff 441075. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128747/new/ https://reviews.llvm.org/D128747 Files: clang/docs/ReleaseNotes.rst clang/lib/Analysis/CFG.cpp

[PATCH] D125944: Template instantiation error recovery

Purva-Chaudhari updated this revision to Diff 441079. Purva-Chaudhari added a comment. Updating CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125944/new/ https://reviews.llvm.org/D125944 Files: clang/include/clang/Sema/Sema.h clang/lib/Interpreter/IncrementalParser.cpp Index:

[PATCH] D105584: [MLIR][OpenMP] Distribute Construct Operation

abidmalikwaterloo added a comment. In D105584#3618040 , @abidmalikwaterloo wrote: > In D105584#3617666 , @clementval > wrote: > >> In D105584#3617456 , >>

[PATCH] D127898: [clang][dataflow] Add API to separate analysis from diagnosis

samestep updated this revision to Diff 441084. samestep added a comment. - Merge branch 'main' into diagnose-api - Fix minor nits Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127898/new/ https://reviews.llvm.org/D127898 Files:

[clang] 5f2cf3a - [Clang][Preprocessor] Fix inconsistent `FLT_EVAL_METHOD` when compiling vs preprocessing

Author: Egor Zhdan Date: 2022-06-29T19:36:22+01:00 New Revision: 5f2cf3a21f3aabff85d178a110602ce150914ff7 URL: https://github.com/llvm/llvm-project/commit/5f2cf3a21f3aabff85d178a110602ce150914ff7 DIFF: https://github.com/llvm/llvm-project/commit/5f2cf3a21f3aabff85d178a110602ce150914ff7.diff

[PATCH] D128814: [Clang][Preprocessor] Fix inconsistent `FLT_EVAL_METHOD` when compiling vs preprocessing

This revision was automatically updated to reflect the committed changes. Closed by commit rG5f2cf3a21f3a: [Clang][Preprocessor] Fix inconsistent `FLT_EVAL_METHOD` when compiling vs… (authored by egorzhdan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D128826: Go-to-type on smart_ptr now also shows Foo

tom-anders added inline comments. Comment at: clang-tools-extra/clangd/HeuristicResolver.h:75 + // could look up the name appearing on the RHS. + const Type *getPointeeType(const Type *T) const; + sammccall wrote: > tom-anders wrote: > > Not sure if it's the

  1   2   3   >