[clang-tools-extra] 39f779a - [clang-tidy][NFC] Remove redundant string creation for comparison

Author: Nathan James Date: 2022-01-20T22:20:10Z New Revision: 39f779afb35420c51e2c7094b2643a53baed9f59 URL: https://github.com/llvm/llvm-project/commit/39f779afb35420c51e2c7094b2643a53baed9f59 DIFF: https://github.com/llvm/llvm-project/commit/39f779afb35420c51e2c7094b2643a53baed9f59.diff LOG:

[PATCH] D101070: Make `llvm_install_symlink` robust with respect to absolute dirs.

Ericson2314 updated this revision to Diff 401784. Ericson2314 added a comment. Much simpler now Code taken from D99484 and then to D117419 and now to here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION ht

[clang] c95afac - [clang-format][NFC] Clean up tryMergeLessLess()

Author: owenca Date: 2022-01-20T14:35:07-08:00 New Revision: c95afac89e000b65edc51dc7d05610250a62c86f URL: https://github.com/llvm/llvm-project/commit/c95afac89e000b65edc51dc7d05610250a62c86f DIFF: https://github.com/llvm/llvm-project/commit/c95afac89e000b65edc51dc7d05610250a62c86f.diff LOG: [

[PATCH] D117759: [clang-format][NFC] Clean up tryMergeLessLess()

This revision was automatically updated to reflect the committed changes. owenpan marked an inline comment as done. Closed by commit rGc95afac89e00: [clang-format][NFC] Clean up tryMergeLessLess() (authored by owenpan). Changed prior to commit: https://reviews.llvm.org/D117759?vs=401579&id=4017

[PATCH] D117840: [clang-tidy] Update bugprone-stringview-nullptr to consistently prefer the empty string when passing arguments to constructors/functions

CJ-Johnson created this revision. Herald added subscribers: carlosgalvezp, xazax.hun. CJ-Johnson requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. Previously, function(nullptr) would have been fixed with function({}). This unfort

[clang-tools-extra] a568411 - [clang-tidy] Update bugprone-stringview-nullptr to consistently prefer the empty string when passing arguments to constructors/functions

Author: CJ Johnson Date: 2022-01-20T18:08:40-05:00 New Revision: a5684114445a72b5c0bb5b7b68a5c6eb3486b66d URL: https://github.com/llvm/llvm-project/commit/a5684114445a72b5c0bb5b7b68a5c6eb3486b66d DIFF: https://github.com/llvm/llvm-project/commit/a5684114445a72b5c0bb5b7b68a5c6eb3486b66d.diff LO

[PATCH] D117840: [clang-tidy] Update bugprone-stringview-nullptr to consistently prefer the empty string when passing arguments to constructors/functions

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa5684114445a: [clang-tidy] Update bugprone-stringview-nullptr to c

[PATCH] D113832: reland: [VFS] Use original path when falling back to external FS

bnbarham added a comment. @keith This patch causes some issues when nesting/chaining `RedirectingFileSystems`. I have a few tests in https://reviews.llvm.org/D117730 for more details. Feedback on that PR would be appreciated, I'm not sure if it's the direction we want to go or not. Repositor

[PATCH] D117730: [DNM][VFS] Do not overwrite the path when nesting RedirectingFileSystems

bnbarham added a comment. Also have bug up at https://github.com/llvm/llvm-project/issues/53306. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117730/new/ https://reviews.llvm.org/D117730 ___ cfe-commits

[PATCH] D117569: Constexpr not supported with __declspec(dllimport).

zahiraam added a comment. In D117569#3259255 , @majnemer wrote: > In D117569#3258307 , @zahiraam > wrote: > >> In D117569#3257121 , @majnemer >> wrote: >> >>> I have a q

[PATCH] D117753: [AArch64] Support for memset tagged intrinsic

tyb0807 marked 3 inline comments as done. tyb0807 added inline comments. Comment at: clang/lib/Headers/arm_acle.h:734 +/* Memory Operations Intrinsics */ +#if __ARM_FEATURE_MOPS && __ARM_FEATURE_MEMORY_TAGGING +#define __arm_mops_memset_tag(tagged_address, value, size)

[clang] e1b7bd9 - [OpenCL] opencl-c.h: add __opencl_c_images and __opencl_c_read_write_images

Author: Dave Airlie Date: 2022-01-21T09:51:01+10:00 New Revision: e1b7bd911d9e491e50db1aa00d340fb3b6b907ef URL: https://github.com/llvm/llvm-project/commit/e1b7bd911d9e491e50db1aa00d340fb3b6b907ef DIFF: https://github.com/llvm/llvm-project/commit/e1b7bd911d9e491e50db1aa00d340fb3b6b907ef.diff L

[PATCH] D107539: [OpenCL] opencl-c.h: add __opencl_c_images and __opencl_c_read_write_images

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGe1b7bd911d9e: [OpenCL] opencl-c.h: add __opencl_c_images and __opencl_c_read_write_images (authored by airlied). Repository: rG LLVM Github Monor

[PATCH] D117569: Constexpr not supported with __declspec(dllimport).

majnemer added a comment. Your example is different from mine as it nests the constexpr variable inside the function rather than having it at translation-unit scope. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117569/new/ https://reviews.llvm.org/D117569

[PATCH] D117806: [OpenMP] Change default visibility to protected for device declarations

JonChesterfield added inline comments. Comment at: clang/lib/Driver/ToolChains/Clang.cpp:5830 + // host, makes the system more robust, and improves performance. + if (IsOpenMPDevice) { +CmdArgs.push_back("-fvisibility"); I think we need some more code here

[clang-tools-extra] d3b188a - [clang-tidy] Include constructor initializers in `bugprone-exception-escape` check

Author: Fabian Wolff Date: 2022-01-21T00:53:57+01:00 New Revision: d3b188a2d72f9c398e4b1a36d23888c4ac783e9f URL: https://github.com/llvm/llvm-project/commit/d3b188a2d72f9c398e4b1a36d23888c4ac783e9f DIFF: https://github.com/llvm/llvm-project/commit/d3b188a2d72f9c398e4b1a36d23888c4ac783e9f.diff

[PATCH] D113507: [clang-tidy] Include constructor initializers in `bugprone-exception-escape` check

This revision was automatically updated to reflect the committed changes. fwolff marked an inline comment as done. Closed by commit rGd3b188a2d72f: [clang-tidy] Include constructor initializers in `bugprone-exception-escape`… (authored by fwolff). Repository: rG LLVM Github Monorepo CHANGES SI

[PATCH] D117806: [OpenMP] Change default visibility to protected for device declarations

jhuber6 updated this revision to Diff 401811. jhuber6 added a comment. Forgot to make this mutually exclusive with user defined visibility value. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117806/new/ https://reviews.llvm.org/D117806 Files: c

[PATCH] D117806: [OpenMP] Change default visibility to protected for device declarations

jhuber6 marked an inline comment as done. jhuber6 added inline comments. Comment at: clang/lib/Driver/ToolChains/Clang.cpp:5830 + // host, makes the system more robust, and improves performance. + if (IsOpenMPDevice) { +CmdArgs.push_back("-fvisibility"); Jo

[PATCH] D117806: [OpenMP] Change default visibility to protected for device declarations

JonChesterfield accepted this revision. JonChesterfield added a comment. LG, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117806/new/ https://reviews.llvm.org/D117806 ___ cfe-commits mailing lis

[PATCH] D117137: [Driver] Add CUDA support for --offload param

dcastagna updated this revision to Diff 401813. dcastagna added a comment. Fix invalid-offload-options.cpp test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117137/new/ https://reviews.llvm.org/D117137 Files: clang/include/clang/Basic/Diagnosti

[PATCH] D117829: [Clang] Add integer add/mul reduction builtins

craig.topper added a comment. In D117829#3259588 , @RKSimon wrote: > I should mention - according to > https://clang.llvm.org/docs/LanguageExtensions.html `__builtin_reduce_add()` > already exists, which I don't think is true. Does that documentation n

[PATCH] D117846: [ASTMatchers] Add `isConstinit` matcher

Izaron created this revision. Izaron added reviewers: aaron.ballman, cor3ntin. Izaron requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Support C++20 constinit variables for AST Matchers. Repository: rG LLVM Github Monorepo https://revie

[PATCH] D117846: [ASTMatchers] Add `isConstinit` matcher

Izaron added a comment. I'm going to use this matcher very soon in my new `clang-tidy` check. If you're interested and want to read more, I have a design document about my ne check - here (gist.github.com)

[PATCH] D117846: [ASTMatchers] Add `isConstinit` matcher

Izaron updated this revision to Diff 401817. Izaron added a comment. Removed the brackets in `{ return }` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117846/new/ https://reviews.llvm.org/D117846 Files: clang/docs/LibASTMatchersReference.html

[PATCH] D117791: [X86] Remove __builtin_ia32_pabs intrinsics and use generic __builtin_elementwise_abs

pengfei accepted this revision. pengfei added inline comments. Comment at: clang/lib/Headers/avx512fintrin.h:31 + * appear in the interface though. */ +typedef signed char __v64qs __attribute__((__vector_size__(64))); + Do we need to declare explicit `signed`?

[PATCH] D117791: [X86] Remove __builtin_ia32_pabs intrinsics and use generic __builtin_elementwise_abs

craig.topper added inline comments. Comment at: clang/lib/Headers/avx512fintrin.h:31 + * appear in the interface though. */ +typedef signed char __v64qs __attribute__((__vector_size__(64))); + pengfei wrote: > Do we need to declare explicit `signed`? The signedne

[PATCH] D117791: [X86] Remove __builtin_ia32_pabs intrinsics and use generic __builtin_elementwise_abs

pengfei added inline comments. Comment at: clang/lib/Headers/avx512fintrin.h:31 + * appear in the interface though. */ +typedef signed char __v64qs __attribute__((__vector_size__(64))); + craig.topper wrote: > pengfei wrote: > > Do we need to declare explicit `si

[PATCH] D117181: [PowerPC] Use IEEE long double in proper toolchain

jsji added inline comments. Comment at: clang/lib/Driver/ToolChains/PPCLinux.cpp:76 + if (SupportIEEEFloat128(getDriver(), getTriple(), DriverArgs) && + !DriverArgs.hasArg(options::OPT_mabi_EQ)) +CC1Args.push_back("-mabi=ieeelongdouble"); What if we hav

[PATCH] D117798: [X86] Remove __builtin_ia32_pmax/min intrinsics and use generic __builtin_elementwise_max/min

pengfei accepted this revision. pengfei added inline comments. Comment at: clang/lib/Headers/avx512bwintrin.h:894 return (__m512i)__builtin_ia32_selectb_512((__mmask64)__M, (__v64qi)_mm512_min_epu8(__A, __B),

[PATCH] D117181: [PowerPC] Use IEEE long double in proper toolchain

jsji added a comment. Can we just update `bool IEEELongDouble = false; ` to `bool IEEELongDouble = true;` in `clang/lib/Driver/ToolChains/Clang.cpp`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117181/new/ https://reviews.llvm.org/D117181 _

[clang] 0dfe953 - [OpenMP] Change default visibility to protected for device declarations

Author: Joseph Huber Date: 2022-01-20T21:06:26-05:00 New Revision: 0dfe953294ba1c0fc43fb710518b9b5a608b223a URL: https://github.com/llvm/llvm-project/commit/0dfe953294ba1c0fc43fb710518b9b5a608b223a DIFF: https://github.com/llvm/llvm-project/commit/0dfe953294ba1c0fc43fb710518b9b5a608b223a.diff

[PATCH] D117806: [OpenMP] Change default visibility to protected for device declarations

This revision was automatically updated to reflect the committed changes. jhuber6 marked an inline comment as done. Closed by commit rG0dfe953294ba: [OpenMP] Change default visibility to protected for device declarations (authored by jhuber6). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D101070: [llvm][cmake] Make `llvm_install_symlink` robust to absolute dirs.

Ericson2314 added a comment. Continuing my bisect of D99484 (since this change was in there at the time it was approved, but failing) I will land this one next. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101070

[PATCH] D108189: [RISCV] Support experimental 'P' extension 0.96

Jim added a comment. Herald added a subscriber: eopXD. In D108189#3188343 , @zixuan-wu wrote: > Hi, @Jim. > What time is P extension going to upstream to community since it's been no > update for some months? Hi @zixuan-wu, From RVP 0.9.7, it has lots

[PATCH] D101070: [llvm][cmake] Make `llvm_install_symlink` robust to absolute dirs.

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 rG58580e922a69: [llvm][cmake] Make `llvm_install_symlink` robust to absolute dirs. (authored by Ericson2314). Repository:

[PATCH] D117569: Constexpr not supported with __declspec(dllimport).

zahiraam added a comment. In D117569#3259928 , @majnemer wrote: > Your example is different from mine as it nests the constexpr variable inside > the function rather than having it at translation-unit scope. And I suppose we are interested in the latter

[PATCH] D7982: [clang-tidy] Add readability-duplicate-include check

LegalizeAdulthood updated this revision to Diff 401837. LegalizeAdulthood added a comment. - Properly apply fixes to header files - Add tests for fixes in header files - Move function to static outside anonymous namespace - Improve documentation CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D7982: [clang-tidy] Add readability-duplicate-include check

LegalizeAdulthood added a comment. @carlosgalvezp I believe I've addressed your comments now. Thanks for the review, it definitely improved this check. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D7982/new/ https://reviews.llvm.org/D7982 ___

[PATCH] D101070: [llvm][cmake] Make `llvm_install_symlink` robust to absolute dirs.

Ericson2314 reopened this revision. Ericson2314 added a comment. This seems to be the part of D99484 that wasn't working. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101070/new/ https://reviews.llvm.org/D101070

[PATCH] D117419: [clang][cmake] Use `GNUInstallDirs` to support custom installation dirs

Ericson2314 updated this revision to Diff 401843. Ericson2314 added a comment. Remove the LLVM part, which is now D101070 , and furthermore known to cause issues. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D11741

[clang] 82af950 - [X86] Enable ibt-seal optimization when LTO is used in Kernel

Author: Joao Moreira Date: 2022-01-21T10:55:34+08:00 New Revision: 82af95029ec947fed8b9c516f04d4f217bd87930 URL: https://github.com/llvm/llvm-project/commit/82af95029ec947fed8b9c516f04d4f217bd87930 DIFF: https://github.com/llvm/llvm-project/commit/82af95029ec947fed8b9c516f04d4f217bd87930.diff

[PATCH] D116070: [X86] Enable ibt-seal optimization when LTO is used in Kernel

This revision was automatically updated to reflect the committed changes. Closed by commit rG82af95029ec9: [X86] Enable ibt-seal optimization when LTO is used in Kernel (authored by joaomoreira, committed by pengfei). Herald added a project: clang. Herald added a subscriber: cfe-commits. Reposito

[PATCH] D117626: [ADT] [NFC] Add StringRef::detectEOL

dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land. Sounds good to me Comment at: llvm/unittests/ADT/StringRefTest.cpp:1114-1116 + for (const auto &Entry : Cases) { +EXPECT_EQ(StringRef("\n"), Entry.detectEOL()); + }

[PATCH] D117753: [AArch64] Support for memset tagged intrinsic

tyb0807 updated this revision to Diff 401851. tyb0807 marked 2 inline comments as done. tyb0807 added a comment. `__ARM_FEATURE_MEMORY_TAGGING` not needed to enable `__builtin_arm_mops_memset_tag`. Follow variable naming convention. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACT

[PATCH] D112987: [RISCV] Bump rvv-related extensions from 0.10 to 1.0

eopXD updated this revision to Diff 401861. eopXD added a comment. Rebase. Resolve conflict due to zvlsseg removal. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112987/new/ https://reviews.llvm.org/D112987 Files: clang/test/Driver/riscv-arch.c

[PATCH] D112986: [Clang][RISCV] Restrict rvv builtin-s with zve macro-s

eopXD updated this revision to Diff 401862. eopXD marked 2 inline comments as done. eopXD added a comment. Rebase and addres comments. Resolve conflcits due to zvlsseg removal. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112986/new/ https://revie

[PATCH] D112986: [Clang][RISCV] Restrict rvv builtin-s with zve macro-s

eopXD added inline comments. Comment at: clang/utils/TableGen/RISCVVEmitter.cpp:149 RV64 = 1 << 5, + VectorMaxELen32 = 1 << 6, + VectorMaxELen64 = 1 << 7, craig.topper wrote: > Do we need VectorMaxELen32 isn't that the minimum? Yes you are correct. We don't

[clang-tools-extra] a7f8aea - [clang-tidy] Fix wrong FixIt in performance-move-const-arg

Author: Sockke Date: 2022-01-21T14:23:52+08:00 New Revision: a7f8aea71485c00db2ffb6288ae58475869048b1 URL: https://github.com/llvm/llvm-project/commit/a7f8aea71485c00db2ffb6288ae58475869048b1 DIFF: https://github.com/llvm/llvm-project/commit/a7f8aea71485c00db2ffb6288ae58475869048b1.diff LOG: [

[PATCH] D107450: [clang-tidy] Fix wrong FixIt in performance-move-const-arg

This revision was automatically updated to reflect the committed changes. Closed by commit rGa7f8aea71485: [clang-tidy] Fix wrong FixIt in performance-move-const-arg (authored by Sockke). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107450/new/ htt

[PATCH] D107450: [clang-tidy] Fix wrong FixIt in performance-move-const-arg

Sockke added a comment. In D107450#3257877 , @aaron.ballman wrote: > Do you need someone to commit on your behalf? Thanks for your review, I committed it myself. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.o

[PATCH] D112986: [Clang][RISCV] Restrict rvv builtin-s with zve macro-s

eopXD updated this revision to Diff 401870. eopXD added a comment. Follow clang-format. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112986/new/ https://reviews.llvm.org/D112986 Files: clang/utils/TableGen/RISCVVEmitter.cpp Index: clang/utils/

[clang] e6de53b - [RISCV] Bump rvv-related extensions from 0.10 to 1.0

Author: eopXD Date: 2022-01-20T23:22:20-08:00 New Revision: e6de53b4de4aecca4ac892500a0907805896ed27 URL: https://github.com/llvm/llvm-project/commit/e6de53b4de4aecca4ac892500a0907805896ed27 DIFF: https://github.com/llvm/llvm-project/commit/e6de53b4de4aecca4ac892500a0907805896ed27.diff LOG: [R

[PATCH] D112987: [RISCV] Bump rvv-related extensions from 0.10 to 1.0

This revision was automatically updated to reflect the committed changes. Closed by commit rGe6de53b4de4a: [RISCV] Bump rvv-related extensions from 0.10 to 1.0 (authored by eopXD). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112987/new/ https://re

[PATCH] D117846: [ASTMatchers] Add `isConstinit` matcher

Izaron updated this revision to Diff 401872. Izaron added a comment. Fix clang-format Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117846/new/ https://reviews.llvm.org/D117846 Files: clang/docs/LibASTMatchersReference.html clang/docs/ReleaseN

[PATCH] D7982: [clang-tidy] Add readability-duplicate-include check

carlosgalvezp accepted this revision. carlosgalvezp added a comment. LGTM! Had some nits that can be fixed without review. Can't see anything else major that needs change. As said I'm fairly new here so probably a more experienced reviewer can find some more improvements/optimizations, especial

[PATCH] D117520: [clang-format] Fix SeparateDefinitionBlocks issues

curdeius added a comment. In D117520#3258454 , @MyDeveloperDay wrote: > I think its been fixed, in the last but one diff. > > Generally speaking we simply cannot have --output-replacements-xml , -n or > --dry-run show replacements that amount to nothing

<    1   2   3