[PATCH] D73120: [Clang] Alternate fix for "expansion of response files in -Wp after integrated-cc1 change"

aganea updated this revision to Diff 239420. aganea added a comment. Remove `Reenter` flag as requested by @hans here . CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73120/new/ https://reviews.llvm.org/D73120 Files: clang/include/clang/Driver

[PATCH] D72996: [Sema] Attempt to perform call-size-specific `__attribute__((alloc_align(param_idx)))` validation

lebedev.ri marked an inline comment as done. lebedev.ri added inline comments. Comment at: clang/lib/Sema/SemaChecking.cpp:4479 +const Expr *Arg = Args[AA->getParamIndex().getASTIndex()]; +if (!Arg->isTypeDependent() && !Arg->isValueDependent()) { + llvm::APSInt I(64

[PATCH] D72998: [IR] Attribute/AttrBuilder: use Value::MaximumAlignment magic constant

lebedev.ri updated this revision to Diff 239421. lebedev.ri added a comment. Add constants to `Sema` class. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72998/new/ https://reviews.llvm.org/D72998 Files: clang/include/clang/Sema/Sema.h clang/l

[PATCH] D72811: [WIP][OPENMP5.0] allow lvalue for motion clause

ABataev added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:15442-15443 + CurComponents.emplace_back(CurE, nullptr); +} else if (auto *CurE = dyn_cast(E)) { + E = CurE->getLHS()->IgnoreParenImpCasts(); } else { cchen wrote: > cchen

[PATCH] D73138: [libcxx] [test] Correct asserted type in subspan test; subspan with count should never produce dynamic_extent

BillyONeal created this revision. BillyONeal added reviewers: ldionne, EricWF, mclow.lists. Herald added subscribers: dexonsmith, christof. I'm not sure if this was a change in the standard behavior or if it was a typo in libcxx from the beginning. http://eel.is/c++draft/span.sub#10 says that if

[PATCH] D72717: [CMake] Disable libc++ filesystem tests in CrossWinToARMLinux cache file

vvereschaka added a comment. > You mean, instead of ninja check-cxx just invoke LIT directly and pass this > argument? No, a little bit different. Probably we need something like 'LIBCXX_LLVM_LIT_ARGS` and pass the LIT arguments into the libc++ tests through that kind of parameter (and most li

[PATCH] D61365: [libcxx] [test] Suppress float->int narrowing warning in vector range-construction test.

BillyONeal abandoned this revision. BillyONeal added a comment. It looks like someone already fixed this. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61365/new/ https://reviews.llvm.org/D61365 ___ cfe-commits mailing list cfe-commits@list

[PATCH] D72747: [objc_direct] Allow for direct messages be sent to `self` when it is a Class

arphaman added a comment. Please add a test that exercises path without ARC enabled. Comment at: clang/lib/Sema/SemaExprObjC.cpp:3019 +if (ReceiverType->isObjCClassType() && !isImplicit && +(!Receiver->isObjCSelfExpr() || !getLangOpts().ObjCAutoRefCount)) { D

[PATCH] D71913: [LTO/WPD] Enable aggressive WPD under LTO option

tejohnson marked 5 inline comments as done. tejohnson added inline comments. Comment at: clang/test/CodeGenCXX/lto-visibility-inference.cpp:73 c1->f(); - // ITANIUM-NOT: type.test{{.*}}!"_ZTS2C2" + // ITANIUM: type.test{{.*}}!"_ZTS2C2" // MS: type.test{{.*}}!"?AUC2@@" ---

[PATCH] D71913: [LTO/WPD] Enable aggressive WPD under LTO option

tejohnson updated this revision to Diff 239428. tejohnson marked 2 inline comments as done. tejohnson added a comment. Address comments and rebase. Also apply modified change to ItaniumCXXABI.cpp and a change to an associated test (cfi-mfcall.cpp) here as discussed in child revision D71907

[PATCH] D71566: New checks for fortified sprintf

aaron.ballman added a comment. (There are still some minor whitespace nits to resolve as well.) Comment at: clang/lib/Sema/SemaChecking.cpp:743 + } + assert(UsedSize && "Found a size estimate"); + if (UsedSize.getValue().ule(ObjectSize)) I think this asserti

[clang] 133a7e6 - [PATCH] Reland [Clang] Un-break scan-build after integrated-cc1 change

Author: Alexandre Ganea Date: 2020-01-21T17:03:00-05:00 New Revision: 133a7e631cee97965e310f0d110739217427fd3d URL: https://github.com/llvm/llvm-project/commit/133a7e631cee97965e310f0d110739217427fd3d DIFF: https://github.com/llvm/llvm-project/commit/133a7e631cee97965e310f0d110739217427fd3d.dif

[PATCH] D73140: [clangd] Add C++20 concepts support to TargetFinder

nridge created this revision. Herald added subscribers: cfe-commits, usaxena95, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73140 Files: clang-tools-extra/clangd/FindTarget.cpp clang-tool

[PATCH] D72434: Support offset of member designator with the arrow for __builtin_offsetof

efriedma added a comment. Testcase? I assume we give appropriate errors if you try to use "->" on a member that's not an array? Comment at: clang/lib/Parse/ParseExpr.cpp:2168 -// FIXME: This loop leaks the index expressions on error. while (1) { Di

[PATCH] D73098: [clang-tidy] readability-identifier-naming disregards parameters restrictions on main like functions

aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp:474 if (const auto *Decl = dyn_cast(D)) { +if (isParamInMainLikeFunction(*Decl)) + return SK_Invalid; I think a better approach may be

[PATCH] D73007: [Sema] Avoid Wrange-loop-analysis false positives

aaron.ballman added inline comments. Comment at: clang/lib/Sema/SemaStmt.cpp:2703-2704 // suggest the const reference type that would do so. // For instance, given "for (const &Foo : Range)", suggest // "for (const Foo : Range)" to denote a copy is made for the loop. If // p

[PATCH] D72982: [Clang] Un-break scan-build after integrated-cc1 change

aganea added a comment. In D72982#1832233 , @aganea wrote: > In D72982#1832209 , @phosek wrote: > > > We're seeing `Driver/cc-print-options.c` test failures after this change: > > > Yes, seeing it too, I will commit

[PATCH] D73140: [clangd] Add C++20 concepts support to TargetFinder

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62082 tests passed, 0 failed and 784 were skipped. {icon question-circle color=gray} clang-tidy: unknown. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D73098: [clang-tidy] readability-identifier-naming disregards parameters restrictions on main like functions

njames93 marked an inline comment as done. njames93 added a comment. In D73098#1832131 , @Mordante wrote: > Would it make sense to also allow wmain with wchar_t? > https://docs.microsoft.com/en-us/cpp/cpp/main-function-command-line-args?view=vs-2019 Doe

[PATCH] D71903: [Coroutines][6/6] Clang schedules new passes

modocache updated this revision to Diff 239437. modocache added a comment. Rebase past D72547 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71903/new/ https://reviews.llvm.org/D71903 Files: clang/lib/CodeGen/Ba

[PATCH] D73005: [Codegen] If reasonable, materialize clang's `AssumeAlignedAttr` as llvm's Alignment Attribute on call-site function return value

jdoerfert added inline comments. Comment at: clang/test/CodeGen/assume-aligned-and-alloc-align-attributes.c:12 // CHECK-NEXT:[[MASKCOND:%.*]] = icmp eq i64 [[MASKEDPTR]], 0 // CHECK-NEXT:call void @llvm.assume(i1 [[MASKCOND]]) // CHECK-NEXT:ret i8* [[CALL]] ---

[PATCH] D73005: [Codegen] If reasonable, materialize clang's `AssumeAlignedAttr` as llvm's Alignment Attribute on call-site function return value

lebedev.ri marked 2 inline comments as done. lebedev.ri added inline comments. Comment at: clang/test/CodeGen/assume-aligned-and-alloc-align-attributes.c:12 // CHECK-NEXT:[[MASKCOND:%.*]] = icmp eq i64 [[MASKEDPTR]], 0 // CHECK-NEXT:call void @llvm.assume(i1 [[MASKCOND]

[PATCH] D69868: Allow "callbr" to return non-void values

void added a comment. @jyknight Do you think you'll have time to review this patch this week? I'd like to get it into the 10.0 release if possible. :-) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69868/new/ https://reviews.llvm.org/D69868 ___

[PATCH] D72996: [Sema] Attempt to perform call-size-specific `__attribute__((alloc_align(param_idx)))` validation

jdoerfert added inline comments. Comment at: clang/lib/Sema/SemaChecking.cpp:4489 +// Alignment calculations can wrap around if it's greater than 2**29. +unsigned MaximumAlignment = 536870912; +if (I > MaximumAlignment) erichkeane wrote: >

[PATCH] D69825: [Clang][Driver] Re-use the calling process instead of creating a new process for the cc1 invocation

hctim added a comment. In D69825#1831691 , @plotfi wrote: > In the meantime I could -fno-integrated-cc1 the driver.c InterfaceStubs > tests to quite the bots if that is what people want (and file a bug on > bugzilla). Yes, please. A temporary workarou

[PATCH] D72901: [OpenMP] [DOCS] Update OMP5.0 feature status table [NFC]

jdoerfert added a comment. @kkwli0 I propose you can merge the parts where discussion has reached a consensus while the other parts are resolved. I'm generally fine with this, we can always improve on it further. Comment at: clang/docs/OpenMPSupport.rst:194 +---

[clang] 6e73fee - List implicit operator== after implicit destructors in a vtable.

Author: Richard Smith Date: 2020-01-21T15:54:40-08:00 New Revision: 6e73fee780839bfa95aff492864e93e79910380b URL: https://github.com/llvm/llvm-project/commit/6e73fee780839bfa95aff492864e93e79910380b DIFF: https://github.com/llvm/llvm-project/commit/6e73fee780839bfa95aff492864e93e79910380b.diff

[PATCH] D72897: List implicit operator== after implicit destructors in a vtable.

rsmith added a comment. In D72897#1832234 , @vvereschaka wrote: > Hi Richard, > > http://lab.llvm.org:8011/builders/llvm-clang-win-x-armv7l builder has been > broken by this patch during few days (failed "Clang::virtual-compare.cpp" > test). > Sorry, bu

[PATCH] D72996: [Sema] Attempt to perform call-size-specific `__attribute__((alloc_align(param_idx)))` validation

hfinkel added inline comments. Comment at: clang/lib/Sema/SemaChecking.cpp:4489 +// Alignment calculations can wrap around if it's greater than 2**29. +unsigned MaximumAlignment = 536870912; +if (I > MaximumAlignment) jdoerfert wrote: > er

[PATCH] D72703: Add a warning, flags and pragmas to limit the number of pre-processor tokens in a translation unit

rnk added a comment. I waited to see if there was any other feedback, but I'm in favor of this. Should we try to come up with better pragma names? `clang max_tokens` doesn't seem to call to mind what it does: warn if there have been more than this many tokens so far in the translation unit. `ma

[PATCH] D72552: [Concepts] Constraint Satisfaction Caching

david_stone added a comment. In D72552#1832293 , @rsmith wrote: > Mechanically, this looks fine. > > There's an ongoing discussion in the committee as to whether this kind of > caching is permissible. But if this is necessary for acceptable performance,

[clang-tools-extra] 5fdad8e - [clang-tidy] Fix check for generic lambda invented template parameters

Author: Saar Raz Date: 2020-01-22T02:46:39+02:00 New Revision: 5fdad8e3f803adce501ca25118f325184e54018d URL: https://github.com/llvm/llvm-project/commit/5fdad8e3f803adce501ca25118f325184e54018d DIFF: https://github.com/llvm/llvm-project/commit/5fdad8e3f803adce501ca25118f325184e54018d.diff LOG:

[PATCH] D70926: [clang-format] Add option for not breaking line before ObjC params

ghvg1313 updated this revision to Diff 239458. ghvg1313 added a comment. - rebase master - formatting code Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70926/new/ https://reviews.llvm.org/D70926 Files: clang/docs/ClangFormatStyleOptions.rst c

[clang] a156da5 - [clang/Darwin] Remove __llvm_profile_counter_bias from export list for profiling

Author: Vedant Kumar Date: 2020-01-21T16:52:01-08:00 New Revision: a156da5fb361fd38ba379ec20856626c9e35f829 URL: https://github.com/llvm/llvm-project/commit/a156da5fb361fd38ba379ec20856626c9e35f829 DIFF: https://github.com/llvm/llvm-project/commit/a156da5fb361fd38ba379ec20856626c9e35f829.diff

[PATCH] D69868: Allow "callbr" to return non-void values

MaskRay added a comment. In D69868#1832559 , @void wrote: > @jyknight Do you think you'll have time to review this patch this week? I'd > like to get it into the 10.0 release if possible. :-) I volunteer as a reviewer:) Repository: rG LLVM Github Mo

[clang] b933d37 - [Concepts] Constraint Satisfaction Caching

Author: Saar Raz Date: 2020-01-22T03:09:53+02:00 New Revision: b933d37cd3774e5431b35e82187eebb59b1ff59e URL: https://github.com/llvm/llvm-project/commit/b933d37cd3774e5431b35e82187eebb59b1ff59e DIFF: https://github.com/llvm/llvm-project/commit/b933d37cd3774e5431b35e82187eebb59b1ff59e.diff LOG:

[PATCH] D69933: [ASTImporter] Limit imports of structs

clayborg added a comment. Clang AST contexts know how to complete types and is done via the external AST source code that will ask a type to complete itself. Each object file has an AST that knows how to lazily complete a type when and only when it is needed. Each object file also only knows ab

[PATCH] D72552: [Concepts] Constraint Satisfaction Caching

This revision was automatically updated to reflect the committed changes. Closed by commit rGb933d37cd377: [Concepts] Constraint Satisfaction Caching (authored by saar.raz). Changed prior to commit: https://reviews.llvm.org/D72552?vs=237474&id=239464#toc Repository: rG LLVM Github Monorepo

[PATCH] D73150: [Concepts] Remove -fconcepts-ts, enable concepts support under -std=c++2a

saar.raz created this revision. saar.raz added a reviewer: rsmith. Herald added a project: clang. Herald added a subscriber: cfe-commits. Enable concepts support under normal -std=c++2a and not under -fconcepts-ts, which is now deprecated. Repository: rG LLVM Github Monorepo https://reviews.

[PATCH] D72996: [Sema] Attempt to perform call-size-specific `__attribute__((alloc_align(param_idx)))` validation

jdoerfert added inline comments. Comment at: clang/lib/Sema/SemaChecking.cpp:4489 +// Alignment calculations can wrap around if it's greater than 2**29. +unsigned MaximumAlignment = 536870912; +if (I > MaximumAlignment) hfinkel wrote: > jd

[PATCH] D73151: [analyzer] Fix handle leak false positive when the handle dies too early

xazax.hun created this revision. xazax.hun added reviewers: NoQ, haowei. xazax.hun added a project: clang. Herald added subscribers: Charusso, gamesh411, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware. In case the handle symbol dies too early, even

[PATCH] D73153: [Concepts] Update ReleaseNotes with Concepts support

saar.raz created this revision. saar.raz added a reviewer: rsmith. Herald added a project: clang. Herald added a subscriber: cfe-commits. Update Clang 10 release notes with news of Concepts support. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73153 Files: clang/docs/Relea

[PATCH] D73151: [analyzer] Fix handle leak false positive when the handle dies too early

xazax.hun updated this revision to Diff 239469. xazax.hun added a comment. - Minor refactoring. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73151/new/ https://reviews.llvm.org/D73151 Files: clang/lib/StaticAnalyzer/Checkers/FuchsiaHandleChecker.cpp clang/test/Analysis/fuchsia_han

[PATCH] D71301: [clang][IFS] Prevent Clang-IFS from Leaking symbols from inside a block.

plotfi added a comment. So, the culprit appears to be D69825 . The way InterfaceStubs assembles the pipeline appears to trigger an asan bug. For now I will alter the tests to get the bots green, but once I get my bugzilla access sorted I intend to file a bug. -P

[PATCH] D72552: [Concepts] Constraint Satisfaction Caching

rtrieu added inline comments. Comment at: clang/lib/AST/ASTConcept.cpp:17 #include "clang/AST/ASTContext.h" +#include "clang/Sema/SemaConcept.h" using namespace clang; This causes a circular dependency between AST and Sema. It looks like you are including thi

[PATCH] D72972: [WebAssembly] Add experimental multivalue calling ABI

aheejin added inline comments. Comment at: clang/lib/CodeGen/TargetInfo.cpp:741 + explicit WebAssemblyABIInfo(CodeGen::CodeGenTypes &CGT, ABIKind _Kind) + : SwiftABIInfo(CGT), defaultInfo(CGT), Kind(_Kind) {} Nit: Do we need `_`? Commen

[PATCH] D73150: [Concepts] Remove -fconcepts-ts, enable concepts support under -std=c++2a

saar.raz updated this revision to Diff 239472. saar.raz added a comment. Readd -fconcepts-ts for deprecation diagnostic Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73150/new/ https://reviews.llvm.org/D73150 Files: clang/include/clang/Basic/Dia

[clang] bb9b964 - [Concepts] Fix circular AST->Sema dependency in ASTConcept.cpp

Author: Saar Raz Date: 2020-01-22T04:08:59+02:00 New Revision: bb9b964072eb42a09e76fe148b36eefcfff077b2 URL: https://github.com/llvm/llvm-project/commit/bb9b964072eb42a09e76fe148b36eefcfff077b2 DIFF: https://github.com/llvm/llvm-project/commit/bb9b964072eb42a09e76fe148b36eefcfff077b2.diff LOG:

[PATCH] D71301: [clang][IFS] Prevent Clang-IFS from Leaking symbols from inside a block.

thegameg added a comment. In D71301#1832801 , @plotfi wrote: > So, the culprit appears to be D69825 . The > way InterfaceStubs assembles the pipeline appears to trigger an asan bug. For > now I will alter the tests to get

[PATCH] D72100: Allow matching "any file" in `VerifyDiagnosticConsumer`.

jkorous added inline comments. Comment at: clang/test/Frontend/verify-any-file.c:1 +// RUN: %clang_cc1 -verify %s 2>&1 + I feel that we should test the output with FileCheck to make it more robust. Seems like other tests for `-verify` do that. For example here: h

[PATCH] D73155: [Concepts] Implement P1616R1 - Using unconstrained template template parameters with constrained templates

saar.raz created this revision. saar.raz added a reviewer: rsmith. Herald added a project: clang. Herald added a subscriber: cfe-commits. Allow unconstrained template template parameters to accept constrainted templates as arguments. Repository: rG LLVM Github Monorepo https://reviews.llvm.o

[PATCH] D72996: [Sema] Attempt to perform call-size-specific `__attribute__((alloc_align(param_idx)))` validation

hfinkel added inline comments. Comment at: clang/lib/Sema/SemaChecking.cpp:4489 +// Alignment calculations can wrap around if it's greater than 2**29. +unsigned MaximumAlignment = 536870912; +if (I > MaximumAlignment) jdoerfert wrote: > hf

[PATCH] D72552: [Concepts] Constraint Satisfaction Caching

JDevlieghere added a comment. It looks like the concept changes broke the debugger (quite spectacularly actually): http://lab.llvm.org:8011/builders/lldb-x86_64-debian/builds/3356 http://lab.llvm.org:8011/builders/lldb-x64-windows-ninja/builds/12872 http://green.lab.llvm.org/green/view/LLDB/job/

[PATCH] D72552: [Concepts] Constraint Satisfaction Caching

JDevlieghere added a comment. In D72552#1832875 , @JDevlieghere wrote: > It looks like the concept changes broke the debugger (quite spectacularly > actually): > > http://lab.llvm.org:8011/builders/lldb-x86_64-debian/builds/3356 > http://lab.llvm.org:80

[PATCH] D73156: [clang] Build clang-shlib on mingw

vchuravy created this revision. Herald added subscribers: cfe-commits, mstorsjo, mgorny. Herald added a project: clang. vchuravy added a reviewer: tstellar. vchuravy added a comment. Some additional context. We [Julia] is cross-compiling LLVM from Linux to mingw32 as part of an automatic buildsys

[PATCH] D65042: [Concept] Placeholder constraints and abbreviated templates

JDevlieghere added a comment. Hey Saar, I have temporarily reverted this because it broke the LLDB bots. Please run the LLDB test suite when you make changes to the AST importer & keep an eye on the bots when you re-land this. Thanks! commit 62e4b501ab3bc4c5815a179fdd2c4b49574506c1 (HEAD -> m

[PATCH] D72747: [objc_direct] Allow for direct messages be sent to `self` when it is a Class

MadCoder updated this revision to Diff 239477. MadCoder added a comment. fixed @ahatanak feedback CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72747/new/ https://reviews.llvm.org/D72747 Files: clang/lib/Sema/SemaExprObjC.cpp clang/test/SemaObjC/method-direct-arc.m Index: clang/t

[PATCH] D73156: [clang] Build clang-shlib on mingw

vchuravy added a comment. Some additional context. We [Julia] is cross-compiling LLVM from Linux to mingw32 as part of an automatic buildsystem for our binary dependencies. The corresponding PR is https://github.com/JuliaPackaging/Yggdrasil/pull/417 Repository: rG LLVM Github Monorepo CHANG

[PATCH] D73156: [clang] Build clang-shlib on mingw

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62088 tests passed, 0 failed and 784 were skipped. {icon question-circle color=gray} clang-tidy: unknown. {icon check-circle color=green} clang-format: pass. Build artifacts

[clang] c38e425 - [clang][IFS][test] Temporary work around for in-process cc1 ASAN issues.

Author: Puyan Lotfi Date: 2020-01-21T22:56:20-05:00 New Revision: c38e42527b21acee8d01a016d5bfa2fb83202e29 URL: https://github.com/llvm/llvm-project/commit/c38e42527b21acee8d01a016d5bfa2fb83202e29 DIFF: https://github.com/llvm/llvm-project/commit/c38e42527b21acee8d01a016d5bfa2fb83202e29.diff L

[PATCH] D68520: [cmake] Fix clang builds with BUILD_SHARED=ON and CLANG_LINK_CLANG_DYLIB=ON

tstellar added a comment. Ping. This bug is preventing upstream projects from using libclang-cpp.so, since building with -DENABLE_SHARED=ON is still common. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68520/new/ https://reviews.llvm.org/D68520

[PATCH] D68520: [cmake] Fix clang builds with BUILD_SHARED=ON and CLANG_LINK_CLANG_DYLIB=ON

mgorny accepted this revision. mgorny added a comment. This revision is now accepted and ready to land. Well, it looks correct to me. Not sure if it's the best approach but it can be refined in the future. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.o

[clang] b54aa05 - Update clang-interpreter example to incorporate changes in ce2207abaf9.

Author: Lang Hames Date: 2020-01-21T22:00:50-08:00 New Revision: b54aa053d3aeeab0cdaecb6286419138b7da5ef4 URL: https://github.com/llvm/llvm-project/commit/b54aa053d3aeeab0cdaecb6286419138b7da5ef4 DIFF: https://github.com/llvm/llvm-project/commit/b54aa053d3aeeab0cdaecb6286419138b7da5ef4.diff LO

[PATCH] D73090: [clang-tidy] Fix PR#44528 'modernize-use-using and enums'

f00kat updated this revision to Diff 239491. f00kat added a comment. Add new line in the end of file checkers/modernize-use-using.cpp CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73090/new/ https://reviews.llvm.org/D73090 Files: clang-tools-extra/clang-tidy/modernize/UseUsingCheck.c

[PATCH] D72703: Add a warning, flags and pragmas to limit the number of pre-processor tokens in a translation unit

kimgr added a comment. I just want to say that finding the correlation between token count and compile time is a bit of a breakthrough! Could you expose a flag for printing token count so users can run their own analysis? Or does that already exist in baseline clang? It's easier to set a maximu

[PATCH] D72829: Implement -fsemantic-interposition

MaskRay added a comment. In D72829#1831017 , @serge-sans-paille wrote: > > Linkages which were not interposable before and can be interposable now: > > available_externally, linkonce_odr, weak_odr, external, and appending. > > @MaskRay I understand the m

[PATCH] D72841: [RFC] Add support for pragma float_control, to control precision and exception behavior at the source level

sepavloff added a comment. I don't see tests for correctness of the pragma stack (`pragma float_control(... push)`, `pragma float_control(pop)`). Can you add them? Comment at: clang/lib/Parse/ParsePragma.cpp:2537 +if (!Actions.CurContext->isTranslationUnit()) { +//FIXME th

<    1   2   3