[PATCH] D109260: [RISCV] Add SiFive cores E and S series

This revision was automatically updated to reflect the committed changes. Closed by commit rG4bc8dbe0cae3: [RISCV] Add SiFive cores E and S series (authored by apivovarov). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109260/new/ https://reviews.ll

[PATCH] D109487: [X86] Support *_set1_pch(Float16 _Complex h)

pengfei created this revision. pengfei added reviewers: LuoYuanke, FreddyYe, craig.topper, RKSimon. pengfei requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D109487 Files: cl

[PATCH] D109489: [OptParser] NFC: Remove unused template arg 'name' from bool opt

c-rhodes created this revision. c-rhodes added a reviewer: jansvoboda11. Herald added a subscriber: dang. c-rhodes requested review of this revision. Herald added projects: clang, LLVM. Herald added a subscriber: cfe-commits. Identified in D109359 . Repository:

[PATCH] D109489: [OptParser] NFC: Remove unused template arg 'name' from bool opt

c-rhodes added a comment. @jansvoboda11 this came up in D109359 with warning: llvm/include/llvm/Option/OptParser.td:217:91: warning: unused template argument: MarshallingInfoBooleanFlag:name I see you've changed worked on this in the past, wasn't sure if it

[PATCH] D108003: [Clang] Extend -Wbool-operation to warn about bitwise and of bools with side effects

rpbeltran added a comment. Sorry for the late reply on this. I ran the warning against ChromeOS and actually found very few false positives and a couple interesting findings to file bugs for. I actually saw about an equal number of cases caught for `&` and `|`. https://source.chromium.org/chro

[PATCH] D109485: [clang-scan-deps] Add an API for clang dependency scanner to perform module lookup by name alone

jansvoboda11 accepted this revision. jansvoboda11 added a comment. This revision is now accepted and ready to land. LGTM, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109485/new/ https://reviews.llvm.org/D109485 __

[PATCH] D109492: [OpenCL] Test case for C++ for OpenCL 2021 in OpenCL C header test

Topotuna created this revision. Topotuna added a reviewer: Anastasia. Herald added subscribers: ldrumm, yaxunl. Topotuna requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. RUN line representing C++ for OpenCL 2021 added to the test. This shoul

[PATCH] D109061: [openmp] No longer use LIBRARY_PATH to find devicertl

JonChesterfield added inline comments. Comment at: clang/test/Driver/openmp-offload-gpu.c:153 -/// bitcode library and add it to the LIBRARY_PATH. -// RUN: env LIBRARY_PATH=%S/Inputs/libomptarget %clang -### -fopenmp=libomp -fopenmp-targets=nvptx64-nvidia-cuda \ -// RUN: -Xo

[PATCH] D102325: [clang-tidy] cppcoreguidelines-virtual-base-class-destructor: a new check

whisperity added a comment. In D102325#2981760 , @mgartmann wrote: > @whisperity Thanks a lot for helping me out! Sorry I got busy with a few official business and then got deeply distracted by what I'm working on, but I'll get to committing this ASAP.

[clang] 55d9396 - [X86] Move _mm256_set_m128* intrinsics before _mm256_loadu2_m128* intrinsics. NFC.

Author: Simon Pilgrim Date: 2021-09-09T11:23:50+01:00 New Revision: 55d939627823a196f78f0e6279fa0ca14d0ae0f8 URL: https://github.com/llvm/llvm-project/commit/55d939627823a196f78f0e6279fa0ca14d0ae0f8 DIFF: https://github.com/llvm/llvm-project/commit/55d939627823a196f78f0e6279fa0ca14d0ae0f8.diff

[PATCH] D109327: [OpenCL][Docs] Release 13 notes

Anastasia updated this revision to Diff 371536. Anastasia added a comment. - Addressed review comments from @Topotuna - Reordered items to group features and fixes CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109327/new/ https://reviews.llvm.org/D109327 Files: clang/docs/ReleaseNote

[PATCH] D109497: [X86][AVX] Update _mm256_loadu2_m128* intrinsics to use _mm256_set_m128* (PR51796)

RKSimon created this revision. RKSimon added reviewers: craig.topper, pengfei, spatel. RKSimon requested review of this revision. Herald added a project: clang. As reported on PR51796, the _mm256_loadu2_m128i in particular was inserting bitcasts and shuffles with different types making it trickie

[PATCH] D109366: [OpenCL] Tests C++ for OpenCL 2021 version macros

Anastasia added inline comments. Comment at: clang/test/Preprocessor/predefined-macros.c:138 // RUN: | FileCheck -match-full-lines %s --check-prefix=CHECK-FRM -// RUN: %clang_cc1 %s -E -dM -o - -x cl -cl-std=clc++ \ +// RUN: %clang_cc1 %s -E -dM -o - -x cl -cl-std=clc++1.0 \

[PATCH] D109370: [OpenCL] Enables .rgba vector extension in C++ for OpenCL 2021

Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109370/new/ https://reviews.llvm.org/D109370 _

[PATCH] D109498: [clang][deps] Stop using `ClangTool` for virtual files

jansvoboda11 created this revision. jansvoboda11 added reviewers: Bigcheese, dexonsmith, ahatanak. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch changes how the dependency scanner creates the fake input file when

[PATCH] D109424: [OpenCL] Supports atomics in C++ for OpenCL 2021

Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109424/new/ https://reviews.llvm.org/D109424

[PATCH] D109492: [OpenCL] Test case for C++ for OpenCL 2021 in OpenCL C header test

Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109492/new/ https://reviews.llvm.org/D109492 _

[PATCH] D109489: [OptParser] NFC: Remove unused template arg 'name' from bool opt

jansvoboda11 accepted this revision. jansvoboda11 added a comment. This revision is now accepted and ready to land. LGTM. To give more background, I think my intention was to add an assertion that `name` and `other_name` are the same (except for the `no_` part) and I was waiting for TableGen as

[PATCH] D109366: [OpenCL] Tests C++ for OpenCL version macros

Topotuna updated this revision to Diff 371544. Topotuna retitled this revision from "[OpenCL] Tests C++ for OpenCL 2021 version macros" to "[OpenCL] Tests C++ for OpenCL version macros". Topotuna edited the summary of this revision. Topotuna added a comment. Test case added for command line flag

[PATCH] D109144: [SPIR-V] Add SPIR-V triple architecture and clang target info

Anastasia added inline comments. Comment at: clang/test/CodeGenOpenCL/spirv32_target.cl:12 +kernel void foo(global long *arg) { + int res1[sizeof(my_st) == 12 ? 1 : -1]; + int res2[sizeof(void *) == 4 ? 1 : -1]; Are these lines tested somehow? You could chang

[PATCH] D109345: MemoryBuffer: Migrate to Expected/llvm::Error from ErrorOr/std::error_code

dblaikie added a comment. In D109345#2987565 , @dexonsmith wrote: > This seems like the right direction to me! Especially like the > look-through-the-ErrorInfo change for `FileError` -- I hit this before and > found it annoying. Thanks for taking a lo

[PATCH] D109345: MemoryBuffer: Migrate to Expected/llvm::Error from ErrorOr/std::error_code

dexonsmith added a comment. In D109345#2990527 , @dblaikie wrote: > (were there other regressions I mentioned/should think about?) I don't have specific concerns; I was just reading between the lines of your description... >> 1. Add `using MemoryBuffer

[PATCH] D109345: MemoryBuffer: Migrate to Expected/llvm::Error from ErrorOr/std::error_code

dblaikie added a comment. In D109345#2990577 , @dexonsmith wrote: > In D109345#2990527 , @dblaikie > wrote: > >> (were there other regressions I mentioned/should think about?) > > I don't have specific concerns;

[PATCH] D109483: [APInt] Normalize naming on keep constructors / predicate methods.

lattner added a comment. This patch has a lot of noise, the important part is APInt.h Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109483/new/ https://reviews.llvm.org/D109483 ___ cfe-commits mailing li

[PATCH] D109483: [APInt] Normalize naming on keep constructors / predicate methods.

craig.topper added inline comments. Comment at: llvm/include/llvm/ADT/APInt.h:384 /// value for the APInt's bit width. bool isMaxValue() const { return isAllOnesValue(); } isAllOnes()? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION h

[PATCH] D109483: [APInt] Normalize naming on keep constructors / predicate methods.

lattner marked an inline comment as done. lattner added inline comments. Comment at: llvm/include/llvm/ADT/APInt.h:384 /// value for the APInt's bit width. bool isMaxValue() const { return isAllOnesValue(); } craig.topper wrote: > isAllOnes()? Yep, good c

[PATCH] D109483: [APInt] Normalize naming on keep constructors / predicate methods.

craig.topper added a comment. I think I read this patch too closely. I'll leave it up to you how much of this you want to do. Comment at: llvm/include/llvm/IR/Constants.h:206 /// Determine if the value is all ones. bool isMinusOne() const { return Val.isAllOnesValue(); }

[PATCH] D109489: [OptParser] NFC: Remove unused template arg 'name' from bool opt

c-rhodes added a comment. In D109489#2991261 , @jansvoboda11 wrote: > LGTM. > > To give more background, I think my intention was to add an assertion that > `name` and `other_name` are the same (except for the `no_` part) and I was > waiting for TableG

[clang-tools-extra] c58c7a6 - [clang-tidy] cppcoreguidelines-virtual-base-class-destructor: a new check

Author: Marco Gartmann Date: 2021-09-09T13:23:38+02:00 New Revision: c58c7a6ea0535a75e382752b37cb68ccea43cde3 URL: https://github.com/llvm/llvm-project/commit/c58c7a6ea0535a75e382752b37cb68ccea43cde3 DIFF: https://github.com/llvm/llvm-project/commit/c58c7a6ea0535a75e382752b37cb68ccea43cde3.diff

[PATCH] D102325: [clang-tidy] cppcoreguidelines-virtual-base-class-destructor: a new check

This revision was automatically updated to reflect the committed changes. Closed by commit rGc58c7a6ea053: [clang-tidy] cppcoreguidelines-virtual-base-class-destructor: a new check (authored by mgartmann, committed by whisperity). Changed prior to commit: https://reviews.llvm.org/D102325?vs=36

[PATCH] D109460: [clang][Darwin] Try to guess the SDK root with xcrun when unspecified

thakis added a comment. This is a nice idea. One drawback is that if you don't pass `-isysroot`, things just wouldn't work before but now every clang invocation will call xcrun, so things will work but will be slower than if you passed `-isysroot`. But the convenience for one-off invocations i

[PATCH] D106804: [test-suite] Add tests for FP classification intrinsics

sepavloff added a comment. In D106804#2956881 , @RKSimon wrote: > Something I noticed is that we don't have much test coverage in > clang/test/codegen for the fpclass intrinsics - including no constexpr > testing afaict (although I don't think __builtin

[clang] 6c8ff40 - [OptParser] NFC: Remove unused template arg 'name' from bool opt

Author: Cullen Rhodes Date: 2021-09-09T12:04:40Z New Revision: 6c8ff4032e2bcf7dd381633b7e6294f23f0173a9 URL: https://github.com/llvm/llvm-project/commit/6c8ff4032e2bcf7dd381633b7e6294f23f0173a9 DIFF: https://github.com/llvm/llvm-project/commit/6c8ff4032e2bcf7dd381633b7e6294f23f0173a9.diff LOG:

[PATCH] D109489: [OptParser] NFC: Remove unused template arg 'name' from bool opt

This revision was automatically updated to reflect the committed changes. Closed by commit rG6c8ff4032e2b: [OptParser] NFC: Remove unused template arg 'name' from bool opt (authored by c-rhodes). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109489/n

[PATCH] D106804: [test-suite] Add tests for FP classification intrinsics

RKSimon added a comment. I've no objections to this - it seems to be orthogonal to the isnan ir intrinsic conversation. Do we have test-suite buildbot coverage on anything other than x86? Cross platform uses of long double is always fun Any other comments? Repository: rT test-suite

[PATCH] D109366: [OpenCL] Tests C++ for OpenCL version macros

Anastasia added inline comments. Comment at: clang/test/Preprocessor/predefined-macros.c:139 // RUN: %clang_cc1 %s -E -dM -o - -x cl -cl-std=clc++ \ +// RUN: | FileCheck -match-full-lines %s --check-prefix=CHECK-CLCPP +// RUN: %clang_cc1 %s -E -dM -o - -x cl -cl-std=clc++1.0 \

[PATCH] D108526: [clang][nfc] Mark as P0692R1 as implemented.

aaron.ballman accepted this revision. aaron.ballman 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/D108526/new/ https://reviews.llvm.org/D108526

[clang] 7fc743f - Mark as P0692R1 as implemented; NFC

Author: Corentin Jabot Date: 2021-09-09T08:45:47-04:00 New Revision: 7fc743ff84f60b12bb12f47d48ceb6f268106e45 URL: https://github.com/llvm/llvm-project/commit/7fc743ff84f60b12bb12f47d48ceb6f268106e45 DIFF: https://github.com/llvm/llvm-project/commit/7fc743ff84f60b12bb12f47d48ceb6f268106e45.diff

[PATCH] D108526: [clang][nfc] Mark as P0692R1 as implemented.

aaron.ballman closed this revision. aaron.ballman added a comment. I committed this on your behalf in 7fc743ff84f60b12bb12f47d48ceb6f268106e45 , thank you for the patch! Repository: rG LLVM Github Monorepo CHANGES SINCE LA

[PATCH] D108366: [clang][deps] Make resource directory deduction configurable

jansvoboda11 updated this revision to Diff 371568. jansvoboda11 added a comment. Attempt to fix CI again Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108366/new/ https://reviews.llvm.org/D108366 Files: clang/test/ClangScanDeps/Inputs/resource_d

[PATCH] D109061: [openmp] No longer use LIBRARY_PATH to find devicertl

tianshilei1992 added inline comments. Comment at: clang/test/Driver/openmp-offload-gpu.c:153 -/// bitcode library and add it to the LIBRARY_PATH. -// RUN: env LIBRARY_PATH=%S/Inputs/libomptarget %clang -### -fopenmp=libomp -fopenmp-targets=nvptx64-nvidia-cuda \ -// RUN: -Xop

[PATCH] D109498: [clang][deps] Stop using `ClangTool` for virtual files

jansvoboda11 updated this revision to Diff 371570. jansvoboda11 added a comment. Updating diff to re-trigger CI... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109498/new/ https://reviews.llvm.org/D109498 Files: clang/include/clang/Tooling/Depe

[PATCH] D109506: [RFC] Print current request context along with the stack trance in clangd

0x1eaf created this revision. 0x1eaf added reviewers: sammccall, ilya-biryukov, nridge. 0x1eaf added projects: clang, clang-tools-extra. Herald added subscribers: usaxena95, kadircet, arphaman, javed.absar, mgorny. 0x1eaf requested review of this revision. Herald added subscribers: cfe-commits, Mas

[PATCH] D109225: [clang-nvlink-wrapper] Add documentation in clang docs

RKSimon added inline comments. Comment at: clang/docs/ClangNvlinkWrapper.rst:17 +files. It reads each input archive file to extract the archived cubin files as +temporary files. These temporary (*.cubin) files are passed to ``nvlink``. + @saiislam The "(*.cubin)"

[PATCH] D106809: [clang-offload-bundler] Make Bundle Entry ID backward compatible

RKSimon added inline comments. Comment at: clang/test/Driver/clang-offload-bundler.c:405 +// Tests to check compatibility between Bundle Entry ID formats i.e. between presence/absence of extra hyphen in case of missing environment field +// RUN: clang-offload-bundler -unbundle -

[PATCH] D108366: [clang][deps] Make resource directory deduction configurable

jansvoboda11 updated this revision to Diff 371597. jansvoboda11 added a comment. Fix CI pls? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108366/new/ https://reviews.llvm.org/D108366 Files: clang/test/ClangScanDeps/Inputs/resource_directory/cdb

[clang] bb3f5f5 - [clang] Array list initialization (pre-p0388)

Author: Nathan Sidwell Date: 2021-09-09T08:30:04-07:00 New Revision: bb3f5f5d788dd9375ab260f77612fab4a707a1ac URL: https://github.com/llvm/llvm-project/commit/bb3f5f5d788dd9375ab260f77612fab4a707a1ac DIFF: https://github.com/llvm/llvm-project/commit/bb3f5f5d788dd9375ab260f77612fab4a707a1ac.diff

[PATCH] D103088: [clang] pre-0388 array parm list initialization

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 rGbb3f5f5d788d: [clang] Array list initialization (pre-p0388) (authored by urnathan). Herald added a project: clang. Herald

[PATCH] D109061: [openmp] No longer use LIBRARY_PATH to find devicertl

JonChesterfield updated this revision to Diff 371605. JonChesterfield added a comment. - Add test checking LIBRARY_PATH is used Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109061/new/ https://reviews.llvm.org/D109061 Files: clang/lib/Driver/To

[PATCH] D109498: [clang][deps] Stop using `ClangTool` for virtual files

dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land. LGTM, with one nit. Comment at: clang/include/clang/Tooling/DependencyScanning/DependencyScanningWorker.h:98 + /// The in-memory filesystem laid on top the physical

[PATCH] D109061: [openmp] No longer use LIBRARY_PATH to find devicertl

tianshilei1992 accepted this revision. tianshilei1992 added a comment. This revision is now accepted and ready to land. LGTM. It's good to go if the test is green. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109061/new/ https://reviews.llvm.org/D

[PATCH] D108366: [clang][deps] Make resource directory deduction configurable

jansvoboda11 added inline comments. Comment at: clang/test/ClangScanDeps/resource_directory.c:3 +// RUN: cp %S/Inputs/resource_directory/* %t +// RUN: sed -e "s|CLANG|/our/custom/bin/clang|g" -e "s|DIR|%/t|g" %S/Inputs/resource_directory/cdb_tu.json > %t/cdb.json + -

[PATCH] D108366: [clang][deps] Make resource directory deduction configurable

dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land. LGTM. Comment at: clang/test/ClangScanDeps/resource_directory.c:3 +// RUN: cp %S/Inputs/resource_directory/* %t +// RUN: sed -e "s|CLANG|/our/custom/bin/clang|g" -e "

[PATCH] D103088: [clang] pre-0388 array parm list initialization

xbolva00 added a comment. Commited without approval? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103088/new/ https://reviews.llvm.org/D103088 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[clang] 17c2948 - [clang-scan-deps] Add an API for clang dependency scanner to perform

Author: Akira Hatanaka Date: 2021-09-09T08:52:50-07:00 New Revision: 17c2948d04431e94376e8d7883b5d89fbe705b5e URL: https://github.com/llvm/llvm-project/commit/17c2948d04431e94376e8d7883b5d89fbe705b5e DIFF: https://github.com/llvm/llvm-project/commit/17c2948d04431e94376e8d7883b5d89fbe705b5e.diff

[PATCH] D109485: [clang-scan-deps] Add an API for clang dependency scanner to perform module lookup by name alone

This revision was automatically updated to reflect the committed changes. Closed by commit rG17c2948d0443: [clang-scan-deps] Add an API for clang dependency scanner to perform (authored by ahatanak). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D1094

[PATCH] D109061: [openmp] No longer use LIBRARY_PATH to find devicertl

JonChesterfield updated this revision to Diff 371614. JonChesterfield added a comment. - rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109061/new/ https://reviews.llvm.org/D109061 Files: clang/lib/Driver/ToolChains/CommonArgs.cpp clang/

[PATCH] D109517: [Clang][ARM][AArch64] Add support for Armv9-A, Armv9.1-A and Armv9.2-A

vhscampos created this revision. Herald added subscribers: dexonsmith, hiraditya, kristof.beyls. vhscampos requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. armv9-a, armv9.1-a and armv9.2-a can be targeted using the -march o

[PATCH] D108643: Introduce _BitInt, deprecate _ExtInt

aaron.ballman added a reviewer: hjl.tools. aaron.ballman added a subscriber: hjl.tools. aaron.ballman added a comment. In D108643#2965852 , @rjmccall wrote: > In D108643#2964740 , @aaron.ballman > wrote: > >> In

[clang] 2a58171 - [openmp] No longer use LIBRARY_PATH to find devicertl

Author: Jon Chesterfield Date: 2021-09-09T17:16:41+01:00 New Revision: 2a581710c1942b265b271e230368b1596132f242 URL: https://github.com/llvm/llvm-project/commit/2a581710c1942b265b271e230368b1596132f242 DIFF: https://github.com/llvm/llvm-project/commit/2a581710c1942b265b271e230368b1596132f242.di

[PATCH] D109061: [openmp] No longer use LIBRARY_PATH to find devicertl

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2a581710c194: [openmp] No longer use LIBRARY_PATH to find devicertl (authored by JonChesterfield). Repository: rG LLVM Github Monorepo CHANGES SI

[PATCH] D109497: [X86][AVX] Update _mm256_loadu2_m128* intrinsics to use _mm256_set_m128* (PR51796)

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

[PATCH] D104285: [analyzer] Retrieve a value from list initialization of constant array declaration in a global scope.

martong added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/RegionStore.cpp:1692-1694 +const bool IsOneDimensionalArray = +!isa(CAT->getElementType()); +if (IsOneDimensionalArray) { aaron.ballman wrote: > +1 f

[PATCH] D108643: Introduce _BitInt, deprecate _ExtInt

hjl.tools added a comment. >> The choice that high bits are unspecified rather than extended is an >> interesting one. Can you speak to that? That's good for +, -, *, &, |, ^, >> <<, and narrowing conversions, but bad for ==, <, /, >>, and widening >> conversions. > > I've added @hjl.tools to

[PATCH] D109483: [APInt] Normalize naming on keep constructors / predicate methods.

lattner added inline comments. Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp:3243 "Don't know how to expand this subtraction!"); -Tmp1 = DAG.getNode(ISD::XOR, dl, VT, Node->getOperand(1), - DAG.getConstant(APInt::getAllOnesValue(VT.getSiz

[PATCH] D109517: [Clang][ARM][AArch64] Add support for Armv9-A, Armv9.1-A and Armv9.2-A

SjoerdMeijer added reviewers: t.p.northover, ab. SjoerdMeijer added a comment. Some first comments after just looking at the plumbing for these new options. Didn't check yet the architecture extensions for the different version. Comment at: clang/lib/Driver/ToolChains/Arch/AAr

[PATCH] D109526: [OpenCL][Docs] Added ref to libclcxx

Anastasia created this revision. Anastasia added a reviewer: olestrohm. Herald added subscribers: ebevhan, yaxunl. Anastasia requested review of this revision. Linked in libclcxx GitHub project page with C++ libraries for OpenCL on OpenCLSupport page. https://reviews.llvm.org/D109526 Files:

[PATCH] D109483: [APInt] Normalize naming on keep constructors / predicate methods.

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

[PATCH] D109483: [APInt] Normalize naming on keep constructors / predicate methods.

lattner added a comment. Thank you for the detailed review Craig! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109483/new/ https://reviews.llvm.org/D109483 ___ cfe-commits mailing list cfe-commits@lists

[PATCH] D109483: [APInt] Normalize naming on keep constructors / predicate methods.

lattner added a comment. I'll take care of the DAG.getAllOnesConstant change, but i'd appreciate it if you could look at the NOT cases. Running tests on the DAG.getAllOnesConstant patch now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109483/ne

[PATCH] D109526: [OpenCL][Docs] Added ref to libclcxx

olestrohm accepted this revision. olestrohm added a comment. This revision is now accepted and ready to land. Looks good! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109526/new/ https://reviews.llvm.org/D109526 ___ cfe-commits mailing list

[PATCH] D109470: Add "profiling" to the list of absl libraries.

ymandel accepted this revision. ymandel added a comment. This revision is now accepted and ready to land. Any idea if there's a test for this matcher that should be updated? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109470/new/ https://reviews.

[PATCH] D109470: Add "profiling" to the list of absl libraries.

nilayvaish added a comment. In D109470#2992119 , @ymandel wrote: > Any idea if there's a test for this matcher that should be updated? I looked at the commit that most recently added a directory to this file: 114c9fa0e46f7bf1d05d92da70da116b19f16911

[PATCH] D109506: [RFC] Print current request context along with the stack trance in clangd

sammccall added a comment. Thanks for doing this, I think this is incredibly useful for debugging. But also subtle, so please forgive a bunch of comments! I don't think we're strictly in defined-behavior territory in much of what these signal handlers are doing, but neither is clang's existing c

[PATCH] D109470: Add "profiling" to the list of absl libraries.

ymandel added a comment. In D109470#2992161 , @nilayvaish wrote: > In D109470#2992119 , @ymandel wrote: > >> Any idea if there's a test for this matcher that should be updated? > > I looked at the commit that mos

[PATCH] D109506: [RFC] Print current request context along with the stack trance in clangd

sammccall added a comment. Oops, forgot one thing: you probably want to instrument the preamble-building in TUScheduler (or in Preamble.cpp?), the background indexing, and code completion. Those all run the clang parser and should be a rich source of clang bugs. Testing idea: `yes '[' | head -

[PATCH] D108592: [clang][Fuchsia] Support __attribute__((availability)) on Fuchsia

phosek added inline comments. Comment at: clang/include/clang/Basic/LangOptions.def:431 +VALUE_LANGOPT(FuchsiaVersion, 32, 0, "Fuchsia Version") + This is more consistent with other options. Comment at: clang/lib/Basic/Targets/OSTargets.h:888

[PATCH] D109470: Add "profiling" to the list of absl libraries.

nilayvaish added a comment. ymandel@, I do not have commit access to the repo. Can you commit this for me? Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109470/new/ https://reviews.llvm.org/D109470 __

[clang] ea685e1 - [X86][AVX] Update _mm256_loadu2_m128* intrinsics to use _mm256_set_m128* (PR51796)

Author: Simon Pilgrim Date: 2021-09-09T19:15:48+01:00 New Revision: ea685e1028c6fe9e2b0f9eb5858bcb867f75bdc8 URL: https://github.com/llvm/llvm-project/commit/ea685e1028c6fe9e2b0f9eb5858bcb867f75bdc8 DIFF: https://github.com/llvm/llvm-project/commit/ea685e1028c6fe9e2b0f9eb5858bcb867f75bdc8.diff

[PATCH] D109497: [X86][AVX] Update _mm256_loadu2_m128* intrinsics to use _mm256_set_m128* (PR51796)

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGea685e1028c6: [X86][AVX] Update _mm256_loadu2_m128* intrinsics to use _mm256_set_m128*… (authored by RKSimon). Repository: rG LLVM Github Monorepo

[PATCH] D109345: MemoryBuffer: Migrate to Expected/llvm::Error from ErrorOr/std::error_code

dexonsmith added a comment. In D109345#2992274 , @dexonsmith wrote: > 4. One or more commits: > 1. Migrate in-tree callers to MemoryBuffer. > 2. Delete MemoryBufferErrorAPI alias. > 5. Delete MemoryBufferErrorCodeAPI wrappers. (Potentially MemoryBuf

[PATCH] D109345: MemoryBuffer: Migrate to Expected/llvm::Error from ErrorOr/std::error_code

dexonsmith added a comment. In D109345#2990612 , @dblaikie wrote: > Given the amount of churn this means, though - reckon it's worth it? Reckon > it needs more llvm-dev thread/buy-in/etc? I think the churn is worth since my intuition is that it has high

[clang] 543604f - [clang-nvlink-wrapper][docs][NFC] Fix sphinx warning about asterisk

Author: Saiyedul Islam Date: 2021-09-09T23:55:15+05:30 New Revision: 543604f30eddc5c9390d0fb01b0ac67937cbba0e URL: https://github.com/llvm/llvm-project/commit/543604f30eddc5c9390d0fb01b0ac67937cbba0e DIFF: https://github.com/llvm/llvm-project/commit/543604f30eddc5c9390d0fb01b0ac67937cbba0e.diff

[PATCH] D109225: [clang-nvlink-wrapper] Add documentation in clang docs

saiislam marked an inline comment as done. saiislam added inline comments. Comment at: clang/docs/ClangNvlinkWrapper.rst:17 +files. It reads each input archive file to extract the archived cubin files as +temporary files. These temporary (*.cubin) files are passed to ``nvlink``.

[PATCH] D109531: [CSSPGO] Enable pseudo probe instrumentation in O0 mode.

hoy created this revision. Herald added subscribers: modimo, wenlei, hiraditya. hoy requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Pseudo probe instrumentation was missing from O0 build. It is needed in cases where some

[PATCH] D109225: [clang-nvlink-wrapper] Add documentation in clang docs

RKSimon added inline comments. Comment at: clang/docs/ClangNvlinkWrapper.rst:17 +files. It reads each input archive file to extract the archived cubin files as +temporary files. These temporary (*.cubin) files are passed to ``nvlink``. + saiislam wrote: > RKSimon

[PATCH] D108592: [clang][Fuchsia] Support __attribute__((availability)) on Fuchsia

aaron.ballman added inline comments. Comment at: clang/test/Frontend/attr-availability-fuchsia.c:2 +// Test that `-mfuchsia-version` is propagated to cc1. +// RUN: %clang -target x86_64-unknown-fuchsia -mfuchsia-version=16 -c %s -### |& FileCheck %s +// Is `|&`

[PATCH] D109470: Add "profiling" to the list of absl libraries.

ymandel added a comment. In D109470#2992275 , @nilayvaish wrote: > ymandel@, I do not have commit access to the repo. Can you commit this for > me? Thanks! Sure! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llv

[PATCH] D109531: [CSSPGO] Enable pseudo probe instrumentation in O0 mode.

wlei accepted this revision. wlei added a comment. This revision is now accepted and ready to land. LGTM, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109531/new/ https://reviews.llvm.org/D109531 __

[PATCH] D109315: [clang] Check unsupported types in expressions

asavonic updated this revision to Diff 371693. asavonic added a comment. - Reworded the diagnostic. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109315/new/ https://reviews.llvm.org/D109315 Files: clang/include/clang/Basic/DiagnosticSemaKinds.t

[PATCH] D106674: Runtime for Interop directive

jdoerfert added a comment. We need a runtime test. Synchronizing the asyn info object and signal out dependences will happen in the next revision. So will minor edits as requested. I commented below on some things. Comment at: openmp/libomptarget/src/interop.cpp:13 +static o

[PATCH] D105876: OMPIRBuilder for Interop directive

jdoerfert accepted this revision. jdoerfert added a comment. This revision is now accepted and ready to land. LGTM, please rebase on top of trunk and use ` ConstantInt::get(Int32, 0)` instead of the APInt way. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.l

[PATCH] D109496: [clang] deprecate frelaxed-template-template-args, make it on by default

mizvekov created this revision. Herald added subscribers: dexonsmith, dang. mizvekov updated this revision to Diff 371538. mizvekov edited the summary of this revision. mizvekov added a comment. mizvekov updated this revision to Diff 371539. mizvekov edited the summary of this revision. mizvekov up

[PATCH] D109541: Increase expected line number for ExtDebugInfo.cpp

Jake-Egan created this revision. Jake-Egan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D109541 Files: clang/test/Modules/ExtDebugInfo.cpp Index: clang/test/Modules/Ext

[clang-tools-extra] e976fc6 - Add "profiling" to the list of absl libraries.

Author: Nilay Vaish Date: 2021-09-09T20:31:06Z New Revision: e976fc61ecd9c789783129c1294e096f8b4b11b0 URL: https://github.com/llvm/llvm-project/commit/e976fc61ecd9c789783129c1294e096f8b4b11b0 DIFF: https://github.com/llvm/llvm-project/commit/e976fc61ecd9c789783129c1294e096f8b4b11b0.diff LOG: A

[PATCH] D109470: Add "profiling" to the list of absl libraries.

This revision was automatically updated to reflect the committed changes. Closed by commit rGe976fc61ecd9: Add "profiling" to the list of absl libraries. (authored by nilayvaish, committed by ymandel). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D10

[PATCH] D109544: [OpenMP] Add flag for setting debug in the offloading device

jhuber6 created this revision. jhuber6 added a reviewer: jdoerfert. Herald added subscribers: dexonsmith, dang, guansong, hiraditya, yaxunl. jhuber6 requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits, sstefan1. Herald added projects: clang, LLVM. This patch intr

[PATCH] D109544: [OpenMP] Add flag for setting debug in the offloading device

jdoerfert added inline comments. Comment at: clang/lib/Frontend/CompilerInvocation.cpp:3895 + Opts.OpenMPTargetDebug = 1; + } + CUDANumSMs? Don't check for NVPTX/AMDGCN but only if the new runtime is used, if not emit a warning that the flag is useless a

[PATCH] D109531: [CSSPGO] Enable pseudo probe instrumentation in O0 mode.

wenlei added a comment. The change makes sense given instr PGO also happens for O0. But practically, if a file is being built with O0, do we care about its profile given we're not really optimizing it anyways? Functions from O0 modules are not supposed to be inlined into O1

[PATCH] D109411: [clang] Enable the special enable_if_t diagnostics for libc++'s __enable_if_t as well.

Quuxplusone added a comment. In D109411#2990494 , @dblaikie wrote: > Looks like most of the testing for the enable_if custom dialog was added in > `clang/test/SemaTemplate/overload-candidates.cpp` in > rG6f8d2c6c9c3451effdf075a7034bbe77045bfeba >

  1   2   >