[PATCH] D72547: [llvm] Make new pass manager's OptimizationLevel a class

tejohnson accepted this revision. tejohnson added a comment. This revision is now accepted and ready to land. LGTM One thing to consider changing/removing in the summary is this comment: "For example, (enum) "Level > 1" captures not only O2 and O3

[PATCH] D72820: Add pragma FP_CONTRACT support.

pengfei updated this revision to Diff 238417. pengfei marked an inline comment as done. pengfei added a comment. Address review comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72820/new/ https://reviews.llvm.org/D72820 Files: clang/lib/Co

[PATCH] D72824: [X86] Add combination for fma and fneg on X86 under strict FP.

pengfei created this revision. pengfei added reviewers: craig.topper, andrew.w.kaylor, uweigand, RKSimon, LiuChen3. pengfei added a project: LLVM. X86 has instructions to calculate fma and fneg at the same time. But we combine the fneg and fma only when fneg is the source operand under strict FP

[PATCH] D72820: Add pragma FP_CONTRACT support.

craig.topper added inline comments. Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:7061 + auto pushOutChain = [this](auto &Result, auto EB) { +assert(Result.getNode()->getNumValues() == 2); Why is Result a reference? It's not modified is

[PATCH] D72820: Add pragma FP_CONTRACT support.

pengfei updated this revision to Diff 238414. pengfei marked an inline comment as done. pengfei added a comment. Address review comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72820/new/ https://reviews.llvm.org/D72820 Files: clang/lib/C

[PATCH] D50360: [Concepts] Requires Expressions

saar.raz marked an inline comment as done. saar.raz added a comment. Addressed comments in latest diff. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D50360/new/ https://reviews.llvm.org/D50360 ___ cfe-

[PATCH] D72820: Add pragma FP_CONTRACT support.

craig.topper added inline comments. Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:7064 + + auto pushOutChain = [&]() { +assert(Result.getNode()->getNumValues() == 2); Can you make the SDValue Result an argument of this and only capture 't

[PATCH] D72820: Add pragma FP_CONTRACT support.

craig.topper added a comment. The title of this review is misleading. It should at least mention FPEnv, constrained intrinsics, or strict fp or something. Right now it sounds like FP_CONTRACT isn't supported at all. Can we split most of the X86 changes into a separate patch? Most of it can be

[PATCH] D52193: RFC: [clang] Multithreaded compilation support -- NOT FOR SUBMIT

aganea added a comment. Hi @tycho ! Sorry for not getting back earlier. I implemented an alternate approach last year, which proved to be better (in terms of build times) than what I proposed in this demo patch. That is, using a thread pool instead of the process pool as implemented here. This m

[PATCH] D72820: Add pragma FP_CONTRACT support.

pengfei updated this revision to Diff 238408. pengfei added a comment. Remove dead code. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72820/new/ https://reviews.llvm.org/D72820 Files: clang/lib/CodeGen/CGExprScalar.cpp clang/test/CodeGen/cons

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

rsmith added a comment. I think this might work out more cleanly if we represented a constrained auto type as a `ConstrainedType` node wrapping an `AutoType` node rather than putting both things into the same object. (This will become more pressing if/when C++ starts allowing, for example, cons

[PATCH] D72820: Add pragma FP_CONTRACT support.

pengfei created this revision. pengfei added reviewers: craig.topper, andrew.w.kaylor, uweigand, RKSimon, LiuChen3. Herald added subscribers: llvm-commits, cfe-commits, jdoerfert, hiraditya. Herald added projects: clang, LLVM. Support pragma FP_CONTRACT. Repository: rG LLVM Github Monorepo h

Re: [clang] 45d7080 - PR42694 Support explicit(bool) in older language modes as an extension.

Hans, could this change be ported to the Clang 10 branch? In PR42694 the MSVC stdlib developers requested that Clang support this because their standard library will soon rely on it. On Wed, 15 Jan 2020 at 18:50, Richard Smith via cfe-commits < cfe-commits@lists.llvm.org> wrote: > > Author: Richa

[clang] 45d7080 - PR42694 Support explicit(bool) in older language modes as an extension.

Author: Richard Smith Date: 2020-01-15T18:38:23-08:00 New Revision: 45d70806f4386adfb62b0d75949a8aad58e0576f URL: https://github.com/llvm/llvm-project/commit/45d70806f4386adfb62b0d75949a8aad58e0576f DIFF: https://github.com/llvm/llvm-project/commit/45d70806f4386adfb62b0d75949a8aad58e0576f.diff

[PATCH] D52193: RFC: [clang] Multithreaded compilation support -- NOT FOR SUBMIT

tycho added a comment. I did some local work to make this build and pass almost all tests on Linux as well, not to make use of the multi-process features but just to avoid having a separate "for Windows only" branch, and to ensure tests pass across platforms. Unfortunately my change is not at a

[PATCH] D72675: Fix -ffast-math/-ffp-contract interaction

mcberg2017 added a comment. We crossed that bridge internally at Apple a while ago, meaning I have some code debt for cleaning up open source for fma formation that uses contract and reassoc differently than we do today, both together and separately, case by case. CHANGES SINCE LAST ACTION

[PATCH] D72675: Fix -ffast-math/-ffp-contract interaction

wristow marked an inline comment as done. wristow added inline comments. Comment at: llvm/test/CodeGen/PowerPC/fmf-propagation.ll:201-203 ; fma(X, 7.0, X * 42.0) --> X * 49.0 -; This is the minimum FMF needed for this transform - the FMA allows reassociation. +; This is the min

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

void marked an inline comment as done. void added inline comments. Comment at: llvm/lib/CodeGen/MachineVerifier.cpp:701 + } else if (MBB->succ_size() == LandingPadSuccs.size() || + MBB->succ_size() == IndirectPadSuccs.size()) { // It's possible that

[PATCH] D71491: [ubsan] Check implicit casts in ObjC for-in statements

vsk added inline comments. Comment at: compiler-rt/lib/ubsan/ubsan_value.cpp:29 +const char *__ubsan::getObjCClassName(ValueHandle Pointer) { +#if defined(__APPLE__) + // We need to query the ObjC runtime for some information, but do not want delcypher wrote: >

[PATCH] D71491: [ubsan] Check implicit casts in ObjC for-in statements

vsk updated this revision to Diff 238393. vsk marked an inline comment as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71491/new/ https://reviews.llvm.org/D71491 Files: clang/docs/UndefinedBehaviorSanitizer.rst clang/include/clang/Basic/Sanitizers.def clang/lib/CodeGen/CGObj

Re: [clang] e8f198d - Fix pack deduction to only deduce the arity of packs that are actually

Hi Hans, Please consider this bugfix for the Clang 10 release branch. On Wed, 15 Jan 2020 at 16:21, Richard Smith via cfe-commits < cfe-commits@lists.llvm.org> wrote: > > Author: Richard Smith > Date: 2020-01-15T16:21:08-08:00 > New Revision: e8f198dd9e9dabed8d50276465906e7c8827cada > > URL: > h

[clang] e8f198d - Fix pack deduction to only deduce the arity of packs that are actually

Author: Richard Smith Date: 2020-01-15T16:21:08-08:00 New Revision: e8f198dd9e9dabed8d50276465906e7c8827cada URL: https://github.com/llvm/llvm-project/commit/e8f198dd9e9dabed8d50276465906e7c8827cada DIFF: https://github.com/llvm/llvm-project/commit/e8f198dd9e9dabed8d50276465906e7c8827cada.diff

[PATCH] D44352: [Concepts] Type Constraints

ychen added a comment. I saw a null pointer dereference for the `p3-2a.cpp` file added here. It is not always reproducible for me, I was able to reproduce it once offline. Any idea? Crash stack is here: https://results.llvm-merge-guard.org/amd64_debian_testing_clang8-1220/console-log.txt From D

[PATCH] D72380: [DataFlow] Factor two worklist implementations out

xazax.hun added a comment. Here is an example: void f() { int i; while (i < 42 && i) { if (i) &i; } } This takes 17 visits before, 16 after. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72380/new/ https://reviews.llvm.org/D72380 __

Re: [clang] 23058f9 - [OPENMP]Do not use RTTI by default for NVPTX devices.

On Wed, Jan 15, 2020 at 3:09 PM Alexey Bataev wrote: > And I disabled it only for device side, which is NVPTX, no? Can host side > target class report that the target is NVPTX? If you look at the patch, it > disable RTTI only if current triple is NVPTX. Can it be true for the host? > You are corr

[clang] 4456076 - Revert "Further implement CWG 2292"

Author: Amy Huang Date: 2020-01-15T15:46:07-08:00 New Revision: 44560762c62d72a103bdceff49ffa70451efd5f8 URL: https://github.com/llvm/llvm-project/commit/44560762c62d72a103bdceff49ffa70451efd5f8 DIFF: https://github.com/llvm/llvm-project/commit/44560762c62d72a103bdceff49ffa70451efd5f8.diff LOG

[clang] b841b9e - [OPENMP]Use regular processing of vtable used when TU is a prefix.

Author: Alexey Bataev Date: 2020-01-15T18:42:23-05:00 New Revision: b841b9e96e605bed5a1f9b846a07aae88c65ce02 URL: https://github.com/llvm/llvm-project/commit/b841b9e96e605bed5a1f9b846a07aae88c65ce02 DIFF: https://github.com/llvm/llvm-project/commit/b841b9e96e605bed5a1f9b846a07aae88c65ce02.diff

[PATCH] D72380: [DataFlow] Factor two worklist implementations out

xazax.hun added a comment. In D72380#1823019 , @xazax.hun wrote: > In D72380#1822927 , @NoQ wrote: > > > The change in uninitialized values analysis gives me a bit of anxiety. > > Could you explain what exactly has

[PATCH] D72380: [DataFlow] Factor two worklist implementations out

xazax.hun added a comment. In D72380#1822927 , @NoQ wrote: > The change in uninitialized values analysis gives me a bit of anxiety. Could > you explain what exactly has changed that caused the change in the stats and > why you think it doesn't make a dif

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

ABataev added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:7709-7712 isa(Next->getAssociatedExpression()) || isa(Next->getAssociatedExpression()) || -isa(Next->getAssociatedExpression())) && +

[PATCH] D70700: [WebAssembly] Mangle the argc/argv `main` as `__main_argc_argv`

sbc100 accepted this revision. sbc100 added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/CodeGen/CodeGenModule.cpp:5589 + // alias for "main" in the no-argument case so that libc can detect when + // new-style no-argument main is in us

Re: [clang] 23058f9 - [OPENMP]Do not use RTTI by default for NVPTX devices.

On Wed, Jan 15, 2020 at 2:52 PM Alexey Bataev wrote: > 1. The problem is that it does not produce errors, > ATM, it does produce errors when it's disabled. > it leads to the emission of some declaration that cannot be resolved by > the linker. This what I was trying to avoid. > I'm OK with disab

[PATCH] D71082: Allow system header to provide their own implementation of some builtin

akhuang added a comment. Reverted in 3d210ed3d1880c615776b07d1916edb400c245a6 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71082/new/ https://reviews.llvm.org/D71082 ___

[clang] 3d210ed - Revert "Allow system header to provide their own implementation of some builtin"

Author: Amy Huang Date: 2020-01-15T15:03:45-08:00 New Revision: 3d210ed3d1880c615776b07d1916edb400c245a6 URL: https://github.com/llvm/llvm-project/commit/3d210ed3d1880c615776b07d1916edb400c245a6 DIFF: https://github.com/llvm/llvm-project/commit/3d210ed3d1880c615776b07d1916edb400c245a6.diff LOG

[PATCH] D50360: [Concepts] Requires Expressions

rsmith added inline comments. Comment at: clang/include/clang/AST/ExprConcepts.h:144-148 + struct SubstitutionDiagnostic { +StringRef SubstitutedEntity; +SourceLocation DiagLoc; +StringRef DiagMessage; + }; Please add a FIXME to store the diagnostic

[PATCH] D70700: [WebAssembly] Mangle the argc/argv `main` as `__main_argc_argv`

sunfish added a comment. @sbc100 Friendly ping :-). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70700/new/ https://reviews.llvm.org/D70700 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http

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

cchen created this revision. cchen added a reviewer: ABataev. Herald added subscribers: cfe-commits, guansong. Herald added a reviewer: jdoerfert. Herald added a project: clang. In OpenMP 5.0 we allow: int *p; int **p; struct S { int struct *sp; double *d; struct S **sp_ar

Re: [clang] 23058f9 - [OPENMP]Do not use RTTI by default for NVPTX devices.

Thank you. In general, RTTI should probably be treated similar to how we deal with inline assembly and ignore errors if they are in the code that we're not going to codegen during this side of compilation. E.g. during host-side compilation we don't complain about GPU-side registers in inline assem

[PATCH] D72380: [DataFlow] Factor two worklist implementations out

NoQ added a comment. I'm excited about this effort!~ The change in uninitialized values analysis gives me a bit of anxiety. Could you explain what exactly has changed that caused the change in the stats and why you think it doesn't make a difference, maybe give an example? (an example could be

[clang] 6b29aa2 - Revert "[OPENMP]Do not use RTTI by default for NVPTX devices."

Author: Alexey Bataev Date: 2020-01-15T17:42:26-05:00 New Revision: 6b29aa21180cf14bfb619d38fc4826913cabfb66 URL: https://github.com/llvm/llvm-project/commit/6b29aa21180cf14bfb619d38fc4826913cabfb66 DIFF: https://github.com/llvm/llvm-project/commit/6b29aa21180cf14bfb619d38fc4826913cabfb66.diff

[PATCH] D71082: Allow system header to provide their own implementation of some builtin

akhuang added a comment. This caused a linker error in chromium: ld.lld: error: undefined symbol: __warn_memset_zero_len Apparently now that the glibc memset is being used, __warn_memset_zero_len gets called from libc++ code (https://github.com/llvm/llvm-project/blob/b72a8c65e4e34779b6bc9e46

Re: [clang] 23058f9 - [OPENMP]Do not use RTTI by default for NVPTX devices.

Alexey, This breaks compilation of our cuda code which happens to transitively include protobuf headers. Can you, please, revert it for now until we figure out how RTTI should be handled? --Artem On Tue, Jan 14, 2020 at 3:15 PM Alexey Bataev via cfe-commits < cfe-commits@lists.llvm.org> wrote:

[PATCH] D72810: [LifetimeAnalysis] Add support for lifetime annotations on functions

xazax.hun created this revision. xazax.hun added reviewers: mgehre, ymandel, gribozavr2, aaron.ballman, rsmith, rjmccall. xazax.hun added a project: clang. Herald added subscribers: Szelethus, Charusso, gamesh411, dkrupp, rnkovacs, mgorny. This patch corresponds to this RFC: http://lists.llvm.o

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

ghvg1313 added a comment. @MyDeveloperDay gentle ping for help merging Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70926/new/ https://reviews.llvm.org/D70926 ___ cfe-commits mailing list cfe-commits@

[PATCH] D50360: [Concepts] Requires Expressions

rsmith added inline comments. Comment at: clang/lib/AST/StmtPrinter.cpp:2296 + S.flush(); + if (Buf.find("typename ") != 0) +OS << "typename "; I can see why this is the most straightforward way to implement this, but ... yuck. Please add a FIX

[PATCH] D72742: Don't assume promotable integers are zero/sign-extended already in x86-64 ABI.

rnk added a comment. In D72742#1822709 , @emilio wrote: > GCC does avoid the zero-extension on the caller sometimes, when not required > by the language and when its optimizer finds it suitable. I believe at the time, faced with ambiguity in the documen

[PATCH] D72612: [AArch64][SVE] Add ImmArg property to intrinsics with immediates

efriedma accepted this revision. efriedma added a comment. This revision is now accepted and ready to land. LGTM with one question Comment at: llvm/lib/Target/AArch64/AArch64InstrFormats.td:1108 + def "" : AsmVectorIndexOpnd, PatLeaf<(ty imm), pred>; + def _timm : AsmVectorIn

[PATCH] D50360: [Concepts] Requires Expressions

saar.raz marked 4 inline comments as done. saar.raz added a comment. Addressed comments in latest diff. Comment at: clang/include/clang/AST/RecursiveASTVisitor.h:2714-2716 + if (RetReq.isTypeConstraint()) +TRY_TO(TraverseTemplateParameterListHelper( +

[PATCH] D72806: [HIP] fix paths for executables not in clang bin directory

DieGoldeneEnte created this revision. DieGoldeneEnte added a reviewer: yaxunl. Herald added subscribers: cfe-commits, mgorny. Herald added a project: clang. The previous code constructed the executable paths for llvm-link, opt, lld, llc and clang-offload-bundler for the path to the compiler. This

[PATCH] D72547: [llvm] Make new pass manager's OptimizationLevel a class

mtrofin updated this revision to Diff 238352. mtrofin marked 4 inline comments as done. mtrofin added a comment. Herald added a subscriber: zzheng. Incorporated feedback: - tests - updated patch description Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.

[PATCH] D67678: PR17164: Change clang's default behavior from -flax-vector-conversions=all to -flax-vector-conversions=integer.

This revision was automatically updated to reflect the committed changes. Closed by commit rGb72a8c65e4e3: PR17164: Change clang's default behavior from -flax-vector-conversions=all to… (authored by Richard Smith , committed by rsmith). Changed prior to commit: http

Re: [clang] b72a8c6 - PR17164: Change clang's default behavior from -flax-vector-conversions=all to -flax-vector-conversions=integer.

Hi Hans, This flag flip just missed the branch point; can it be applied to the Clang 10 branch? (You'll also need 388eaa1.) Let me know if that's OK, and I'll remove the changes to the release notes from master. (Otherwise the version number in which this applies needs to be bumped.) On Wed, 15 J

[PATCH] D72742: Don't assume promotable integers are zero/sign-extended already in x86-64 ABI.

emilio added a comment. In D72742#1822554 , @rnk wrote: > +@chandlerc @rjmccall > > Didn't we work this out already when John added the alignment tracking stuff? > I remember this bug involving libjpegturbo standalone assembly receiving a > 32-bit argume

[clang] b72a8c6 - PR17164: Change clang's default behavior from -flax-vector-conversions=all to -flax-vector-conversions=integer.

Author: Richard Smith Date: 2020-01-15T13:14:57-08:00 New Revision: b72a8c65e4e34779b6bc9e466203f553f5294486 URL: https://github.com/llvm/llvm-project/commit/b72a8c65e4e34779b6bc9e466203f553f5294486 DIFF: https://github.com/llvm/llvm-project/commit/b72a8c65e4e34779b6bc9e466203f553f5294486.diff

[clang] 388eaa1 - Work around PR43337: don't try to use the vec_sel overloads for vector long long, since clang's doesn't provide it yet!

Author: Richard Smith Date: 2020-01-15T13:14:57-08:00 New Revision: 388eaa1270c2762d61b756759b6db8cf15bd3a83 URL: https://github.com/llvm/llvm-project/commit/388eaa1270c2762d61b756759b6db8cf15bd3a83 DIFF: https://github.com/llvm/llvm-project/commit/388eaa1270c2762d61b756759b6db8cf15bd3a83.diff

[PATCH] D72635: Add "context" capability to Thread Safety Analysis

eti-p-doray added a comment. I added an example in the description. From doc https://clang.llvm.org/docs/ThreadSafetyAnalysis.html, it sounds like we should be allowed to declare our class with CAPABILITY("context"), but it turns out that only "mutex" and "role" are allowed. I could otherwise

[PATCH] D67847: [Support] make report_fatal_error `abort` instead of `exit`

ychen added a comment. Apologies that it takes so long. I'll fix the test and land it today. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67847/new/ https://reviews.llvm.org/D67847 ___ cfe-commits mai

[PATCH] D72073: [Sema] Fix location of star ('*') inside MemberPointerTypeLoc

gribozavr2 added inline comments. Comment at: clang/lib/Parse/ParseDecl.cpp:5556 + SourceLocation StarLoc = Tok.getLocation(); SourceLocation Loc = ConsumeToken(); D.SetRangeEnd(Loc); ConsumeToken() returns Tok.getLocation(), so why do we need

[PATCH] D67847: [Support] make report_fatal_error `abort` instead of `exit`

jyknight added a comment. In D67847#1693694 , @rnk wrote: > In D67847#1691898 , @jyknight wrote: > > > The `abort()` function raises SIGABRT, for which the default behavior is to > > trigger a coredump. Do we actua

[PATCH] D72334: [Syntax] Build nodes for template declarations.

gribozavr2 added inline comments. Comment at: clang/lib/Tooling/Syntax/BuildTree.cpp:930 +Builder.markChildToken(TemplateKW, syntax::NodeRole::IntroducerKeyword); +Builder.markMaybeDelayedChild( +TemplatedDeclaration, Why is this range maybe-delay

[PATCH] D72334: [Syntax] Build nodes for template declarations.

gribozavr2 accepted this revision. gribozavr2 added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Tooling/Syntax/BuildTree.cpp:190 + // Set role for the node that may or may not be delayed. Node must span + // exactly \p Range. + void

[PATCH] D67847: [Support] make report_fatal_error `abort` instead of `exit`

rnk added a comment. I'm still in favor of this change. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67847/new/ https://reviews.llvm.org/D67847 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D72742: Don't assume promotable integers are zero/sign-extended already in x86-64 ABI.

rnk added subscribers: rjmccall, chandlerc. rnk added a comment. +@chandlerc @rjmccall Didn't we work this out already when John added the alignment tracking stuff? I remember this bug involving libjpegturbo standalone assembly receiving a 32-bit argument, and then using the full 64-bit RDI reg

[PATCH] D67847: [Support] make report_fatal_error `abort` instead of `exit`

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 61892 tests passed, 2 failed and 782 were skipped. failed: Clang.CXX/temp/temp_arg/temp_arg_template/p3-2a.cpp failed: LLVM.CodeGen/SystemZ/mverify-optypes.mir {icon question-circle color=gray} clang-tidy: unk

[PATCH] D71451: Support to emit extern variables debuginfo with "-fstandalone-debug"

dblaikie added a comment. In D71451#1822497 , @Jac1494 wrote: > >> I'd be curious to the answer to David's questions. If the size increase is > >> because of unused extern variables coming in from libc or something then > >> it doesn't seem worth the cos

[PATCH] D72282: [clang-tidy] Add `bugprone-unintended-adl`

logan-5 updated this revision to Diff 238328. logan-5 added a comment. Added TODO comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72282/new/ https://reviews.llvm.org/D72282 Files: clang-tools-extra/clang-tidy/bugprone/BugproneTidyModule.c

[PATCH] D72675: Fix -ffast-math/-ffp-contract interaction

spatel added inline comments. Comment at: llvm/test/CodeGen/PowerPC/fmf-propagation.ll:201-203 ; fma(X, 7.0, X * 42.0) --> X * 49.0 -; This is the minimum FMF needed for this transform - the FMA allows reassociation. +; This is the minimum FMF needed for this transform - the 'c

[PATCH] D72284: [clang-tidy] Factor out renaming logic from readability-identifier-naming

logan-5 marked 2 inline comments as done. logan-5 added inline comments. Comment at: clang-tools-extra/clang-tidy/utils/RenamerClangTidyCheck.h:29 +public: + RenamerClangTidyCheck(StringRef CheckName, ClangTidyContext *Context); + ~RenamerClangTidyCheck(); loga

[PATCH] D71451: Support to emit extern variables debuginfo with "-fstandalone-debug"

Jac1494 added a comment. >> I'd be curious to the answer to David's questions. If the size increase is >> because of unused extern variables coming in from libc or something then it >> doesn't seem worth the cost. For above case clang size is increase because ,it is difference between clang bu

[PATCH] D72786: [clang] Set function attributes on SEH filter functions correctly.

rnk added a comment. Thanks, the code change looks good, but please adjust the tests a bit. Comment at: clang/test/CodeGen/exceptions-seh-finally.c:284-286 -// Look for the absence of noinline. Enum attributes come first, so check that -// a string attribute is the first to ver

[PATCH] D72793: [clang-format] Expand the SpacesAroundConditions option to include catch statements

timwoj created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This diff expands the SpacesAroundConditions option added in D68346 to include adding spaces to catch statements. Repository: rG LLVM Github Monorepo https:/

[PATCH] D69878: Consoldiate internal denormal flushing controls

arsenm added a comment. Herald added a subscriber: kerbowa. ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69878/new/ https://reviews.llvm.org/D69878 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin

[PATCH] D71199: [clang-tidy] New check cppcoreguidelines-prefer-initialization-list

Eugene.Zelenko added a comment. By the word, please rebase, because Release Notes were reset after version 10 branching. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71199/new/ https://reviews.llvm.org/D71199 ___ cfe-commits mailing list

[PATCH] D72761: [ARM][MVE][Intrinsics] Add VMINAQ, VMINNMAQ, VMAXAQ, VMAXNMAQ intrinsics.

dmgreen added a comment. Nice one. Good to see codegen changes coming out of these intrinsics. It took a while for me to figure out what the integer instruction was doing. That's a strange one. The fp case I have a question about below. Comment at: llvm/lib/Target/ARM/ARMIns

[PATCH] D72785: [Matrix] Add __builtin_matrix_scalar_multiply to Clang (WIP).

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon question-circle color=gray} clang-format: unknown. Build artifacts : diff

[PATCH] D72786: [clang] Set function attributes on SEH filter functions correctly.

sanwou01 created this revision. sanwou01 added a reviewer: rnk. Herald added a project: clang. Herald added a subscriber: cfe-commits. When compiling with -munwind-tables, the SEH filter funclet needs the uwtable function attribute, which gets automatically added if we use SetInternalFunctionAttri

[PATCH] D72785: [Matrix] Add __builtin_matrix_scalar_multiply to Clang (WIP).

fhahn created this revision. Herald added a subscriber: tschuett. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D72785 Files: clang/include/clang/Basic/Builtins.def clang/include/clang/Basic/DiagnosticSemaKinds.td clang/include/clang/Sema/Sem

[PATCH] D72782: [Matrix] Add __builtin_matrix_column_store to Clang (WIP).

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon question-circle color=gray} clang-format: unknown. Build artifacts : diff

[PATCH] D72781: [Matrix] Add __builtin_matrix_column_load to Clang (WIP).

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon question-circle color=gray} clang-format: unknown. Build artifacts : diff

[PATCH] D72380: [DataFlow] Factor two worklist implementations out

xazax.hun added a comment. Ping for the other reviewers :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72380/new/ https://reviews.llvm.org/D72380 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailm

[PATCH] D72769: Replace CLANG_SPAWN_CC1 env var with a driver mode flag

thakis added a comment. cc cfe-commits for completeness CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72769/new/ https://reviews.llvm.org/D72769 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/

[PATCH] D72304: [OpenMP]{OMPIRBuilder] Add Directives (master and critical) to OMPBuilder.

fghanim updated this revision to Diff 238301. fghanim marked 25 inline comments as done. fghanim added a comment. Addressing reviewer comments - Added regression tests - styling - adding asserts and todo where needed. Also, Now `EmitInlinedRegion` will merge blocks where legal. Repository: r

[PATCH] D72782: [Matrix] Add __builtin_matrix_column_store to Clang (WIP).

fhahn created this revision. Herald added a subscriber: tschuett. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D72782 Files: clang/include/clang/Basic/Builtins.def clang/include/clang/Sema/Sema.h clang/lib/CodeGen/CGBuiltin.cpp clang/lib/S

[PATCH] D72778: [Matrix] Add __builtin_matrix_transpose to Clang (WIP).

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon question-circle color=gray} clang-format: unknown. Build artifacts : diff

[PATCH] D72777: [clangd] Dont display `` kinds in hover board

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

[PATCH] D72761: [ARM][MVE][Intrinsics] Add VMINAQ, VMINNMAQ, VMAXAQ, VMAXNMAQ intrinsics.

This revision was automatically updated to reflect the committed changes. Closed by commit rGda9d57d2c2dc: [ARM][MVE][Intrinsics] Add VMINAQ, VMINNMAQ, VMAXAQ, VMAXNMAQ intrinsics. (authored by MarkMurrayARM). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm

[PATCH] D72781: [Matrix] Add __builtin_matrix_column_load to Clang (WIP).

fhahn created this revision. Herald added a subscriber: tschuett. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D72781 Files: clang/include/clang/Basic/Builtins.def clang/include/clang/Sema/Sema.h clang/lib/CodeGen/CGBuiltin.cpp clang/lib/S

[clang] 8e5018e - Replace CLANG_SPAWN_CC1 env var with a driver mode flag

Author: Nico Weber Date: 2020-01-15T12:22:40-05:00 New Revision: 8e5018e990b701391e6c33ba85b012343df67272 URL: https://github.com/llvm/llvm-project/commit/8e5018e990b701391e6c33ba85b012343df67272 DIFF: https://github.com/llvm/llvm-project/commit/8e5018e990b701391e6c33ba85b012343df67272.diff LO

[clang] da9d57d - [ARM][MVE][Intrinsics] Add VMINAQ, VMINNMAQ, VMAXAQ, VMAXNMAQ intrinsics.

Author: Mark Murray Date: 2020-01-15T17:20:15Z New Revision: da9d57d2c2dc821979490a425142afde5107066c URL: https://github.com/llvm/llvm-project/commit/da9d57d2c2dc821979490a425142afde5107066c DIFF: https://github.com/llvm/llvm-project/commit/da9d57d2c2dc821979490a425142afde5107066c.diff LOG: [

[PATCH] D72777: [clangd] Dont display `` kinds in hover board

kadircet created this revision. kadircet added a reviewer: usaxena95. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Currently when hovering over an `auto` or `decltype` that resolve to a builtin-type, clangd would display `` as th

[PATCH] D72778: [Matrix] Add __builtin_matrix_transpose to Clang (WIP).

fhahn created this revision. Herald added a subscriber: tschuett. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D72778 Files: clang/include/clang/Basic/Builtins.def clang/include/clang/Sema/Sema.h clang/lib/CodeGen/CGBuiltin.cpp clang/lib/S

[PATCH] D72774: [Matrix] Add __builtin_matrix_multiply to Clang (WIP).

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon question-circle color=gray} clang-format: unknown. Build artifacts : diff

[PATCH] D72774: [Matrix] Add __builtin_matrix_multiply to Clang (WIP).

fhahn created this revision. Herald added a subscriber: tschuett. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D72774 Files: clang/include/clang/Basic/Builtins.def clang/include/clang/Basic/DiagnosticSemaKinds.td clang/include/clang/Sema/Sem

[clang-tools-extra] 041650d - [clangd] Extract string literals in macro arguments to unbreak gcc buildbots

Author: Kadir Cetinkaya Date: 2020-01-15T17:59:10+01:00 New Revision: 041650da67051266eb92b5bb07223394fe1bdab1 URL: https://github.com/llvm/llvm-project/commit/041650da67051266eb92b5bb07223394fe1bdab1 DIFF: https://github.com/llvm/llvm-project/commit/041650da67051266eb92b5bb07223394fe1bdab1.dif

[PATCH] D72723: Built-in functions for AMDGPU MFMA instructions.

kpyzhov added a comment. In D72723#1821917 , @arsenm wrote: > Having two subtarget features for the same feature is an issue ? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72723/new/ https://reviews.llvm.org/D72723

[PATCH] D72773: [Matrix] Add __builtin_matrix_{add,sub} to Clang (WIP).

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon question-circle color=gray} clang-format: unknown. Build artifacts : diff

[PATCH] D72773: [Matrix] Add __builtin_matrix_{add,sub} to Clang (WIP).

fhahn created this revision. Herald added a subscriber: tschuett. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D72773 Files: clang/include/clang/Basic/Builtins.def clang/include/clang/Basic/DiagnosticSemaKinds.td clang/include/clang/Sema/Sem

[PATCH] D72723: Built-in functions for AMDGPU MFMA instructions.

arsenm accepted this revision. arsenm added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72723/new/ https://reviews.llvm.org/D72723 ___ cfe-commits mailing list cfe-commits@l

[PATCH] D72772: [Matrix] Add __builtin_matrix_extract to Clang (WIP).

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon question-circle color=gray} clang-format: unknown. Build artifacts : diff

[PATCH] D72772: [Matrix] Add __builtin_matrix_extract to Clang (WIP).

fhahn created this revision. Herald added a subscriber: tschuett. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D72772 Files: clang/include/clang/Basic/Builtins.def clang/include/clang/Sema/Sema.h clang/lib/CodeGen/CGBuiltin.cpp clang/lib/S

  1   2   >