[PATCH] D104475: [Clang][Codegen] emit noprofile IR Fn Attr for new Fn Attr no_profile

MaskRay added a comment. Should `no_profile` specify the inlining behavior? Though `no_sanitize_*` don't specify that, either. As I commented on https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80223 , the Linux kernel specifies `noinline` so the exact inlining behavior doesn't matter. #define n

[PATCH] D88666: DirectoryWatcher: add an implementation for Windows

compnerd added a comment. Interesting, are the logs from the runs available? I have run the test ~1 times locally and its been stable. Perhaps the logs can show what is going on. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88666/new/ https

[PATCH] D88666: DirectoryWatcher: add an implementation for Windows

stella.stamenova added a comment. In D88666#2825300 , @compnerd wrote: > Interesting, are the logs from the runs available? I have run the test > ~1 times locally and its been stable. Perhaps the logs can show what is > going on. Unfortunately, no

[PATCH] D103750: [analyzer] Handle std::make_unique for SmartPtrModeling

NoQ added a comment. In D103750#2823741 , @RedDocMD wrote: > I would suppose that constructor calls are properly handled. (ie, member > constructors are called properly). Do the above tests pass when your new `evalCall` modeling is enabled? Repository

[PATCH] D104465: [clang][deps] Prevent PCH validation failures by padding minimized files

dexonsmith added a comment. > This patch fixes dependency scanning of a TU with prebuilt modular/PCH > dependencies. > > Such modules/PCH might have been built using non-minimized files, while > dependency scanner does use the minimized versions of source files. Implicit > build doesn't like th

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

ychen updated this revision to Diff 352810. ychen added a comment. - Rebase correctly Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97915/new/ https://reviews.llvm.org/D97915 Files: clang/lib/CodeGen/CGBuiltin.cpp clang/lib/CodeGen/CGCoroutine

[PATCH] D104475: [Clang][Codegen] emit noprofile IR Fn Attr for new Fn Attr no_profile

void added inline comments. Comment at: clang/include/clang/Basic/Attr.td:1974 +def NoProfileFunction : InheritableAttr { + let Spellings = [GCC<"no_profile">]; + let Subjects = SubjectList<[Function]>; Is this an attribute that exists in GCC? I couldn't find i

[PATCH] D102875: [PowerPC] Add PowerPC compare and multiply related builtins and instrinsics for XL compatibility

lei added inline comments. Comment at: clang/test/CodeGen/builtins-ppc-xlcompat-compare-64bit-only.c:6 +// RUN: %clang_cc1 -triple powerpc64-unknown-aix \ +// RUN: -emit-llvm %s -o - -target-cpu pwr9 | FileCheck %s --check-prefix=CHECK-64 +// RUN: not %clang_cc1 -triple powerp

[PATCH] D102875: [PowerPC] Add PowerPC compare and multiply related builtins and instrinsics for XL compatibility

lei added inline comments. Comment at: clang/test/CodeGen/builtins-ppc-xlcompat-compare-64bit-only.c:8 +// RUN: not %clang_cc1 -triple powerpc-unknown-aix \ +// RUN: -emit-llvm %s -o - -target-cpu pwr9 2>&1 | FileCheck %s -check-prefix=CHECK-32 + `-check-prefi

[PATCH] D104342: [IR] convert warn-stack-size from module attr to fn attr

nickdesaulniers added a comment. In D104342#2820811 , @dblaikie wrote: > what're the rules about module level attributes like this? For instance: Does > this affect/hinder inlining (if the attributes don't match between the caller > and callee)? Other s

[PATCH] D104462: [clang][lex] Add minimizer option to pad the output to the input size

dexonsmith requested changes to this revision. dexonsmith added a comment. This revision now requires changes to proceed. I'm not sure D104465 (the motivation for this patch) is the right approach, but we can have that discussion over there.

[PATCH] D99005: [clang] Implement P2266 Simpler implicit move

mizvekov updated this revision to Diff 352818. mizvekov added a comment. - Change block capture NRVO semantics to match P2266 in C++2b mode. - Update cxx_status. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99005/n

[PATCH] D99005: [clang] Implement P2266 Simpler implicit move

mizvekov updated this revision to Diff 352821. mizvekov added a comment. . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99005/new/ https://reviews.llvm.org/D99005 Files: clang/include/clang/Sema/Sema.h clang/lib/Sema/Sema.cpp clang/lib/Sema

[PATCH] D104350: [clang][AST] Make `getLocalOrImportedSubmoduleID` work with const `Module*`. NFC.

dexonsmith accepted this revision. dexonsmith 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/D104350/new/ https://reviews.llvm.org/D104350 __

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

andrewjcg added inline comments. Comment at: clang/test/Modules/implicit-module-header-maps.cpp:27 +#define FOO +// This include will fail if modules weren't used. The include name itself +// doesn't exist and relies on the header map to remap it to the real header.

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

andrewjcg added inline comments. Comment at: clang/test/Modules/implicit-module-header-maps.cpp:27 +#define FOO +// This include will fail if modules weren't used. The include name itself +// doesn't exist and relies on the header map to remap it to the real header.

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

andrewjcg updated this revision to Diff 352824. andrewjcg added a comment. fix comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103930/new/ https://reviews.llvm.org/D103930 Files: clang/lib/Lex/HeaderSearch.cpp clang/test/Modules/Inputs/i

[PATCH] D104342: [IR] convert warn-stack-size from module attr to fn attr

nickdesaulniers added inline comments. Comment at: llvm/lib/CodeGen/PrologEpilogInserter.cpp:282 +.getValueAsString() +.getAsInteger(0, Threshold); if (StackSize > Threshold) { nickdesaulniers wrote: > dblaikie wrote: > > I guess the 0 value he

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

dexonsmith added inline comments. Comment at: clang/test/Modules/implicit-module-header-maps.cpp:27 +#define FOO +// This include will fail if modules weren't used. The include name itself +// doesn't exist and relies on the header map to remap it to the real header. ---

[PATCH] D104058: ThinLTO: Fix inline assembly references to static functions with CFI

samitolvanen updated this revision to Diff 352825. samitolvanen added a comment. Moved the test to llvm/test/Transforms/ThinLTOBitcodeWriter. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104058/new/ https://reviews.llvm.org/D104058 Files: llvm/

[PATCH] D103855: [clang] Exclude function pointers on DefaultedComparisonAnalyzer

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Thanks, let's go with this for now. Discussion in WG21 seems to be heading towards removing the builtin candidates for `operator<=>`, `operator<`, `operator<=`, `operator>`, and `operator>=` f

[PATCH] D104484: [clang] Add cc1 option for dumping layout for all complete types

daltenty created this revision. Herald added subscribers: dexonsmith, dang. daltenty requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This change adds an option which, in addition to dumping the record layout as is done by -fdump-record-layo

[PATCH] D88666: DirectoryWatcher: add an implementation for Windows

stella.stamenova added a comment. This is the best I can do from the online builds. I'll try and repro locally as well: FAIL: Clang-Unit :: DirectoryWatcher/./DirectoryWatcherTests.exe/DirectoryWatcherTest.InitialScanAsync (75980 of 75980) TEST 'Clang-Unit :: Director

[PATCH] D104484: [clang] Add cc1 option for dumping layout for all complete types

dexonsmith added inline comments. Comment at: clang/lib/AST/Decl.cpp:4585 + + ASTContext &Ctx=getASTContext(); + if (Ctx.getLangOpts().DumpRecordLayoutsComplete) { Nit: please add whitespace around `=`. (Have you run git-clang-format? That should fix this.)

[PATCH] D104424: [Sema] Fix for PR50741

abhinavgaba added a comment. Thanks for fixing this, @chrish_ericsson_atx! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104424/new/ https://reviews.llvm.org/D104424 ___ cfe-commits mailing list cfe-comm

[PATCH] D104462: [clang][lex] Add minimizer option to pad the output to the input size

arphaman added a comment. I agree with Duncan, it would be good to avoid modifying the existing test cases if possible Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104462/new/ https://reviews.llvm.org/D104462

[PATCH] D93528: [clang-format] Add basic support for formatting JSON

HazardyKnusperkeks added a reviewer: HazardyKnusperkeks. HazardyKnusperkeks accepted this revision. HazardyKnusperkeks added a comment. This revision is now accepted and ready to land. Nice trick! :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93528/new/ https://reviews.llvm.org/D9352

[PATCH] D104342: [IR] convert warn-stack-size from module attr to fn attr

dblaikie added a comment. In D104342#2825357 , @nickdesaulniers wrote: > In D104342#2820811 , @dblaikie > wrote: > >> what're the rules about module level attributes like this? For instance: >> Does this affect

[PATCH] D104351: [HeaderSearch] Use `isImport` only for imported headers and not for `#pragma once`.

vsapsai added a comment. In D104351#2824505 , @jansvoboda11 wrote: > This seems reasonable on first look. Can you add a test that demonstrates the > problem this patch solves? There is no externally observable change that can be tested, I'm only making

[PATCH] D104342: [IR] convert warn-stack-size from module attr to fn attr

nickdesaulniers updated this revision to Diff 352841. nickdesaulniers marked an inline comment as done. nickdesaulniers added a comment. - verifier checks, inliner test, use base 10 radix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104342/new/ ht

[PATCH] D104459: [clang][lex] Ensure minimizer output is never larger than input

dexonsmith added a comment. Another thing to consider: a client that wants the minimized source to be "no bigger than" the original source can use the original source if it ends up growing. For example, https://reviews.llvm.org/D104462 could check the resulting size, and just return the origina

[PATCH] D104475: [Clang][Codegen] emit noprofile IR Fn Attr for new Fn Attr no_profile

melver added a comment. In D104475#2825297 , @MaskRay wrote: > Should `no_profile` specify the inlining behavior? Though `no_sanitize_*` > don't specify that, either. I think it's somehow implicit, but I can't quite say how. There are some tests that c

[PATCH] D104475: [Clang][Codegen] emit noprofile IR Fn Attr for new Fn Attr no_profile

nickdesaulniers updated this revision to Diff 352844. nickdesaulniers marked 3 inline comments as done. nickdesaulniers added a comment. - fix td files and test CHECKs Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104475/new/ https://reviews.llvm.o

[PATCH] D104248: [compiler-rt][hwasan] Refactor Thread::Init

leonardchan added a comment. Will submit around end of day if there's no more comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104248/new/ https://reviews.llvm.org/D104248 ___ cfe-commits mailing

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

bruno added a comment. @dexonsmith and @jansvoboda11 thanks for the fast reply and the extra testing. Comment at: clang/lib/Lex/HeaderSearch.cpp:421 + auto FixupSearchPathAndFindUsableModule = + [&](auto file) -> Optional { if (SearchPath) { Can you

[PATCH] D104155: Add documentation for -fsanitize-address-use-after-return.

kda updated this revision to Diff 352849. kda marked an inline comment as done. kda added a comment. Herald added a subscriber: dang. - Responding to comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104155/new/ https://reviews.llvm.org/D1041

[PATCH] D104475: [Clang][Codegen] emit noprofile IR Fn Attr for new Fn Attr no_profile

MaskRay added a comment. In D104475#2825666 , @melver wrote: > In D104475#2825297 , @MaskRay wrote: > >> Should `no_profile` specify the inlining behavior? Though `no_sanitize_*` >> don't specify that, either. >

[PATCH] D99797: [analyzer] Implemented RangeSet::Factory::unite function to handle intersections and adjacency

ASDenysPetrov updated this revision to Diff 352851. ASDenysPetrov added a comment. Updated. Removed `F` as flag. Replaced `goto` with closure. Detailed comments and fixed typos. @vsavchenko made changes according your suggestions. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99797/new

[PATCH] D104491: [Docs][Clang][Attr] mark no_stack_protector+no_sanitize GCC compatible

nickdesaulniers created this revision. nickdesaulniers added reviewers: void, melver. Herald added a reviewer: aaron.ballman. nickdesaulniers requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. GCC has supported the function attributes __attrib

[PATCH] D99810: [ifs] Prepare llvm-ifs for elfabi/ifs merging.

haowei marked an inline comment as done. haowei added inline comments. Comment at: llvm/include/llvm/InterfaceStub/ELFObjHandler.h:25 namespace elfabi { phosek wrote: > Shouldn't this be in namespace `ifs`? This was renamed to ifs in D100139. Rename it here

[PATCH] D104475: [Clang][Codegen] emit noprofile IR Fn Attr for new Fn Attr no_profile

melver added a comment. In D104475#2825711 , @MaskRay wrote: > In D104475#2825666 , @melver wrote: > >> In D104475#2825297 , @MaskRay >> wrote: >> >>> Should `no_profile`

[PATCH] D104475: [Clang][Codegen] emit noprofile IR Fn Attr for new Fn Attr no_profile

nickdesaulniers added a comment. In D104475#2825711 , @MaskRay wrote: > So if we don't want to offer guarantee for IR/C/C++ attributes, we can > document that users may need to additionally specify > `__attribute__((noinline))` to suppress inlining. I

[PATCH] D104475: [Clang][Codegen] emit noprofile IR Fn Attr for new Fn Attr no_profile

melver added a comment. In D104475#2825772 , @nickdesaulniers wrote: > In D104475#2825711 , @MaskRay wrote: > >> So if we don't want to offer guarantee for IR/C/C++ attributes, we can >> document that users may

[PATCH] D104381: [analyzer] Added a test case for PR46264

ASDenysPetrov added a comment. @NoQ: > I guess there might be some value in building an older Clang from around when > the bug was reported so that to see if the crash was indeed there and we're > not just reproducing it wrong. It would also allow us to understand where we > fixed it through b

[PATCH] D104465: [clang][deps] Prevent PCH validation failures by padding minimized files

dexonsmith added a comment. In D104465#2825343 , @dexonsmith wrote: > A few other ideas: > > 1. Disable PCH validation when minimizing. (Is that less robust than your > current workaround? If so, can you explain why?) > 2. Use the original PCH header in

[PATCH] D104475: [Clang][Codegen] emit noprofile IR Fn Attr for new Fn Attr no_profile

MaskRay added a comment. In D104475#2825772 , @nickdesaulniers wrote: > In D104475#2825711 , @MaskRay wrote: > >> So if we don't want to offer guarantee for IR/C/C++ attributes, we can >> document that users may

[PATCH] D88666: DirectoryWatcher: add an implementation for Windows

amccarth added a comment. In D88666#2825557 , @stella.stamenova wrote: > One thing we've run into in the past is that running some of these tests **on > a drive other than the OS drive** can cause weird failure. I am not sure if > that may be the case h

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

ahatanak added a comment. I think these places have to be fixed too: https://github.com/llvm/llvm-project/blob/bf9f21a28be171dc500cc68b4cb1fcd3fc33f229/clang/lib/CodeGen/CGBuiltin.cpp#L1705 https://github.com/llvm/llvm-project/blob/bf9f21a28be171dc500cc68b4cb1fcd3fc33f229/clang/lib/CodeGen/CGNonTr

[PATCH] D103750: [analyzer] Handle std::make_unique for SmartPtrModeling

xazax.hun added a comment. I believe there are a couple of comments that are done but not marked accordingly. I agree with Artem, if we could craft code that fails due to not calling ctors, we should probably include them in the tests, even if they don't reflect the desired behavior they are a

[PATCH] D104475: [Clang][Codegen] emit noprofile IR Fn Attr for new Fn Attr no_profile

nickdesaulniers added a comment. In D104475#2825795 , @MaskRay wrote: > In D104475#2825772 , > @nickdesaulniers wrote: > >> In D104475#2825711 , @MaskRay >> wrote: >> >>

[PATCH] D104300: [analyzer] Handle std::swap for std::unique_ptr

xazax.hun added inline comments. Comment at: clang/include/clang/StaticAnalyzer/Core/BugReporter/BugReporter.h:435 + void markNotInteresting(SymbolRef sym); + Bikeshedding: I wonder if we prefer `Uninteresting` to `NotInteresting`. Or alternatively, if we wa

[PATCH] D98726: [analyzer] Enabling MallocChecker to take up after SmartPtrModelling

xazax.hun added a comment. We probably did not update this PR with the discussions we had offline. Basically we had a bug with a sink node due to calling into a destructor of `unique_ptr`. The catch is that the ctor was evalcalled, so the store did not have the correct bindings. Thus, after inl

[PATCH] D104494: [dfsan] Replace dfs$ prefix with .dfsan suffix

kcc added a comment. Yey, great idea! :) (I am not reviewing the code; but the change looks straightforward) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104494/new/ https://reviews.llvm.org/D104494 __

[PATCH] D104494: [dfsan] Replace dfs$ prefix with .dfsan suffix

stephan.yichao.zhao added inline comments. Comment at: llvm/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp:1134 + Asm.replace(Pos, 1, Suffix + "@"); +} GV->getParent()->setModuleInlineAsm(Asm); Based on http://web.mit.edu/rhel-doc/3/rhel-as-en

[PATCH] D104387: [clang-cl] Implement /external:I, /external:env, and EXTERNAL_INCLUDE support (PR36003)

thakis accepted this revision. thakis added a comment. This revision is now accepted and ready to land. That's a good argument. lgtm. Comment at: clang/lib/Driver/ToolChains/MSVC.cpp:1273 + +if (include_var) { + StringRef(*include_var) Why not keep the

[clang] 722c514 - [clang][AST] Make `getLocalOrImportedSubmoduleID` work with const `Module*`. NFC.

Author: Volodymyr Sapsai Date: 2021-06-17T17:29:12-07:00 New Revision: 722c51473c7a3bfe13d929734615a3b46d44c09a URL: https://github.com/llvm/llvm-project/commit/722c51473c7a3bfe13d929734615a3b46d44c09a DIFF: https://github.com/llvm/llvm-project/commit/722c51473c7a3bfe13d929734615a3b46d44c09a.di

[PATCH] D104350: [clang][AST] Make `getLocalOrImportedSubmoduleID` work with const `Module*`. NFC.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG722c51473c7a: [clang][AST] Make `getLocalOrImportedSubmoduleID` work with const `Module*`. (authored by vsapsai). Repository: rG LLVM Github Mono

[PATCH] D104494: [dfsan] Replace dfs$ prefix with .dfsan suffix

gbalats marked an inline comment as done. gbalats added inline comments. Comment at: llvm/lib/Transforms/Instrumentation/DataFlowSanitizer.cpp:1134 + Asm.replace(Pos, 1, Suffix + "@"); +} GV->getParent()->setModuleInlineAsm(Asm); stephan.yichao.zhao

[PATCH] D104299: Handle interactions between reserved identifier and user-defined suffixes

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Thanks, looks good. Just some minor comments. Comment at: clang/include/clang/Sema/Sema.h:4132 + bool checkLiteralOperatorId(const CXXScopeSpec &SS, const UnqualifiedId &Id,

[clang] 05d0f1a - Frontend: Respect -fno-temp-file when creating a PCH

Author: Zachary Henkel Date: 2021-06-17T18:34:10-07:00 New Revision: 05d0f1a8ea012a6b7b8ea65893ec4121106444b5 URL: https://github.com/llvm/llvm-project/commit/05d0f1a8ea012a6b7b8ea65893ec4121106444b5 DIFF: https://github.com/llvm/llvm-project/commit/05d0f1a8ea012a6b7b8ea65893ec4121106444b5.diff

[PATCH] D104500: DRAFT: [clang] Apply P1825 as DR for all modes below C++20.

mizvekov created this revision. mizvekov requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Signed-off-by: Matheus Izvekov Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D104500 Files: clang/include/clang/Sema/Sema.h c

[PATCH] D104500: DRAFT: [clang] Apply P1825 as DR for all modes below C++20.

mizvekov updated this revision to Diff 352894. mizvekov added a comment. set parent revision. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104500/new/ https://reviews.llvm.org/D104500 Files: clang/include/clang/Sema/Sema.h clang/lib/Sema/Sema

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

andrewjcg updated this revision to Diff 352898. andrewjcg added a comment. capitalize param Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103930/new/ https://reviews.llvm.org/D103930 Files: clang/lib/Lex/HeaderSearch.cpp clang/test/Modules/Inp

[PATCH] D104350: [clang][AST] Make `getLocalOrImportedSubmoduleID` work with const `Module*`. NFC.

vsapsai added a comment. Thanks for the review! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104350/new/ https://reviews.llvm.org/D104350 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://

[PATCH] D104505: [HIP] Defer operator overloading errors

yaxunl created this revision. yaxunl added reviewers: tra, rjmccall, rsmith. yaxunl requested review of this revision. nvcc does not diagnose overloading resolution diagnostics if it happens in functions not emitted, e.g., if a device function calls a host function, it is not diagnosed in host com

[clang] 6aaf4fa - Bring our handling of -Wframe-larger-than more in line with GCC.

Author: Richard Smith Date: 2021-06-17T20:29:13-07:00 New Revision: 6aaf4fa2885600b0e31042071ad06f78218ab0f2 URL: https://github.com/llvm/llvm-project/commit/6aaf4fa2885600b0e31042071ad06f78218ab0f2 DIFF: https://github.com/llvm/llvm-project/commit/6aaf4fa2885600b0e31042071ad06f78218ab0f2.diff

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

ychen updated this revision to Diff 352908. ychen marked 2 inline comments as done. ychen added a comment. - Not use `void *` in EmitBuiltinAlignTo signature. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97915/new/ https://reviews.llvm.org/D97915

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

ychen added inline comments. Comment at: clang/lib/CodeGen/CGCoroutine.cpp:437-475 +void emitDynamicAlignedDealloc(CodeGenFunction &CGF, + llvm::BasicBlock *AlignedFreeBB, + llvm::CallInst *CoroFree) { + llvm::CallInst

[PATCH] D104300: [analyzer] Handle std::swap for std::unique_ptr

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. I think this patch is good to go as long as other folks are happy. I'm glad that uninterestingness is becoming a thing! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://re

<    1   2