[clang] eaa7d00 - [analyzer][NFC] Add missing LLVM header blurb (and license)

Author: Gabor Marton Date: 2020-03-25T17:57:29+01:00 New Revision: eaa7d00f366ba53e9cf4391163b93253065c2e6c URL: https://github.com/llvm/llvm-project/commit/eaa7d00f366ba53e9cf4391163b93253065c2e6c DIFF: https://github.com/llvm/llvm-project/commit/eaa7d00f366ba53e9cf4391163b93253065c2e6c.diff

[clang] bb4da94 - [ARM, CDE] Implement predicated Q-register CDE intrinsics

Author: Mikhail Maltsev Date: 2020-03-25T17:08:19Z New Revision: bb4da94e5b5f64ea68197de9be44c7c5a4c91ce7 URL: https://github.com/llvm/llvm-project/commit/bb4da94e5b5f64ea68197de9be44c7c5a4c91ce7 DIFF: https://github.com/llvm/llvm-project/commit/bb4da94e5b5f64ea68197de9be44c7c5a4c91ce7.diff

[PATCH] D75561: Remove const qualifier from Modules returned by ExternalASTSource. (NFC)

aprantl added a comment. I added a comment to D75488 . It's in getOrCreateModule(). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75561/new/ https://reviews.llvm.org/D75561 ___ cfe-commits mailing list

[PATCH] D75661: Remove SequentialType from the type heirarchy.

efriedma updated this revision to Diff 252645. efriedma added a comment. Address lint comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75661/new/ https://reviews.llvm.org/D75661 Files: clang/lib/CodeGen/CGExprConstant.cpp

[PATCH] D75685: Add MS Mangling for OpenCL Pipe types, add mangling test.

erichkeane marked 3 inline comments as done. erichkeane added a comment. Both done here: fe5c719eaf572e23b700e75832ec37a3761b337b . Thanks for the review @Anastasia Comment at:

[PATCH] D75638: [Hexagon] Support for Linux/Musl ABI.

bcain added a comment. In D75638#1917565 , @bcain wrote: > @kparzysz do you have any thoughts about those review comments? @sidneym is this just pulling existing contents downstream to go upstream? If so let's expedite this please Repository: rG

[PATCH] D76572: Replace `T(x)` with `reinterpret_cast(x)` everywhere it means reinterpret_cast. No functional change

Quuxplusone marked 2 inline comments as done. Quuxplusone added a comment. Btw, @jhenderson and @rsmith, if you wanted to land your own files' pieces of this patch while waiting on whoever-it-is to approve the pieces in other files, that'd be cool. As I mentioned initially, I don't have the

[PATCH] D76761: [clang-tidy] Fix crash in readability-redundant-string-cstr

njames93 added a comment. In D76761#1941476 , @gribozavr2 wrote: > Right -- what I meant is a more detailed description of why, for example, > `tryGetCallExprAncestorForCxxConstructExpr` can't find the `CallExpr` in this > case -- is it not there, or

[PATCH] D76761: [clang-tidy] Fix crash in readability-redundant-string-cstr

njames93 updated this revision to Diff 252646. njames93 added a comment. - Fixed by replacing cumbersome code with the arguably proper solution Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76761/new/ https://reviews.llvm.org/D76761 Files:

[PATCH] D76795: [HIP] Change default --gpu-max-threads-per-block value to 1024

b-sumner added inline comments. Comment at: clang/lib/CodeGen/TargetInfo.cpp:8123 +// --gpu-max-threads-per-block=n or its default value for HIP. +const unsigned OpenCLMaxWorkGroupSize = 256; +const unsigned MaxWorkGroupSize = I'd like to see the

[PATCH] D70411: [analyzer] CERT: STR31-C

Charusso marked 6 inline comments as done. Charusso added a comment. "To prevent such errors, either limit copies through truncation or, preferably, ensure that the destination is of sufficient size to hold the character data" - from the rule's page. Most of the projects are fine truncating by

[PATCH] D74918: Add method to TargetInfo to get CPU cache line size

lebedev.ri added a comment. Like i said in previous comments, this can be moved later, that is not a blocker. Ship it first :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74918/new/ https://reviews.llvm.org/D74918

[clang] b915aec - Add method to TargetInfo to get CPU cache line size

Author: zoecarver Date: 2020-03-25T09:50:38-07:00 New Revision: b915aec6b591bdd9a9ffef73db241353b5ba2e2c URL: https://github.com/llvm/llvm-project/commit/b915aec6b591bdd9a9ffef73db241353b5ba2e2c DIFF: https://github.com/llvm/llvm-project/commit/b915aec6b591bdd9a9ffef73db241353b5ba2e2c.diff

[PATCH] D76741: [clangd] Support multiple cursors in selectionRange.

This revision was automatically updated to reflect the committed changes. Closed by commit rG8f237f9b09aa: [clangd] Support multiple cursors in selectionRange. (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76741/new/

[PATCH] D74918: Add method to TargetInfo to get CPU cache line size

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rGb915aec6b591: Add method to TargetInfo to get CPU cache line size (authored by zoecarver). Repository: rG LLVM Github

[PATCH] D75685: Add MS Mangling for OpenCL Pipe types, add mangling test.

Anastasia added a comment. Sorry for the delayed comments. It would be good to address those in a separate commit if possible. Comment at: clang/test/CodeGenOpenCL/pipe_types_mangling.cl:12 +// WINDOWS: define dso_local void @"?test1@@YAXU?$ocl_pipe@H$00@__clang@@@Z" +//

[PATCH] D76365: [cuda][hip] Add CUDA builtin surface/texture reference support.

tra added inline comments. Comment at: clang/lib/Headers/__clang_cuda_runtime_wrapper.h:82-94 #undef __CUDACC__ #if CUDA_VERSION < 9000 #define __CUDABE__ #else +#define __CUDACC__ #define __CUDA_LIBDEVICE__ #endif hliao wrote: > hliao wrote: > > tra

[PATCH] D76443: Use ConstantExpr cached APValues if present for code generation

wchilders updated this revision to Diff 252610. wchilders added a comment. Updated to remove duplicated (and out dated) APValue generation logic, and fixed formatting issues. (Thanks for pointing out something was awry Tyker) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76443/new/

[PATCH] D76782: [CodeGenObjC] Fix a crash when attempting to copy a zero-sized bit-field in a non-trivial C struct

erik.pilkington created this revision. erik.pilkington added a reviewer: ahatanak. Herald added subscribers: ributzka, dexonsmith, jkorous. Zero sized bit-fields aren't included in the CGRecordLayout, so we shouldn't be calling EmitLValueForField for them. rdar://60695105

[PATCH] D76610: [ARM,CDE] Implement predicated Q-register CDE intrinsics

This revision was automatically updated to reflect the committed changes. Closed by commit rGbb4da94e5b5f: [ARM,CDE] Implement predicated Q-register CDE intrinsics (authored by miyuki). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76610/new/

[PATCH] D76525: Expose cache line size in __builtin_get_cpu_cache_line_size

zoecarver added a comment. @jyknight yes, the current plan is to use it in libc++'s implementation. I can put that implementation (which uses this builtin) up for review before this lands. That way we can discuss the implementation before adding a (possibly unneeded) builtin. Sound good?

[PATCH] D76772: [AMDGPU] Add __builtin_amdgcn_workgroup_size_x/y/z

yaxunl updated this revision to Diff 252621. yaxunl marked 9 inline comments as done. yaxunl added a comment. Revised by Matt's comments CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76772/new/ https://reviews.llvm.org/D76772 Files: clang/include/clang/Basic/BuiltinsAMDGPU.def

[PATCH] D76772: [AMDGPU] Add __builtin_amdgcn_workgroup_size_x/y/z

yaxunl added inline comments. Comment at: clang/lib/CodeGen/CGBuiltin.cpp:13428 +return Call; + return CGF.Builder.CreateAddrSpaceCast(Call, RetTy); +} arsenm wrote: > Why is this necessary? The builtin always has the same return type? due to

[PATCH] D74541: [Analyzer] Use note tags to track iterator increments and decrements

baloghadamsoftware marked 4 inline comments as done. baloghadamsoftware added inline comments. Comment at: clang/test/Analysis/iterator-modelling.cpp:434 + //expected-note@-1 0-1{{Calling 'next}} + //expected-note@-2 0-1{{Passing the value 1 via 2nd parameter 'n'}} +

[clang] d788814 - Suppress a few -Wunreachable-code warnings.

Author: Nico Weber Date: 2020-03-25T13:55:42-04:00 New Revision: d7888149aa813fb543d562d0891739b1f85f5622 URL: https://github.com/llvm/llvm-project/commit/d7888149aa813fb543d562d0891739b1f85f5622 DIFF: https://github.com/llvm/llvm-project/commit/d7888149aa813fb543d562d0891739b1f85f5622.diff

[PATCH] D73898: [analyzer] StdLibraryFunctionsChecker: Add argument constraints

NoQ added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/StdLibraryFunctionsChecker.cpp:409 - Optional FoundSummary = findFunctionSummary(FD, CE, C); + for (const ValueConstraintPtr& VC : Summary.ArgConstraints) { +ProgramStateRef SuccessSt =

[PATCH] D75661: Remove SequentialType from the type heirarchy.

ctetreau added a comment. I'd like the see the clang-format complaints actually addressed, but don't bother unless you need to upload another patch for some other reason anyways. That aside, LGTM. This has been looked at by 3 people (plus yourself). So far, nobody has had any serious

[PATCH] D76384: Move FPFeatures from BinaryOperator bitfields to Trailing storage

mibintc marked an inline comment as done. mibintc added inline comments. Comment at: clang/include/clang/AST/Expr.h:3465 +SubExprs[RHS] = rhs; +hasFPFeatures = true; +isCompoundAssign = 1; rjmccall wrote: > Okay, so this is *always* adding trailing

[clang] 044c51d - Fix vector type scalar checking when the scalar operand is dependent

Author: Erich Keane Date: 2020-03-25T11:38:58-07:00 New Revision: 044c51d8d433da885438fcb141b15c80d7b62eda URL: https://github.com/llvm/llvm-project/commit/044c51d8d433da885438fcb141b15c80d7b62eda DIFF: https://github.com/llvm/llvm-project/commit/044c51d8d433da885438fcb141b15c80d7b62eda.diff

[PATCH] D74918: Add method to TargetInfo to get CPU cache line size

zoecarver added a comment. @lebedev.ri my misunderstanding. I'll commit :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74918/new/ https://reviews.llvm.org/D74918 ___ cfe-commits mailing list

[PATCH] D59321: WIP: AMDGPU: Teach toolchain to link rocm device libs

arsenm updated this revision to Diff 252630. arsenm added a comment. Herald added a subscriber: kerbowa. Rebase CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59321/new/ https://reviews.llvm.org/D59321 Files: clang/include/clang/Basic/DiagnosticDriverKinds.td

[PATCH] D73720: [Analyzer] Use note tags to track container begin and and changes

NoQ accepted this revision. NoQ added a comment. Lovely, thank you! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73720/new/ https://reviews.llvm.org/D73720 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D73307: Unique Names for Functions with Internal Linkage

davidxl added inline comments. Comment at: clang/docs/UsersManual.rst:1683 + Controls whether Clang emits a unique (best-effort) symbol name for internal + linkage symbols. The unique name is obtained by appending the MD5 hash of the + full module name to the original

[clang-tools-extra] 8f237f9 - [clangd] Support multiple cursors in selectionRange.

Author: Sam McCall Date: 2020-03-25T17:59:09+01:00 New Revision: 8f237f9b09aa10fcb684a2ceddc3128e1cafadc7 URL: https://github.com/llvm/llvm-project/commit/8f237f9b09aa10fcb684a2ceddc3128e1cafadc7 DIFF: https://github.com/llvm/llvm-project/commit/8f237f9b09aa10fcb684a2ceddc3128e1cafadc7.diff

[PATCH] D73898: [analyzer] StdLibraryFunctionsChecker: Add argument constraints

martong marked an inline comment as done. martong added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/StdLibraryFunctionsChecker.cpp:409 - Optional FoundSummary = findFunctionSummary(FD, CE, C); + for (const ValueConstraintPtr& VC : Summary.ArgConstraints) {

[PATCH] D76790: [analyzer] StdLibraryFunctionsChecker: fix bug with arg constraints

martong created this revision. martong added reviewers: NoQ, Szelethus, baloghadamsoftware. Herald added subscribers: cfe-commits, ASDenysPetrov, steakhal, Charusso, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet, xazax.hun, whisperity. Herald added a project: clang.

[PATCH] D73898: [analyzer] StdLibraryFunctionsChecker: Add argument constraints

martong added a comment. I just created a quick fix for the issue: https://reviews.llvm.org/D76790 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73898/new/ https://reviews.llvm.org/D73898 ___

[PATCH] D76790: [analyzer] StdLibraryFunctionsChecker: fix bug with arg constraints

martong added a comment. I was thinking about the below test, but then I reverted back because I don't want to add "fake" summaries for testing purposes. Perhaps adding a new checker option could enable these "fake" summaries, @Szelethus what's your opinion? diff --git

[PATCH] D76795: [HIP] Change default --gpu-max-threads-per-block value to 1024

yaxunl created this revision. yaxunl added reviewers: b-sumner, tra. Herald added subscribers: kerbowa, nhaehnle, jvesely. This better matches CUDA behavior. https://reviews.llvm.org/D76795 Files: clang/include/clang/Basic/LangOptions.def clang/lib/CodeGen/TargetInfo.cpp

[PATCH] D74144: [OPENMP50]Add basic support for array-shaping operation.

rjmccall added a comment. Okay, so these array-shaping expressions are only allowed as operands to specific OpenMP directives? That's a plausible interpretation of "The shape-operator can appear only in clauses where it is explicitly allowed" from the spec

[PATCH] D75779: [OpenMP] `omp begin/end declare variant` - part 2, sema (+"CG")

hfinkel added inline comments. Comment at: clang/lib/AST/DeclarationName.cpp:147 +Name = Name.split(getOpenMPVariantManglingSeparatorStr()).first; + OS << Name; +} Would it make sense to print " (omp variant)" after the name to disambiguate?

[clang] fe5c719 - Implement post-commit comments for D75685/rG86e0a6c60627

Author: Erich Keane Date: 2020-03-25T12:24:56-07:00 New Revision: fe5c719eaf572e23b700e75832ec37a3761b337b URL: https://github.com/llvm/llvm-project/commit/fe5c719eaf572e23b700e75832ec37a3761b337b DIFF: https://github.com/llvm/llvm-project/commit/fe5c719eaf572e23b700e75832ec37a3761b337b.diff

[PATCH] D74183: [IRGen] Add an alignment attribute to underaligned sret parameters

efriedma added a comment. If it's just tramp3d-v4, I'm not that concerned... but that's a weird result. On x86 in particular, alignment markings have almost no effect on optimization, generally. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D75638: [Hexagon] Support for Linux/Musl ABI.

kparzysz added a comment. I'm not sure if it's worth defining these macros especially if this is the only place where they are used. We'd need a more coordinated effort to replace all 32's with named constants where it refers to a register size, and I'm not even sure if we could classify

[PATCH] D76791: [Matrix] Implement matrix index expressions ([][]).

fhahn created this revision. fhahn added reviewers: rjmccall, anemet, Bigcheese, rsmith, martong. Herald added subscribers: tschuett, arphaman, dexonsmith, rnkovacs. Herald added a project: clang. This patch implements matrix index expressions (matrix[RowIdx][ColumnIdx]). It does so by utilizing

[PATCH] D76725: [clangd] Build ASTs only with fresh preambles or after building a new preamble

kadircet updated this revision to Diff 252633. kadircet added a comment. - Build golden asts in ASTWorker thread, rather than preamble thread. - Explain new model in file comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76725/new/

[PATCH] D76725: [clangd] Build ASTs only with fresh preambles or after building a new preamble

kadircet added a comment. In D76725#1940282 , @sammccall wrote: > So I think we need a careful description of the new model somewhere. Not so > much on specific changes/constraints parts of the code operate under, but > what it's trying to do. > > My

[PATCH] D72281: [Matrix] Add matrix type to Clang.

fhahn added a reviewer: rjmccall. fhahn marked 2 inline comments as done. fhahn added inline comments. Comment at: clang/lib/AST/ASTStructuralEquivalence.cpp:647 + return false; +if (Mat1->getNumRows() != Mat2->getNumRows()) + return false; martong

[PATCH] D72281: [Matrix] Add matrix type to Clang.

fhahn updated this revision to Diff 252634. fhahn edited the summary of this revision. fhahn added a comment. Include columns in structural equi check, fixed type printing todo & rebased Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72281/new/

[PATCH] D76793: [Matrix] Implement + and - operators for MatrixType.

fhahn created this revision. fhahn added reviewers: rjmccall, anemet, Bigcheese, rsmith, martong. Herald added subscribers: tschuett, dexonsmith, rnkovacs. Herald added a project: clang. This patch implements the + and - binary operators for values of MatrixType. It adds support for matrix +/-

[PATCH] D72283: [Matrix] Add __builtin_matrix_insert to Clang (WIP).

fhahn abandoned this revision. fhahn added a comment. superseded by D76791 using operators. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72283/new/ https://reviews.llvm.org/D72283

[PATCH] D76794: [Matrix] Implement * binary operator for MatrixType.

fhahn created this revision. fhahn added reviewers: rjmccall, anemet, Bigcheese, rsmith, martong. Herald added subscribers: tschuett, dexonsmith, rnkovacs. Herald added a project: clang. This patch implements the * binary operator for values of MatrixType. It adds support for matrix * matrix,

[PATCH] D76384: Move FPFeatures from BinaryOperator bitfields to Trailing storage

rjmccall added inline comments. Comment at: clang/include/clang/AST/Expr.h:3465 +SubExprs[RHS] = rhs; +hasFPFeatures = true; +isCompoundAssign = 1; mibintc wrote: > rjmccall wrote: > > Okay, so this is *always* adding trailing storage. Is there a

[PATCH] D76770: [CodeComplete] Don't replace the rest of line in #include completion.

sammccall added a comment. Does this regress the case where you complete a file rather than a directory? e.g. `#include "foo/^one.h"` If you select "two.h", then we definitely want to replace "one.h", right? The root here is that we have to decide what to replace independently of the completion

[PATCH] D76525: Expose cache line size in __builtin_get_cpu_cache_line_size

jyknight added a comment. I don't think there's a particularly good reason to expose this as a builtin, unless it's to be used by the standard library implementation. (Which again I do not think we should implement.) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D76342: [OpenMP] Implement '#pragma omp tile'

JonChesterfield added a comment. edit: actually you've already done the clang-format on trunk as I hoped, phabricator mislead me. Apologies for the noise Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76342/new/ https://reviews.llvm.org/D76342

[clang] c46a0c0 - [clang codegen] Address review comment on comment in constWithPadding.

Author: Eli Friedman Date: 2020-03-25T10:58:03-07:00 New Revision: c46a0c07a634d85d3fed2ef94747ea19fb4d2d69 URL: https://github.com/llvm/llvm-project/commit/c46a0c07a634d85d3fed2ef94747ea19fb4d2d69 DIFF: https://github.com/llvm/llvm-project/commit/c46a0c07a634d85d3fed2ef94747ea19fb4d2d69.diff

[PATCH] D75779: [OpenMP] `omp begin/end declare variant` - part 2, sema (+"CG")

jdoerfert added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:5396 + IdentifierInfo = Context.Idents.get( + (D.getIdentifier()->getName() + "." + DVScope.NameSuffix).str()); + D.SetIdentifier(, D.getBeginLoc()); hfinkel wrote: > jdoerfert

[PATCH] D76724: Prevent immediate evaluations inside of decltype

Tyker added inline comments. Comment at: clang/lib/Sema/SemaExpr.cpp:15389 if (!E.isUsable() || !Decl || !Decl->isConsteval() || isConstantEvaluated() || - RebuildingImmediateInvocation) + isInDeclType(*this) || RebuildingImmediateInvocation) return E;

[PATCH] D76691: [AST][SVE] Treat built-in SVE types as trivially copyable

efriedma accepted this revision. efriedma 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/D76691/new/ https://reviews.llvm.org/D76691

[PATCH] D76690: [AST][SVE] Treat built-in SVE types as POD

efriedma accepted this revision. efriedma added a comment. This revision is now accepted and ready to land. Okay, then LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76690/new/ https://reviews.llvm.org/D76690

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

fhahn added a comment. Superseded by D76794 using operators. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72774/new/ https://reviews.llvm.org/D72774 ___ cfe-commits

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

fhahn abandoned this revision. fhahn added a comment. Superseded by D76794 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72785/new/ https://reviews.llvm.org/D72785

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

fhahn abandoned this revision. fhahn added a comment. superseded by D76791 using operators. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72772/new/ https://reviews.llvm.org/D72772

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

fhahn abandoned this revision. fhahn added a comment. superseded by D76793 using operators. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72773/new/ https://reviews.llvm.org/D72773

[PATCH] D76692: [AST][SVE] Treat built-in SVE types as trivial

efriedma accepted this revision. efriedma 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/D76692/new/ https://reviews.llvm.org/D76692

[clang] df48e39 - [Clang] Fix clang-tidy errors.

Author: Alexander Belyaev Date: 2020-03-25T20:13:43+01:00 New Revision: df48e3920a56b128ad691c258372c641b076f21c URL: https://github.com/llvm/llvm-project/commit/df48e3920a56b128ad691c258372c641b076f21c DIFF:

[PATCH] D76801: [AST] Print a> without extra spaces in C++11 or later.

sammccall updated this revision to Diff 252671. sammccall added a comment. update/move comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76801/new/ https://reviews.llvm.org/D76801 Files: clang-tools-extra/clangd/unittests/HoverTests.cpp

[PATCH] D76795: [HIP] Change default --gpu-max-threads-per-block value to 1024

b-sumner added a comment. Thanks. This looks fine to me. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76795/new/ https://reviews.llvm.org/D76795 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D76663: [clangd] Support new semanticTokens request from LSP 3.16.

sammccall updated this revision to Diff 252676. sammccall added a comment. Update SemanticHighlighting comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76663/new/ https://reviews.llvm.org/D76663 Files:

[PATCH] D76801: [AST] Print a> without extra spaces in C++11 or later.

sammccall added a comment. @kadircet sending this to you as I noticed it as a clangd hover bug :-) The highest impact is probably in diagnostic messages though. AFAICT the state before this patch is: - the type printer always printed the space - the stmt printer calls into the type printer to

[PATCH] D73649: [CodeComplete] Member completion for concept-constrained types.

sammccall added a comment. @kadircet I realized this change was gathering dust, and you've touched SemaCodeComplete in recent memory... any interest? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73649/new/ https://reviews.llvm.org/D73649

[PATCH] D76663: [clangd] Support new semanticTokens request from LSP 3.16.

sammccall added a comment. For some context here, I was doing some digging as Heyward Fann is looking at hooking up our existing syntax highlighting to coc.nvim, and he and Jack Guo (of jackguo380/vim-lsp-cxx-highlight) were asking about the protocol. The new LSP protocol looks really solid,

[PATCH] D59321: WIP: AMDGPU: Teach toolchain to link rocm device libs

arsenm updated this revision to Diff 252715. arsenm added a comment. Herald added a subscriber: hiraditya. Handle wave64 library CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59321/new/ https://reviews.llvm.org/D59321 Files: clang/include/clang/Basic/DiagnosticDriverKinds.td

[PATCH] D75661: Remove SequentialType from the type heirarchy.

efriedma updated this revision to Diff 252659. efriedma added a comment. Herald added subscribers: Joonsoo, liufengdb, lucyrfox, mgester, arpith-jacob, csigg, nicolasvasilache, antiagainst, shauheen, burmako, jpienaar, rriddle, mehdi_amini. Fix bug in GlobalOpt. Add fixes for MLIR.

[PATCH] D74144: [OPENMP50]Add basic support for array-shaping operation.

ABataev marked an inline comment as done. ABataev added inline comments. Comment at: clang/lib/Sema/SemaExpr.cpp:4751 + LParenLoc, RParenLoc, Dims, Brackets); + if (!BaseTy->isAnyPointerType()) +return ExprError(Diag(Base->getExprLoc(),

[PATCH] D74144: [OPENMP50]Add basic support for array-shaping operation.

ABataev marked 4 inline comments as done. ABataev added a comment. In D74144#1941971 , @rjmccall wrote: > Okay, so these array-shaping expressions are only allowed as operands to > specific OpenMP directives? That's a plausible interpretation of "The >

[PATCH] D71687: Fix full loop unrolling initialization in new pass manager

vitalybuka accepted this revision. vitalybuka added a comment. Can this be landed? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71687/new/ https://reviews.llvm.org/D71687 ___ cfe-commits mailing list

[PATCH] D74144: [OPENMP50]Add basic support for array-shaping operation.

rjmccall added a comment. > We cannot build an array type since this expression does not represent a > supported array type. Sizes may be non-constant anywhere in the operation, > variable array type does not support it. I don't think that's true; the element type of a C99 VAT can still be

[PATCH] D76801: [AST] Print a> without extra spaces in C++11 or later.

sammccall created this revision. sammccall added a reviewer: kadircet. Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous. Herald added a project: clang. It's not 1998 anymore. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D76801 Files:

[PATCH] D76795: [HIP] Change default --gpu-max-threads-per-block value to 1024

yaxunl updated this revision to Diff 252661. yaxunl added a comment. change variable names CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76795/new/ https://reviews.llvm.org/D76795 Files: clang/include/clang/Basic/LangOptions.def clang/lib/CodeGen/TargetInfo.cpp

[PATCH] D75951: Keep a list of already-included pragma-once files for mods.

oontvoo added a comment. @jyknight Actually, please hold on on the review ... still working on it Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75951/new/ https://reviews.llvm.org/D75951 ___

[PATCH] D76795: [HIP] Change default --gpu-max-threads-per-block value to 1024

yaxunl marked 2 inline comments as done. yaxunl added inline comments. Comment at: clang/lib/CodeGen/TargetInfo.cpp:8123 +// --gpu-max-threads-per-block=n or its default value for HIP. +const unsigned OpenCLMaxWorkGroupSize = 256; +const unsigned MaxWorkGroupSize =

[clang-tools-extra] c2d7a1f - [clangd] Add test for FindTarget+RecoveryExpr (which already works). NFC

Author: Sam McCall Date: 2020-03-26T00:40:29+01:00 New Revision: c2d7a1f79cedfc9fcb518596aa839da4de0adb69 URL: https://github.com/llvm/llvm-project/commit/c2d7a1f79cedfc9fcb518596aa839da4de0adb69 DIFF: https://github.com/llvm/llvm-project/commit/c2d7a1f79cedfc9fcb518596aa839da4de0adb69.diff

[PATCH] D75661: Remove SequentialType from the type heirarchy.

ctetreau added inline comments. Comment at: mlir/lib/Target/LLVMIR/ModuleTranslation.cpp:81 static llvm::Type *getInnermostElementType(llvm::Type *type) { - while (isa(type)) -type = type->getSequentialElementType(); - return type; + do { +if (auto *arrayTy =

[PATCH] D59321: WIP: AMDGPU: Teach toolchain to link rocm device libs

arsenm updated this revision to Diff 252692. arsenm added a comment. Use -nogpulib instead of -nodefaultlibs CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59321/new/ https://reviews.llvm.org/D59321 Files: clang/include/clang/Basic/DiagnosticDriverKinds.td

[PATCH] D73307: Unique Names for Functions with Internal Linkage

tmsriram updated this revision to Diff 252688. tmsriram marked 4 inline comments as done. tmsriram added a comment. Changes to description of flag, remove redundant check. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73307/new/ https://reviews.llvm.org/D73307 Files:

[PATCH] D73369: [clangd] Simplify "preferred" vs "definition" logic a bit in XRefs AST code.

sammccall added inline comments. Comment at: clang-tools-extra/clangd/XRefs.cpp:257 const NamedDecl *Def = getDefinition(D); -const NamedDecl *Preferred = Def ? Def : D; +if (const NamedDecl *C = llvm::dyn_cast(D->getCanonicalDecl())) + D = C;

[PATCH] D73369: [clangd] Simplify "preferred" vs "definition" logic a bit in XRefs AST code.

sammccall updated this revision to Diff 252690. sammccall added a comment. clang-format Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73369/new/ https://reviews.llvm.org/D73369 Files: clang-tools-extra/clangd/XRefs.cpp

[PATCH] D73369: [clangd] Simplify "preferred" vs "definition" logic a bit in XRefs AST code.

sammccall updated this revision to Diff 252689. sammccall marked 3 inline comments as done. sammccall added a comment. address comments, rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73369/new/ https://reviews.llvm.org/D73369 Files:

[PATCH] D76342: [OpenMP] Implement '#pragma omp tile'

Meinersbur added a comment. In D76342#1941591 , @JonChesterfield wrote: > edit: actually you've already done the clang-format on trunk as I hoped, > phabricator mislead me. Apologies for the noise The previous diff was against a branch with only the

[PATCH] D74144: [OPENMP50]Add basic support for array-shaping operation.

ABataev updated this revision to Diff 252700. ABataev added a comment. Rebase + fixes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74144/new/ https://reviews.llvm.org/D74144 Files: clang/include/clang-c/Index.h

[PATCH] D69585: PerformPendingInstatiations() already in the PCH

rsmith added a comment. This needs to be done behind a flag. It's an explicit design goal that compilation behavior using a PCH or precompiled preamble behaves identically to compilation not using a PCH / precompiled preamble. The behavior of this patch is also non-conforming: we're only

[PATCH] D59321: WIP: AMDGPU: Teach toolchain to link rocm device libs

arsenm updated this revision to Diff 252708. arsenm added a comment. Cleanup a bit. This should eventually merge with the HIP library handling when it switches to using clang cc1 instead of llc CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59321/new/ https://reviews.llvm.org/D59321

[PATCH] D75153: [ThinLTO] Allow usage of all SMT threads in the system

aganea marked 3 inline comments as done. aganea added a subscriber: respindola. aganea added a comment. In D75153#1906580 , @MaskRay wrote: > Does `taskset -c 0-3 lld -flavor ...` restrict the number of cores? > > cpu_set_t cpu; > sched_getaffinity(0,

[PATCH] D75951: Keep a list of already-included pragma-once files for mods.

oontvoo updated this revision to Diff 252685. oontvoo added a comment. Handle the case where the file is first seen Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75951/new/ https://reviews.llvm.org/D75951 Files:

[clang-tools-extra] 43c6334 - [clangd] Work around gcc bug after 8f237f9b0

Author: Sam McCall Date: 2020-03-26T00:06:54+01:00 New Revision: 43c63349f55d89e335108692aa0970a4602f4385 URL: https://github.com/llvm/llvm-project/commit/43c63349f55d89e335108692aa0970a4602f4385 DIFF: https://github.com/llvm/llvm-project/commit/43c63349f55d89e335108692aa0970a4602f4385.diff

[PATCH] D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata

lanza updated this revision to Diff 252706. lanza added a comment. Rename and address some issues Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75574/new/ https://reviews.llvm.org/D75574 Files: clang/include/clang/AST/DeclObjC.h

[PATCH] D76812: [X86] Add Indirect Thunk Support to X86 to mitigate Load Value Injection (LVI) [3/3]

sconstab created this revision. sconstab added reviewers: craig.topper, andrew.w.kaylor, zbrid, chandlerc. Herald added subscribers: jfb, hiraditya. sconstab added a parent revision: D76811: [X86] Refactor X86IndirectThunks.cpp to Accomodate Mitigations other than Retpoline [2/3]. sconstab

[PATCH] D75574: RFC: Implement objc_direct_protocol attribute to remove protocol metadata

lanza updated this revision to Diff 252707. lanza added a comment. Reword commit Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75574/new/ https://reviews.llvm.org/D75574 Files: clang/include/clang/AST/DeclObjC.h

  1   2   >