[PATCH] D96771: [OpenCL] Add distinct file extension for C++ for OpenCL

Anastasia updated this revision to Diff 331317. Anastasia added a comment. Back to .clcpp! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96771/new/ https://reviews.llvm.org/D96771 Files: clang/include/clang/Basic/LangStandard.h clang/include/clang/Driver/Types.def clang/lib/Drive

[PATCH] D96771: [OpenCL] Add distinct file extension for C++ for OpenCL

Anastasia added a comment. Thanks, I have changed it to '.clcpp' and also replied in RFC: https://lists.llvm.org/pipermail/cfe-dev/2021-March/067936.html Let's wait a few days for some more feedback. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96771/new/ https://reviews.llvm.org/D96

[PATCH] D93347: [Test] Fix undef var in attr-speculative-load-hardening.c

kristof.beyls accepted this revision. kristof.beyls added a comment. This revision is now accepted and ready to land. LGTM, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93347/new/ https://reviews.llvm.org/D93347 __

[PATCH] D98520: [OpenCL] Remove spurious atomic_fetch tablegen builtins

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

[PATCH] D98798: Produce waring for performing pointer arithmetic on a null pointer.

jamieschmeiser created this revision. jamieschmeiser added a reviewer: andrew.w.kaylor. jamieschmeiser requested review of this revision. Herald added a project: clang. Test and produce warning for subtracting a pointer from null or subtracting null from a pointer. Reuse existing warning that th

[PATCH] D98798: Produce waring for performing pointer arithmetic on a null pointer.

jamieschmeiser added a comment. I used formatting similar to the existing code, which is not what clang-format is expecting. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98798/new/ https://reviews.llvm.org/D98798

[PATCH] D97371: [clang][parser] Remove questionable ProhibitAttributes() call in objc parsing

rjmccall added a comment. In D97371#2596643 , @tbaeder wrote: > They are being applied to the `@try` at least (`-ast-print` prints them) and > we do some error checking for missing call expressions in > `handleNoMergeAttr()` in `SemaStmtAttr.cpp`. I don'

[PATCH] D98799: [UniqueLinkageName] Use consistent checks when mangling symbo linkage name and debug linkage name.

hoy created this revision. Herald added a subscriber: wenlei. hoy requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. C functions may be declared and defined in different prototypes like below. This patch unifies the checks for mangling names

[PATCH] D97915: [Coroutines] Handle overaligned frame allocation

lxfind added a comment. I am not sure how this would work, maybe I am missing something. But this patch tries to round up the frame pointer by looking at the difference between the alignment of new and the alignment of the frame. The alignment of new only gives you the guaranteed alignment for ne

[clang] 2426b1f - [Test] Fix undef var in attr-speculative-load-hardening.c

Author: Thomas Preud'homme Date: 2021-03-17T19:12:25Z New Revision: 2426b1fa66f95d2b6b874e422edceccdf6431162 URL: https://github.com/llvm/llvm-project/commit/2426b1fa66f95d2b6b874e422edceccdf6431162 DIFF: https://github.com/llvm/llvm-project/commit/2426b1fa66f95d2b6b874e422edceccdf6431162.diff

[PATCH] D93347: [Test] Fix undef var in attr-speculative-load-hardening.c

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2426b1fa66f9: [Test] Fix undef var in attr-speculative-load-hardening.c (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE L

[PATCH] D98494: [NFC] Minor cleanup to use default setting of getLastArg()

asl accepted this revision. asl added a comment. This revision is now accepted and ready to land. Looks trivial enough for post-commit review :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98494/new/ https://reviews.llvm.org/D98494 _

[PATCH] D96843: [Clang][RISCV] Add vsetvl and vsetvlmax.

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/D96843/new/ https://reviews.llvm.org/D96843 _

[PATCH] D98792: [ASTMatchers][NFC] Use move semantics when passing matchers around.

njames93 updated this revision to Diff 331340. njames93 added a comment. Fix PP guards erroneously using #ifdef Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98792/new/ https://reviews.llvm.org/D98792 Files: clang/include/clang/ASTMatchers/ASTMa

[PATCH] D98493: [WoA][MSVC] Use default linker setting in MSVC-compatible driver

asl added inline comments. Comment at: clang/lib/Driver/ToolChains/MSVC.cpp:582 + StringRef Linker = Args.getLastArgValue(options::OPT_fuse_ld_EQ, + CLANG_DEFAULT_LINKER); + if (Linker.empty()) How is CLANG_DEFAULT_LINKER

[PATCH] D98388: [RISCV][Clang] Add RVV vle/vse intrinsic functions.

craig.topper added inline comments. Comment at: clang/utils/TableGen/RISCVVEmitter.cpp:700 +for (auto Idx : CTypeOrder) { + if (Seen.count(Idx)) +PrintFatalError( You can use ``` if (!Seen.insert(Idx).second) PrintFatalError ``` This avoids w

[PATCH] D98493: [WoA][MSVC] Use default linker setting in MSVC-compatible driver

maxim-kuvyrkov added inline comments. Comment at: clang/lib/Driver/ToolChains/MSVC.cpp:582 + StringRef Linker = Args.getLastArgValue(options::OPT_fuse_ld_EQ, + CLANG_DEFAULT_LINKER); + if (Linker.empty()) asl wrote: > How

[PATCH] D98388: [RISCV][Clang] Add RVV vle/vse intrinsic functions.

craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM other than that one comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98388/new/ https://reviews.llvm.org/D98388

[PATCH] D98191: [flang][driver] Add support for `-fdebug-dump-symbols-sources`

kiranchandramohan accepted this revision. kiranchandramohan added inline comments. This revision is now accepted and ready to land. Comment at: flang/include/flang/Frontend/FrontendOptions.h:60 + + /// Parse, run semantics and the dump symbol sources map + GetSymbolsSources ---

[PATCH] D98493: [WoA][MSVC] Use default linker setting in MSVC-compatible driver

asl accepted this revision. asl added a comment. This revision is now accepted and ready to land. LGTM, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98493/new/ https://reviews.llvm.org/D98493 __

[PATCH] D97915: [Coroutines] Handle overaligned frame allocation

ychen added a comment. In D97915#2632493 , @lxfind wrote: > I am not sure how this would work, maybe I am missing something. > But this patch tries to round up the frame pointer by looking at the > difference between the alignment of new and the alignment

[PATCH] D20689: [clang-tidy] Add 'readability-suspicious-call-argument' check

aaron.ballman added a comment. Pinging @alexfh for opinions about the check (especially any concerns about true or false positive rates). I continue to think this is a good check that's worth moving forward on. Comment at: clang-tools-extra/clang-tidy/readability/SuspiciousC

[PATCH] D98792: [ASTMatchers][NFC] Use move semantics when passing matchers around.

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM aside from a small nit. Comment at: clang/include/clang/ASTMatchers/ASTMatchersInternal.h:1564 public: - PolymorphicMatcher(const ParamTypes &... Params)

[PATCH] D98792: [ASTMatchers][NFC] Use move semantics when passing matchers around.

njames93 added inline comments. Comment at: clang/include/clang/ASTMatchers/ASTMatchersInternal.h:1564 public: - PolymorphicMatcher(const ParamTypes &... Params) : Params(Params...) {} + PolymorphicMatcher(const ParamTypes &...Params) : Params(Params...) {}

[PATCH] D98792: [ASTMatchers][NFC] Use move semantics when passing matchers around.

njames93 updated this revision to Diff 331353. njames93 added a comment. Fix formatting Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98792/new/ https://reviews.llvm.org/D98792 Files: clang/include/clang/ASTMatchers/ASTMatchersInternal.h clang

[PATCH] D98738: [clang-tidy] performance-* checks: Match AllowedTypes against qualified type names when they contain "::".

hokein added inline comments. Comment at: clang-tools-extra/clang-tidy/utils/Matchers.h:52 AST_MATCHER_P(NamedDecl, matchesAnyListedName, std::vector, NameList) { worth some comments on this. Comment at: clang-tools-extra/clan

[PATCH] D98812: [OPENMP]Map data field with l-value reference types.

ABataev created this revision. ABataev added reviewers: jdoerfert, jyu2, mikerice. Herald added subscribers: guansong, yaxunl. ABataev requested review of this revision. Herald added subscribers: openmp-commits, sstefan1. Herald added projects: clang, OpenMP. Added initial support dfor the mapping

[PATCH] D98783: [CUDA][HIP] Remove unused addr space casts

yaxunl updated this revision to Diff 331363. yaxunl added a comment. bug fix. only remove unused casts. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98783/new/ https://reviews.llvm.org/D98783 Files: clang/lib/CodeGen/CodeGenModule.cpp clang/lib/CodeGen/CodeGenModule.h clang/test

[PATCH] D95460: [flang][driver] Add forced form flags and -ffixed-line-length

protze.joachim added a comment. Before this patch, clang would happily ignore a `-ffree-form` flag. With this patch, none of `-Wno-error=unknown-argument`, `-Wno-unknown-argument` , `-Qunused-arguments` help to avoid clang from exiting with error: unknown argument: '-ffree-form' Why can't cl

[PATCH] D96033: [clang-repl] Land initial infrastructure for incremental parsing

rjmccall added a comment. In D96033#2630978 , @v.g.vassilev wrote: >>> I'm nervous in general about the looming idea of declaration unloading, but >>> the fact that it's been working in Cling for a long time gives me some >>> confidence that we can do t

[PATCH] D98814: [CUDA][HIP] Mark device var used by host only

yaxunl created this revision. yaxunl added a reviewer: tra. yaxunl requested review of this revision. Add device variables to llvm.compiler.used if they are ODR-used by either host or device functions. This is necessary to prevent them from being eliminated by whole-program optimization where the

[PATCH] D98816: PR49585: Emit the jump destination for a for loop 'continue' from within the scope of the condition variable.

rsmith created this revision. rsmith added a reviewer: rjmccall. rsmith requested review of this revision. Herald added a project: clang. The condition variable is in scope in the loop increment, so we need to emit the jump destination from wthin the scope of the condition variable. For GCC compa

[PATCH] D98816: PR49585: Emit the jump destination for a for loop 'continue' from within the scope of the condition variable.

rsmith added a comment. Note that this fixes a miscompile where we'd destroy a for loop condition variable twice when encountering a `continue` (once before the increment and again after). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98816/new/

[clang] 48ab967 - [ASTMatchers][NFC] Use move semantics when passing matchers around.

Author: Nathan James Date: 2021-03-17T22:03:08Z New Revision: 48ab9674b21be2c6206ccc04602d4a3e4c812953 URL: https://github.com/llvm/llvm-project/commit/48ab9674b21be2c6206ccc04602d4a3e4c812953 DIFF: https://github.com/llvm/llvm-project/commit/48ab9674b21be2c6206ccc04602d4a3e4c812953.diff LOG:

[PATCH] D98792: [ASTMatchers][NFC] Use move semantics when passing matchers around.

This revision was automatically updated to reflect the committed changes. Closed by commit rG48ab9674b21b: [ASTMatchers][NFC] Use move semantics when passing matchers around. (authored by njames93). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98792

[PATCH] D74361: [Clang] Undef attribute for global variables

JonChesterfield added a comment. This appears to have missed a case for openmp. Credit to @jdoerfert for the repro: https://godbolt.org/z/xWTYbv struct DST { long X; long Y; }; #pragma omp declare target DST G2 [[clang::loader_uninitialized, clang::address_space(5)]]; #prag

[PATCH] D98824: [Tooling] Handle compilation databases with clang-cl commands generated by CMake 3.19+

jnykiel created this revision. jnykiel added a reviewer: aganea. Herald added subscribers: usaxena95, kadircet, abidh. jnykiel requested review of this revision. Herald added subscribers: cfe-commits, ilya-biryukov. Herald added a project: clang. As of CMake commit https://gitlab.kitware.com/cmake

[PATCH] D98816: PR49585: Emit the jump destination for a for loop 'continue' from within the scope of the condition variable.

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Well, that's certainly some bug. Patch LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98816/new/ https://reviews.llvm.org/D98816

[clang] c615927 - [OPENMP51]Initial support for the use clause.

Author: Mike Rice Date: 2021-03-17T15:46:14-07:00 New Revision: c615927c8e38d8608d7b5fa294a25dc44df0eb68 URL: https://github.com/llvm/llvm-project/commit/c615927c8e38d8608d7b5fa294a25dc44df0eb68 DIFF: https://github.com/llvm/llvm-project/commit/c615927c8e38d8608d7b5fa294a25dc44df0eb68.diff LOG

[PATCH] D98815: [OPENMP51]Initial support for the use clause

This revision was automatically updated to reflect the committed changes. Closed by commit rGc615927c8e38: [OPENMP51]Initial support for the use clause. (authored by mikerice). Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo CHANGES SI

[clang] a875721 - PR49585: Emit the jump destination for a for loop 'continue' from within the scope of the condition variable.

Author: Richard Smith Date: 2021-03-17T16:24:04-07:00 New Revision: a875721d8a2dacb894106a2cefa18828bf08f25d URL: https://github.com/llvm/llvm-project/commit/a875721d8a2dacb894106a2cefa18828bf08f25d DIFF: https://github.com/llvm/llvm-project/commit/a875721d8a2dacb894106a2cefa18828bf08f25d.diff

[PATCH] D98816: PR49585: Emit the jump destination for a for loop 'continue' from within the scope of the condition variable.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa875721d8a2d: PR49585: Emit the jump destination for a for loop 'continue' from within the… (authored by rsmith). Repository: rG LLVM Github Monor

[PATCH] D98824: [Tooling] Handle compilation databases with clang-cl commands generated by CMake 3.19+

aganea added inline comments. Comment at: clang/lib/Tooling/InterpolatingCompilationDatabase.cpp:181 + // ...including when the inputs are passed after -- + if (Opt.matches(OPT__DASH_DASH)) { +continue; You can leave out the braces on a one-line

[PATCH] D98827: [AST] Ensure that an empty json file is generated if compile errors

steveire created this revision. steveire added a reviewer: njames93. steveire 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/D98827 Files: clang/lib/Tooling/DumpTool/ASTSrc

[PATCH] D98775: [AST] Add introspection support for Decls

steveire updated this revision to Diff 331413. steveire added a comment. Herald added a subscriber: mgorny. Update Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98775/new/ https://reviews.llvm.org/D98775 Files: clang/include/clang/Tooling/NodeIn

[PATCH] D98783: [CUDA][HIP] Remove unused addr space casts

tra added a comment. In D98783#2632244 , @yaxunl wrote: > In D98783#2632143 , @tra wrote: > >> Shouldn't the unused ASCs be eliminated by DCE? We seem to be cleaning up >> the consequences of a problem that happe

[PATCH] D98814: [CUDA][HIP] Mark device var used by host only

tra added inline comments. Comment at: clang/lib/CodeGen/CGCUDANV.cpp:1102 + Info.D->isUsed() && !Info.D->hasAttr()) { +CGM.addCompilerUsedGlobal(Info.Var); + } Do we want to limit it further to only externally-visible variables? I think we

[clang] 3315bd0 - PR49619: Remove delayed call to noteFailed.

Author: Richard Smith Date: 2021-03-17T17:25:18-07:00 New Revision: 3315bd0beb4cf23f838bd522a1f0e3fcc0a9fae2 URL: https://github.com/llvm/llvm-project/commit/3315bd0beb4cf23f838bd522a1f0e3fcc0a9fae2 DIFF: https://github.com/llvm/llvm-project/commit/3315bd0beb4cf23f838bd522a1f0e3fcc0a9fae2.diff

[PATCH] D98458: Revert "[CodeGenModule] Set dso_local for Mach-O GlobalValue"

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd672d5219a72: Revert "[CodeGenModule] Set dso_local for Mach-O GlobalValue" (authored by arphaman). Changed prior to commit: https://reviews.llvm.

[clang] d672d52 - Revert "[CodeGenModule] Set dso_local for Mach-O GlobalValue"

Author: Alex Lorenz Date: 2021-03-17T17:27:41-07:00 New Revision: d672d5219a72d2e13dcc257116876d41955e36b2 URL: https://github.com/llvm/llvm-project/commit/d672d5219a72d2e13dcc257116876d41955e36b2 DIFF: https://github.com/llvm/llvm-project/commit/d672d5219a72d2e13dcc257116876d41955e36b2.diff L

[PATCH] D98757: [AMX] Not fold constant bitcast into amx intrisic

xiangzhangllvm added a comment. In D98757#2631042 , @lebedev.ri wrote: > The ongoing special-casing of `X86_AMXTy` through the llvm due to the > inability of the existing backend passes to handle certain llvm ir constructs. We have bring up it to llvm-de

[PATCH] D54943: WIP [clang-tidy] implement const-transformation for cppcoreguidelines-const-correctness

tiagoma added a comment. Herald added a subscriber: shchenz. Can we get this in? I work in Microsoft Office and we have been using this checker and it works great! There are a couple of issues with it and I would like to contribute fixes. Repository: rG LLVM Github Monorepo CHANGES SINCE LA

[PATCH] D96033: [clang-repl] Land initial infrastructure for incremental parsing

rjmccall added a comment. To be clear, what I'm trying to say is that — absent consensus that it's a major architecture shift is appropriate — we need to consider what functionality is reasonably achievable without it. I'm sure that covers most of what you're trying to do; it just may not incl

[PATCH] D98688: [-Wcalled-once-parameter] Harden analysis in terms of block use

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Great, thanks!! Comment at: clang/test/SemaObjC/warn-called-once.m:861 // We consider captures by blocks as escapes - [self indirect_call:(^{ + [self indirect_call:(^{ // expe

[PATCH] D97916: [Driver][RISCV] Support parsing multi-lib config from GCC.

kito-cheng updated this revision to Diff 331448. kito-cheng added a comment. ChangeLog: - Add more verbose message during reading GCC multilib configuration Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97916/new/ https://reviews.llvm.org/D97916

[clang] 95c0125 - [Clang][RISCV] Add rvv vsetvl and vsetvlmax intrinsic functions.

Author: Zakk Chen Date: 2021-03-17T20:26:06-07:00 New Revision: 95c0125f2bc610d9c51d4fbdd1144fcab40f3b51 URL: https://github.com/llvm/llvm-project/commit/95c0125f2bc610d9c51d4fbdd1144fcab40f3b51 DIFF: https://github.com/llvm/llvm-project/commit/95c0125f2bc610d9c51d4fbdd1144fcab40f3b51.diff LOG

[PATCH] D96843: [Clang][RISCV] Add vsetvl and vsetvlmax.

This revision was automatically updated to reflect the committed changes. Closed by commit rG95c0125f2bc6: [Clang][RISCV] Add rvv vsetvl and vsetvlmax intrinsic functions. (authored by khchen). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96843/new/

[PATCH] D98388: [RISCV][Clang] Add RVV vle/vse intrinsic functions.

khchen marked an inline comment as done. khchen added inline comments. Comment at: clang/utils/TableGen/RISCVVEmitter.cpp:700 +for (auto Idx : CTypeOrder) { + if (Seen.count(Idx)) +PrintFatalError( craig.topper wrote: > You can use > > ``` > if

[clang] c75b226 - [analyzer] Introduce common bug category "Unused code".

Author: Artem Dergachev Date: 2021-03-17T20:58:27-07:00 New Revision: c75b2261a0aada6bf7ddd91f91139c6f06a8e367 URL: https://github.com/llvm/llvm-project/commit/c75b2261a0aada6bf7ddd91f91139c6f06a8e367 DIFF: https://github.com/llvm/llvm-project/commit/c75b2261a0aada6bf7ddd91f91139c6f06a8e367.dif

[PATCH] D98741: [analyzer] Introduce common bug category "Unused code".

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc75b2261a0aa: [analyzer] Introduce common bug category "Unused code". (authored by dergachev.a). Herald added a project: clang. Repository: rG LLV

[PATCH] D98638: [RFC][Coroutine] Force stack allocation after await_suspend() call

bruno added a comment. Hi Xun, great to see more improvements in this area. Comment at: clang/lib/CodeGen/CGCoroutine.cpp:221 CGF.EmitBlock(RealSuspendBlock); + } else if (ForcestackStart) { +Builder.CreateCall( ChuanqiXu wrote: > lxfind wrote: > > Ch

[PATCH] D97183: [analyzer] Add NoteTag for smart-ptr get()

NoQ added a comment. By tracking the call-expression you're basically tracking the raw pointer value because that's what operators `*` and `->` return. Of course operator `*` returns an lvalue reference rather than a pointer but we don't make a difference when it comes to `SVal` representation.

[PATCH] D97183: [analyzer] Add NoteTag for smart-ptr get()

RedDocMD updated this revision to Diff 331461. RedDocMD added a comment. Added a negative test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97183/new/ https://reviews.llvm.org/D97183 Files: clang/lib/StaticAnalyzer/Checkers/SmartPtrChecker.cpp

[PATCH] D97183: [analyzer] Add NoteTag for smart-ptr get()

RedDocMD added a comment. > Does the following test work? @NoQ, it seems to be working. > So you're saying that simply by always tracking the (final) raw pointer value > and checking whether the raw value is interesting upon .get() you dodge the > communication problem entirely I would not sa

[PATCH] D97183: [analyzer] Add NoteTag for smart-ptr get()

RedDocMD updated this revision to Diff 331469. RedDocMD added a comment. Added some more positive tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97183/new/ https://reviews.llvm.org/D97183 Files: clang/lib/StaticAnalyzer/Checkers/SmartPtrChe

[PATCH] D98638: [RFC][Coroutine] Force stack allocation after await_suspend() call

junparser added a comment. In D98638#2630786 , @lxfind wrote: > Well, I guess another potential solution is to force emitting lifetime > intrinsics for this part of coroutine in the front-end. > Like this: > > diff --git a/clang/lib/CodeGen/CGDecl.cpp b

[PATCH] D98688: [-Wcalled-once-parameter] Harden analysis in terms of block use

vsavchenko added inline comments. Comment at: clang/test/SemaObjC/warn-called-once.m:861 // We consider captures by blocks as escapes - [self indirect_call:(^{ + [self indirect_call:(^{ // expected-note{{previous call is here}} callback(); NoQ wro

[PATCH] D97265: [clang] Allow clang-check to customize analyzer output file or dir name

OikawaKirie added a comment. @sammccall ping. Updated as you mentioned in D97265#2581653 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97265/new/ https://reviews.llvm.org/D97265 ___ cfe-commits mail

<    1   2