[PATCH] D70222: [clangd] Add support for .rsp files in compile_commands.json

lh123 updated this revision to Diff 231357. lh123 marked 6 inline comments as done. lh123 added a comment. address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70222/new/ https://reviews.llvm.org/D70222 Files:

[PATCH] D70799: [OpenMP] Lower taskyield using OpenMP IR Builder

rogfer01 created this revision. rogfer01 added reviewers: jdoerfert, kiranchandramohan. Herald added subscribers: llvm-commits, cfe-commits, guansong, hiraditya. Herald added projects: clang, LLVM. rogfer01 added a parent revision: D69922: [OpenMP] Use the OpenMP-IR-Builder. This is similar to

[PATCH] D70769: [Support] add vfs support for ExpandResponseFiles

lh123 updated this revision to Diff 231353. lh123 added a comment. address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70769/new/ https://reviews.llvm.org/D70769 Files: llvm/include/llvm/Support/CommandLine.h

[PATCH] D69785: [OpenMP] Introduce the OpenMP-IR-Builder

rogfer01 added inline comments. Comment at: llvm/include/llvm/Frontend/OpenMPKinds.def:165 + +__OMP_RTL(__kmpc_barrier, false, Void, IdentPtr, Int32) +__OMP_RTL(__kmpc_cancel_barrier, false, Int32, IdentPtr, Int32) As we migrate, we will end with a significant

[PATCH] D70769: [Support] add vfs support for ExpandResponseFiles

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

[PATCH] D69498: IR: Invert convergent attribute handling

mehdi_amini added a comment. Talked with Matt at the last social, they thought I was strongly opposed to this: I am not, and I told them I would clarify here: I don't buy the argument of "frontend projects have consistently run into problems related to this", I think this is not a good

[PATCH] D70694: Use InitLLVM in clang-tidy

This revision was automatically updated to reflect the committed changes. Closed by commit rGa7acba29c19a: Use InitLLVM in clang-tidy (authored by ruiu). Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang-tools-extra] a7acba2 - Use InitLLVM in clang-tidy

Author: Rui Ueyama Date: 2019-11-28T13:50:35+09:00 New Revision: a7acba29c19ac67c77ed282ec9432602ae21268d URL: https://github.com/llvm/llvm-project/commit/a7acba29c19ac67c77ed282ec9432602ae21268d DIFF: https://github.com/llvm/llvm-project/commit/a7acba29c19ac67c77ed282ec9432602ae21268d.diff

[PATCH] D66035: [WebAssembly] WIP: Add support for reference types

vchuravy updated this revision to Diff 231343. vchuravy marked 6 inline comments as done. vchuravy added a comment. - add test for passing anyref through a function - fix wrong name for funcref - fixes and formatting - fix error message Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D66035: [WebAssembly] WIP: Add support for reference types

vchuravy marked 4 inline comments as done. vchuravy added a comment. Rebased onto current master and added an initial test. (I will start adding more as I start integrating this with the rest of the toolchain) Comment at: llvm/lib/Object/WasmObjectFile.cpp:998 +if

[PATCH] D69785: [OpenMP] Introduce the OpenMP-IR-Builder

JonChesterfield added a comment. I'd very much like this to land soon. It's the prereq for a lot of other patches and the code looks good. It's tricky to test the infra before the users are landed so the unit test is particularly appreciated. Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D70411: [analyzer] CERT: StrChecker: 31.c

Charusso marked 2 inline comments as done. Charusso added a comment. The false positive suppression by going backwards on the bug-path of stored reports was a very simple concept, which turned out to be a useless one. The rough idea was that to invalidate every report in a path, and every other

[PATCH] D70411: [analyzer] CERT: StrChecker: 31.c

Charusso updated this revision to Diff 231341. Charusso added a comment. - Do not emit a report on re-binding a not null-terminated string, it may will be properly terminated (test added). - Added a test for the necessity of `SValVisitor` in this checker. - Some refactor. CHANGES SINCE LAST

[clang] 789a7aa - Properly disambiguate between array declarators and array subscript expressions.

Author: Richard Smith Date: 2019-11-27T17:54:26-08:00 New Revision: 789a7aa37d0cca70d6e48908ce3e8bb4e761e266 URL: https://github.com/llvm/llvm-project/commit/789a7aa37d0cca70d6e48908ce3e8bb4e761e266 DIFF: https://github.com/llvm/llvm-project/commit/789a7aa37d0cca70d6e48908ce3e8bb4e761e266.diff

[PATCH] D46234: Mark if a null statement is the result of constexpr folding

rsmith added a comment. In D46234#1762104 , @xazax.hun wrote: > Just a note for anyone willing to pick this up, PR32203 is also related. Further note: the right approach here would likely be to add a `DiscardedStmt` class deriving from `Stmt` that

[PATCH] D70302: [CodeGen] Fix clang crash on aggregate initialization of array of labels

xbolva00 added a comment. I think this was commited too soon. You should receive LGTM from clang maintainer(s). In D70302#1747372 , @nickie wrote: > LGTM, for all that it's worth. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D70302: [CodeGen] Fix clang crash on aggregate initialization of array of labels

johannes added a comment. Committed as 1ac700cdef787383ad49a0e37d9894491ef19480 - this should be a safe fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70302/new/

[PATCH] D70302: [CodeGen] Fix clang crash on aggregate initialization of array of labels

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 rG1ac700cdef78: [CodeGen] Fix clang crash on aggregate initialization of array of labels (authored by johannes).

[clang] 1ac700c - [CodeGen] Fix clang crash on aggregate initialization of array of labels

Author: Johannes Altmanninger Date: 2019-11-28T00:59:25+01:00 New Revision: 1ac700cdef787383ad49a0e37d9894491ef19480 URL: https://github.com/llvm/llvm-project/commit/1ac700cdef787383ad49a0e37d9894491ef19480 DIFF:

[PATCH] D70302: [CodeGen] Fix clang crash on aggregate initialization of array of labels

johannes updated this revision to Diff 231330. johannes edited the summary of this revision. johannes added a comment. update commit message Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70302/new/ https://reviews.llvm.org/D70302 Files:

[PATCH] D70739: [OPENMP50]Add device/isa context selector support.

ABataev marked an inline comment as done. ABataev added inline comments. Comment at: clang/include/clang/Sema/Sema.h:9320 using OMPCtxSelectorData = - OpenMPCtxSelectorData, ExprResult>; + OpenMPCtxSelectorData, ExprResult>; jdoerfert wrote: >

[PATCH] D70739: [OPENMP50]Add device/isa context selector support.

jdoerfert added inline comments. Comment at: clang/include/clang/Sema/Sema.h:9320 using OMPCtxSelectorData = - OpenMPCtxSelectorData, ExprResult>; + OpenMPCtxSelectorData, ExprResult>; ABataev wrote: > jdoerfert wrote: > > ABataev wrote: > > >

[PATCH] D60455: [SYCL] Implement SYCL device code outlining

aaron.ballman accepted this revision. aaron.ballman added a comment. The attribute bits LGTM aside from a wording nit with the diagnostic; I have no opinion on the CodeGen question. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:10080 + "template parameter of a

[PATCH] D70253: [AArch64][SVE2] Implement remaining SVE2 floating-point intrinsics

efriedma added inline comments. Comment at: llvm/include/llvm/IR/IntrinsicsAArch64.td:898 + llvm_i32_ty], +[IntrNoMem]>; + kmclaughlin wrote: > sdesmalen wrote: > > I'd expect the `llvm_i32_ty` to be an immediate for these

[PATCH] D70469: [attributes] [analyzer] Add handle related attributes

aaron.ballman added inline comments. Comment at: clang/include/clang/Basic/Attr.td:3445 + let Spellings = [Clang<"acquire_handle">]; + let Subjects = SubjectList<[Function, ParmVar], ErrorDiag>; + let Documentation = [AcquireHandleDocs]; xazax.hun wrote: >

[PATCH] D70094: [clang-tidy] new altera ID dependent backward branch check

Eugene.Zelenko added inline comments. Comment at: clang-tidy/altera/IdDependentBackwardBranchCheck.cpp:115 +IdDependentBackwardBranchCheck::hasIdDepField(const Expr *Expression) { + if (const MemberExpr *MemberExpression = dyn_cast(Expression)) { +const FieldDecl

[PATCH] D70764: build: reduce CMake handling for zlib

compnerd added a comment. @labath I think you are misunderstanding the patch. This is not autoselecting the dependencies. It is simply doing that based on an existing option that we have - `LLVM_ENABLE_ZLIB`. We could always search for zlib and override the results with `LLVM_ENABLE_ZLIB`

[PATCH] D70739: [OPENMP50]Add device/isa context selector support.

ABataev marked an inline comment as done. ABataev added inline comments. Comment at: clang/include/clang/Sema/Sema.h:9320 using OMPCtxSelectorData = - OpenMPCtxSelectorData, ExprResult>; + OpenMPCtxSelectorData, ExprResult>; jdoerfert wrote: >

[PATCH] D70739: [OPENMP50]Add device/isa context selector support.

jdoerfert added inline comments. Comment at: clang/include/clang/Sema/Sema.h:9320 using OMPCtxSelectorData = - OpenMPCtxSelectorData, ExprResult>; + OpenMPCtxSelectorData, ExprResult>; ABataev wrote: > jdoerfert wrote: > > I would like to avoid

[PATCH] D70094: [clang-tidy] new altera ID dependent backward branch check

ffrankies updated this revision to Diff 231309. ffrankies added a comment. Implemented changes requested by @Eugene.Zelenko Also changed for loop in `hasIdDepVar` and `hasIdDepField` to range-based for loops, and updated the license information in IdDependentBackwardBranchCheck.cpp CHANGES

[PATCH] D70071: [ConstExprPreter] Removed the flag forcing the use of the interpreter

This revision was automatically updated to reflect the committed changes. Closed by commit rGf584f04dab69: [ConstExprPreter] Removed the flag forcing the use of the interpreter (authored by nand). Changed prior to commit: https://reviews.llvm.org/D70071?vs=230791=231310#toc Repository: rG

[clang] f584f04 - [ConstExprPreter] Removed the flag forcing the use of the interpreter

Author: Nandor Licker Date: 2019-11-27T20:07:19Z New Revision: f584f04dab69ab15c8942753a145f0c6e7693bcc URL: https://github.com/llvm/llvm-project/commit/f584f04dab69ab15c8942753a145f0c6e7693bcc DIFF: https://github.com/llvm/llvm-project/commit/f584f04dab69ab15c8942753a145f0c6e7693bcc.diff

[PATCH] D70469: [attributes] [analyzer] Add handle related attributes

xazax.hun added inline comments. Comment at: clang/include/clang/Basic/Attr.td:3445 + let Spellings = [Clang<"acquire_handle">]; + let Subjects = SubjectList<[Function, ParmVar], ErrorDiag>; + let Documentation = [AcquireHandleDocs]; aaron.ballman wrote: >

[PATCH] D70779: AArch64: add support for newer Apple CPUs

fhahn accepted this revision. fhahn added a comment. This revision is now accepted and ready to land. LGTM, but it might be good to wait with committing until next week, so people in the US have a chance to take a look as well. Comment at:

[PATCH] D70469: [attributes] [analyzer] Add handle related attributes

xazax.hun updated this revision to Diff 231307. xazax.hun marked 5 inline comments as done. xazax.hun added a comment. - Address review comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70469/new/ https://reviews.llvm.org/D70469 Files: clang/include/clang/Basic/Attr.td

[PATCH] D70469: [attributes] [analyzer] Add handle related attributes

xazax.hun marked an inline comment as done. xazax.hun added inline comments. Comment at: clang/lib/Sema/SemaDeclAttr.cpp:6524 + if (auto *PVD = dyn_cast(D)) { +if (PVD->getType()->isIntegerType()) { + S.Diag(AL.getLoc(), diag::err_attribute_output_parameter)

[PATCH] D65591: [AST] Add a flag indicating if any subexpression had errors

rsmith added a comment. In D65591#1625744 , @aaron.ballman wrote: > In D65591#1625228 , @ilya-biryukov > wrote: > > > We can also assume they're cheap, use the visitor-based implementation and > > later switch

[PATCH] D70791: Workaround for MSVC 16.3.* pre-c++17 type trait linkage

erichkeane added a comment. In D70791#1762155 , @rnk wrote: > My version of cl.exe claims to have version 19.23.28107 and my headers are in > Tools/MSVC/14.23.28105/include, and I see that this is fixed in xtr1common. > is_integral_v and the helpers are

[PATCH] D70791: Workaround for MSVC 16.3.* pre-c++17 type trait linkage

rnk added a comment. My version of cl.exe claims to have version 19.23.28107 and my headers are in Tools/MSVC/14.23.28105/include, and I see that this is fixed in xtr1common. is_integral_v and the helpers are defined with implicit template instantiations. There are no fully specialized

[PATCH] D70693: [scan-build-py] Set of small fixes

xazax.hun added a comment. So after investigating the effects of this patch, the result is the following: Verbose mode (`-v`) that also prints findings to the command line will no longer be colored after the patch. If we do not want this regression there are two easy ways around it: 1. Make

[PATCH] D69746: [analyzer] FixItHint: Apply and test hints with the Clang Tidy's script

alexfh added a comment. In D69746#1756448 , @gribozavr2 wrote: > The only way I know people apply fixits is with the help of IDEs. This depends on the infrastructure available. Talking specifically about clang-tidy in our environment, I know of at

[PATCH] D46027: [clang-tidy] Fix PR35824

This revision was automatically updated to reflect the committed changes. Closed by commit rG5c5e860535d8: [clang-tidy] Fix PR35824 (authored by xazax.hun). Herald added a project: clang. Changed prior to commit: https://reviews.llvm.org/D46027?vs=231078=231301#toc Repository: rG LLVM

[PATCH] D46234: Mark if a null statement is the result of constexpr folding

xazax.hun added a comment. Herald added subscribers: Charusso, gamesh411, Szelethus. Herald added a project: clang. Just a note for anyone willing to pick this up, PR32203 is also related. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D46234/new/

[clang-tools-extra] 5c5e860 - [clang-tidy] Fix PR35824

Author: Gabor Horvath Date: 2019-11-27T11:07:16-08:00 New Revision: 5c5e860535d8924a3d6eb950bb8a4945df01e9b7 URL: https://github.com/llvm/llvm-project/commit/5c5e860535d8924a3d6eb950bb8a4945df01e9b7 DIFF: https://github.com/llvm/llvm-project/commit/5c5e860535d8924a3d6eb950bb8a4945df01e9b7.diff

[PATCH] D70605: [OpenCL] Fix address space for implicit conversion (PR43145)

rjmccall added inline comments. Comment at: clang/lib/Sema/SemaExprCXX.cpp:4108 + NewToType = Context.getPointerType(NewToType); +} + Hmm, sorry, just noticed this. Could you write this so that it preserves the original kind of pointer of `ToType`? I

[PATCH] D60455: [SYCL] Implement SYCL device code outlining

ABataev added inline comments. Comment at: clang/lib/CodeGen/CodeGenModule.cpp:2477 + if (LangOpts.SYCLIsDevice && Global->hasAttr()) { +// SYCL kernels can be templated and not called from anywhere in the bader wrote: > ABataev wrote: > > bader wrote: >

[PATCH] D69938: [OpenCL] Use __generic addr space when generating internal representation of lambda

rjmccall added inline comments. Comment at: clang/lib/Sema/SemaLambda.cpp:892 +if (AS != LangAS::Default) + EPI.TypeQuals.addAddressSpace(getDefaultCXXMethodAddrSpace()); + This should just be `AS`. Could you go ahead and change the other, existing

[PATCH] D70791: Workaround for MSVC 16.3.* pre-c++17 type trait linkage

erichkeane added a comment. In D70791#1762081 , @rnk wrote: > I thought we agreed to remove the workaround that we had for this and just > declare that the old MSVC STL release was buggy and users should update: >

[PATCH] D70791: Workaround for MSVC 16.3.* pre-c++17 type trait linkage

rnk added a subscriber: thakis. rnk added a comment. I thought we agreed to remove the workaround that we had for this and just declare that the old MSVC STL release was buggy and users should update: https://bugs.llvm.org/show_bug.cgi?id=42027 It's a point release that preserves ABI

[PATCH] D46027: [clang-tidy] Fix PR35824

alexfh accepted this revision. alexfh added a comment. > This approach will also introduce false negatives. Could you add a test showing this with a FIXME comment? > A better approach would be to check if the null statement is the result of > folding an if constexpr. > The current AST API

[PATCH] D70693: [scan-build-py] Set of small fixes

xazax.hun marked an inline comment as done. xazax.hun added inline comments. Comment at: clang/tools/scan-build-py/libscanbuild/clang.py:47 cmd.insert(1, '-###') +cmd.append('-fno-color-diagnostics') phosek wrote: > Alternative would be to set

[PATCH] D49466: Initial implementation of -fmacro-prefix-map and -ffile-prefix-map

probinson added a comment. In D49466#1761156 , @MaskRay wrote: > The ugly path separator pattern `{{(/|)}}` appears in 60+ tests. Can we > teach clang and other tools to > > 1. accept both `/` and `\` input In general they do, AFAIK, although it's

[PATCH] D70253: [AArch64][SVE2] Implement remaining SVE2 floating-point intrinsics

kmclaughlin added inline comments. Comment at: llvm/include/llvm/IR/IntrinsicsAArch64.td:898 + llvm_i32_ty], +[IntrNoMem]>; + sdesmalen wrote: > I'd expect the `llvm_i32_ty` to be an immediate for these instructions, > right? If

[PATCH] D46027: [clang-tidy] Fix PR35824

xazax.hun added a comment. Thanks! Updated context for this patch: A superior fix would be to follow through with the approach suggested by Richard in https://reviews.llvm.org/D46234 . However, since I do not have time to finish that and there are people complaining in the PR, I think it is

[PATCH] D46027: [clang-tidy] Fix PR35824

JonasToth accepted this revision. JonasToth added a comment. This revision is now accepted and ready to land. Maybe a short notice in the release notes wouldn't hurt. Otherwise LGTM *EDIT*: Aaron commented as well. Plz let him react before committing :) CHANGES SINCE LAST ACTION

[PATCH] D70791: Workaround for MSVC 16.3.* pre-c++17 type trait linkage

erichkeane created this revision. erichkeane added reviewers: rnk, rjmccall, majnemer. MSVC's type_trait header for the 16.3.* release in pre-c++17 mode exposes explicitly specialized constexpr variables for _Is_integral, is_void_v, and _Is_floating_point as not-inline (since that isn't available

[PATCH] D70605: [OpenCL] Fix address space for implicit conversion (PR43145)

svenvh updated this revision to Diff 231285. svenvh added a comment. Added test for references too. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70605/new/ https://reviews.llvm.org/D70605 Files: clang/lib/Sema/SemaExprCXX.cpp clang/test/CodeGenOpenCLCXX/addrspace-derived-base.cl

[PATCH] D70769: [Support] add vfs support for ExpandResponseFiles

kadircet added inline comments. Comment at: llvm/lib/Support/CommandLine.cpp:1053 + llvm::ErrorOr CurrDir = FS.getCurrentWorkingDirectory(); + if (!CurrDir) +return false; this comment has moved into a irrelevant line and wasn't addressed, so re-stating

[PATCH] D70222: [clangd] Add support for .rsp files in compile_commands.json

kadircet added inline comments. Comment at: clang/lib/Tooling/ExpandResponseFilesCompilationDatabase.cpp:28 + llvm::IntrusiveRefCntPtr FS) + : Base(std::move(Base)), Tokenizer(Tokenizer), FS(FS) { +assert(this->Base != nullptr); nit:

[PATCH] D70755: [LifetimeAnalysis] Fix PR44150

This revision was automatically updated to reflect the committed changes. Closed by commit rGbcd0798c47ca: [LifetimeAnalysis] Fix PR44150 (authored by xazax.hun). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70755/new/

[PATCH] D60455: [SYCL] Implement SYCL device code outlining

bader marked an inline comment as done. bader added inline comments. Comment at: clang/lib/CodeGen/CodeGenModule.cpp:2477 + if (LangOpts.SYCLIsDevice && Global->hasAttr()) { +// SYCL kernels can be templated and not called from anywhere in the ABataev

[clang] bcd0798 - [LifetimeAnalysis] Fix PR44150

Author: Gabor Horvath Date: 2019-11-27T09:15:14-08:00 New Revision: bcd0798c47ca865f95226859893016a17402441e URL: https://github.com/llvm/llvm-project/commit/bcd0798c47ca865f95226859893016a17402441e DIFF: https://github.com/llvm/llvm-project/commit/bcd0798c47ca865f95226859893016a17402441e.diff

[PATCH] D70787: [Syntax] Build SimpleDeclaration node that groups multiple declarators

merge_guards_bot added a comment. Build result: pass - 60223 tests passed, 0 failed and 732 were skipped. Log files: console-log.txt , CMakeCache.txt

[PATCH] D70787: [Syntax] Build SimpleDeclaration node that groups multiple declarators

merge_guards_bot added a comment. Build result: pass - 60223 tests passed, 0 failed and 732 were skipped. Log files: console-log.txt , CMakeCache.txt

[PATCH] D70788: [Syntax] A tool to dump syntax tree and token buffer

merge_guards_bot added a comment. Build result: pass - 60223 tests passed, 0 failed and 732 were skipped. Log files: console-log.txt , CMakeCache.txt

[PATCH] D70547: [ARM][MVE][Intrinsics] Add MVE VAND/VORR/VORN/VEOR/VBIC intrinsics.

This revision was automatically updated to reflect the committed changes. Closed by commit rGa048bf87fb65: [ARM][MVE][Intrinsics] Add MVE VAND/VORR/VORN/VEOR/VBIC intrinsics. Add unit… (authored by MarkMurrayARM). Changed prior to commit: https://reviews.llvm.org/D70547?vs=231057=231272#toc

[PATCH] D70546: [ARM][MVE][Intrinsics] Add MVE VMUL intrinsics.

This revision was automatically updated to reflect the committed changes. Closed by commit rGe8a8dbe9c458: [ARM][MVE][Intrinsics] Add MVE VMUL intrinsics. Remove annoying t1 from VMUL*… (authored by MarkMurrayARM). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D70545: [ARM][MVE][Intrinsics] Add MVE VABD intrinsics.

This revision was automatically updated to reflect the committed changes. Closed by commit rGf4bba07b87ce: [ARM][MVE][Intrinsics] Add MVE VABD intrinsics. Add unit tests. (authored by MarkMurrayARM). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D70765: LTOVisibility.rst: fix up syntax in example

pcc accepted this revision. pcc 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/D70765/new/ https://reviews.llvm.org/D70765 ___

[clang] f4bba07 - [ARM][MVE][Intrinsics] Add MVE VABD intrinsics. Add unit tests.

Author: Mark Murray Date: 2019-11-27T16:52:04Z New Revision: f4bba07b87ce7ad60d908d2fe02abe88d2d48fa4 URL: https://github.com/llvm/llvm-project/commit/f4bba07b87ce7ad60d908d2fe02abe88d2d48fa4 DIFF: https://github.com/llvm/llvm-project/commit/f4bba07b87ce7ad60d908d2fe02abe88d2d48fa4.diff LOG:

[PATCH] D70739: [OPENMP50]Add device/isa context selector support.

merge_guards_bot added a comment. Build result: pass - 60340 tests passed, 0 failed and 732 were skipped. Log files: console-log.txt , CMakeCache.txt

[PATCH] D70787: [Syntax] Build SimpleDeclaration node that groups multiple declarators

ilya-biryukov updated this revision to Diff 231269. ilya-biryukov added a comment. - Remove obsolete FIXME - Fix a typo Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70787/new/ https://reviews.llvm.org/D70787 Files:

[PATCH] D70788: [Syntax] A tool to dump syntax tree and token buffer

ilya-biryukov created this revision. Herald added subscribers: cfe-commits, mgorny. Herald added a project: clang. ilya-biryukov added a comment. ilya-biryukov planned changes to this revision. Would want to add some tests before landing this. But this is tool is generally useful to experiment

[PATCH] D70788: [Syntax] A tool to dump syntax tree and token buffer

ilya-biryukov added a comment. Would want to add some tests before landing this. But this is tool is generally useful to experiment with syntax trees, so posting it early (so that it doesn't get lost) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D70787: [Syntax] Build SimpleDeclaration node that groups multiple declarators

ilya-biryukov created this revision. ilya-biryukov added a reviewer: gribozavr2. Herald added a project: clang. Also remove the temporary TopLevelDeclaration node and add UnknownDeclaration to represent other unknown nodes. See the follow-up change for building more top-level declarations.

[PATCH] D69221: [clang][darwin] Fix search path logic for C_INCLUDE_DIRS

mhinz added a comment. I applied the fix to all toolchains now, but I'm unsure how to write a test for it. Neither could I find tests for similar options like `-DDEFAULT_SYSROOT`. My use case: I want to build a clang that has the same search path as the clang shipped with macOS. $

[PATCH] D69221: [clang][darwin] Fix search path logic for C_INCLUDE_DIRS

mhinz updated this revision to Diff 231266. mhinz added a comment. Herald added subscribers: aheejin, jgravelle-google, sbc100, dschuff. Apply fix to all toolchains. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69221/new/ https://reviews.llvm.org/D69221 Files:

[PATCH] D70739: [OPENMP50]Add device/isa context selector support.

ABataev marked 2 inline comments as done. ABataev added inline comments. Comment at: clang/include/clang/Sema/Sema.h:9320 using OMPCtxSelectorData = - OpenMPCtxSelectorData, ExprResult>; + OpenMPCtxSelectorData, ExprResult>; jdoerfert wrote: > I

[PATCH] D70739: [OPENMP50]Add device/isa context selector support.

ABataev updated this revision to Diff 231265. ABataev added a comment. Address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70739/new/ https://reviews.llvm.org/D70739 Files: clang/include/clang/Basic/Attr.td

[PATCH] D60455: [SYCL] Implement SYCL device code outlining

ABataev added inline comments. Comment at: clang/lib/CodeGen/CodeGenModule.cpp:2477 + if (LangOpts.SYCLIsDevice && Global->hasAttr()) { +// SYCL kernels can be templated and not called from anywhere in the bader wrote: > ABataev wrote: > > Need to check

[PATCH] D70773: [clangd] Handle the missing call expr in targetDecl.

This revision was automatically updated to reflect the committed changes. Closed by commit rG939544add98e: [clangd] Handle the missing call expr in targetDecl. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70773/new/

[PATCH] D60455: [SYCL] Implement SYCL device code outlining

bader marked an inline comment as done. bader added inline comments. Comment at: clang/lib/CodeGen/CodeGenModule.cpp:2477 + if (LangOpts.SYCLIsDevice && Global->hasAttr()) { +// SYCL kernels can be templated and not called from anywhere in the ABataev

[clang-tools-extra] 939544a - [clangd] Handle the missing call expr in targetDecl.

Author: Haojian Wu Date: 2019-11-27T16:22:20+01:00 New Revision: 939544add98ee6463d6abd6c28fa6c9ac4b6e104 URL: https://github.com/llvm/llvm-project/commit/939544add98ee6463d6abd6c28fa6c9ac4b6e104 DIFF: https://github.com/llvm/llvm-project/commit/939544add98ee6463d6abd6c28fa6c9ac4b6e104.diff

[PATCH] D70769: [Support] add vfs support for ExpandResponseFiles

merge_guards_bot added a comment. Build result: pass - 60338 tests passed, 0 failed and 732 were skipped. Log files: console-log.txt , CMakeCache.txt

[PATCH] D70512: [clangd] Rethink how SelectionTree deals with macros and #includes.

hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. Thanks, the patch looks good! please also update the patch description. Comment at: clang-tools-extra/clangd/Selection.cpp:63 + // Claims whichever expanded tokens in

[clang] f59614d - [OPENMP50]Add if clause in parallel for simd directive.

Author: Alexey Bataev Date: 2019-11-27T09:56:25-05:00 New Revision: f59614d906b5428f3687a44ee018df5840b301dd URL: https://github.com/llvm/llvm-project/commit/f59614d906b5428f3687a44ee018df5840b301dd DIFF: https://github.com/llvm/llvm-project/commit/f59614d906b5428f3687a44ee018df5840b301dd.diff

[PATCH] D69938: [OpenCL] Use __generic addr space when generating internal representation of lambda

Anastasia updated this revision to Diff 231247. Anastasia added a comment. Added `getDefaultCXXMethodAddrSpace` helper to Sema CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69938/new/ https://reviews.llvm.org/D69938 Files: clang/include/clang/Sema/Sema.h clang/lib/Sema/Sema.cpp

[PATCH] D70769: [Support] add vfs support for ExpandResponseFiles

lh123 marked 3 inline comments as done. lh123 added inline comments. Comment at: llvm/include/llvm/Support/CommandLine.h:47 +class FileSystem; +IntrusiveRefCntPtr getRealFileSystem(); + kadircet wrote: > now that we are also pulling the the function, I think it

[PATCH] D70769: [Support] add vfs support for ExpandResponseFiles

lh123 updated this revision to Diff 231242. lh123 marked an inline comment as done. lh123 added a comment. address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70769/new/ https://reviews.llvm.org/D70769 Files:

[PATCH] D70739: [OPENMP50]Add device/isa context selector support.

ABataev marked an inline comment as done. ABataev added inline comments. Comment at: clang/test/OpenMP/nvptx_declare_variant_device_isa_codegen.cpp:48 + +#pragma omp declare variant(foo) match(device = {isa("nvptx64")}) +int bar() { return 1; } jdoerfert wrote:

[PATCH] D60455: [SYCL] Implement SYCL device code outlining

ABataev added inline comments. Comment at: clang/lib/CodeGen/CodeGenModule.cpp:2477 + if (LangOpts.SYCLIsDevice && Global->hasAttr()) { +// SYCL kernels can be templated and not called from anywhere in the Need to check if the decl must be emitted at all.

[PATCH] D70500: [WebAssembly] Enable use of wasm-opt and LTO-enabled system libraries

sunfish added a comment. In D70500#1759831 , @ilya-biryukov wrote: > I'm not an expert in driver code and how it should behave, but being > consistent with how other tools are found definitely looks much better than > special-casing a single tool. >

[PATCH] D70780: [WebAssembly] Find wasm-opt with GetProgramPath

sunfish created this revision. sunfish added reviewers: dschuff, ilya-biryukov. Herald added subscribers: cfe-commits, aheejin, jgravelle-google, sbc100. Herald added a project: clang. Instead of just searching for wasm-opt in PATH, use `GetProgramPath`, which checks the `COMPILER_PATH`

[PATCH] D70539: [clang][CodeGen] Implicit Conversion Sanitizer: handle increment/derement (PR44054)

lebedev.ri reopened this revision. lebedev.ri added a comment. This revision is now accepted and ready to land. Temporarily reverted in rG62098354ab1c6c0dd2cecc8ac526d411bfe8aa20 - the assertion does not hold, need to remove it likely. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D61446: Generalize the pass registration mechanism used by Polly to any third-party tool

Alouest added a comment. Greetings, I stumbled upon a bug while toying with your example Bye.cpp. I modified your pass to make a ModulePass and while it works with /build/bin/opt -goodbye -wave-goodbye test.ll It's making clang crash with the command /build/bin/clang test.c I've

[PATCH] D70539: [clang][CodeGen] Implicit Conversion Sanitizer: handle increment/derement (PR44054)

lebedev.ri updated this revision to Diff 231237. lebedev.ri added a comment. Rebased. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70539/new/ https://reviews.llvm.org/D70539 Files: clang/docs/ReleaseNotes.rst

[clang] cbfa237 - Revert "[clang][CodeGen] Implicit Conversion Sanitizer: handle increment/decrement (PR44054)"

Author: Roman Lebedev Date: 2019-11-27T17:05:21+03:00 New Revision: cbfa237892e55b7129a1178c9b03f26683d643af URL: https://github.com/llvm/llvm-project/commit/cbfa237892e55b7129a1178c9b03f26683d643af DIFF: https://github.com/llvm/llvm-project/commit/cbfa237892e55b7129a1178c9b03f26683d643af.diff

[PATCH] D70222: [clangd] Add support for .rsp files in compile_commands.json

merge_guards_bot added a comment. Build result: FAILURE - Log files: console-log.txt , CMakeCache.txt Repository: rG

[PATCH] D70769: [Support] add vfs support for ExpandResponseFiles

kadircet added inline comments. Comment at: llvm/include/llvm/Support/CommandLine.h:47 +class FileSystem; +IntrusiveRefCntPtr getRealFileSystem(); + now that we are also pulling the the function, I think it is OK to include the header instead.

[PATCH] D70769: [Support] add vfs support for ExpandResponseFiles

jhenderson added a comment. Aside from one nit, this looks okay to me. I'd like someone else to look at it though too. I'm not at all familiar with the file system stuff to be confident to say everything there is good. Comment at: llvm/lib/Support/CommandLine.cpp:1148 +

  1   2   >