[clang] [llvm] IR: Store the default subtarget info in named metadata instead of the context. (PR #98673)

2024-07-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: None (pcc) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/98673.diff 8 Files Affected: - (modified) clang/lib/CodeGen/BackendUtil.cpp (+2) - (modified) clang/lib/CodeGen/CodeGenAction.cpp (-6) - (modified) llvm/incl

[clang] [llvm] IR: Store the default subtarget info in named metadata instead of the context. (PR #98673)

2024-07-12 Thread via cfe-commits
pcc wrote: This is a WIP. Because I'm adding new metadata, several tests fail, e.g. check-clang: ``` Testing Time: 244.38s Total Discovered Tests: 37911 Skipped :31 (0.08%) Unsupported : 102 (0.27%) Passed : 37689 (99.41%) Expectedly Failed:28 (0.07%)

[clang] [llvm] IR: Store the default subtarget info in named metadata instead of the context. (PR #98673)

2024-07-12 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 3a744283f4c56b57adb2c381c0aeaf7faf5120ec 6976f169ec1befd902d0a0d0cc05b375d6ae094b --e

[clang] Fix assertion failure during operator overload resolution. (PR #98671)

2024-07-12 Thread Daniel M. Katz via cfe-commits
https://github.com/katzdm ready_for_review https://github.com/llvm/llvm-project/pull/98671 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Fix assertion failure during conversion function overload resolution. (PR #98671)

2024-07-12 Thread Daniel M. Katz via cfe-commits
https://github.com/katzdm edited https://github.com/llvm/llvm-project/pull/98671 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Fix assertion failure during conversion function overload resolution. (PR #98671)

2024-07-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Daniel M. Katz (katzdm) Changes When clang is built with assertions, an otherwise silent (and seemingly innocuous) assertion failure from `SemaConcept.cpp` is triggered by the following program: ```cpp struct S { operator int(); temp

[clang] Reset PostAllocaInsertPt when the AllocaInsertPt is changed in clang::CodeGenFunction (PR #98668)

2024-07-12 Thread Eli Friedman via cfe-commits
efriedma-quic wrote: Does this affect the generated IR in some existing case? If it does, can you include a regression test in clang/test/CodeGen? https://github.com/llvm/llvm-project/pull/98668 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [llvm] IR: Store the default subtarget info in named metadata instead of the context. (PR #98673)

2024-07-12 Thread Fangrui Song via cfe-commits
MaskRay wrote: > This is a WIP. Because I'm adding new metadata, several tests fail, e.g. > check-clang: > > ``` > Testing Time: 244.38s > > Total Discovered Tests: 37911 > Skipped :31 (0.08%) > Unsupported : 102 (0.27%) > Passed : 37689 (99.41%) > Expecte

[clang] Reset PostAllocaInsertPt when the AllocaInsertPt is changed in clang::CodeGenFunction (PR #98668)

2024-07-12 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 6cea40400df542a1a4a6d35b45cbe3367f2c32b7 db22886b15f6bcab77b41846742f07639544bbe7 --e

[clang] The pragma STDC CX_LIMITED_RANGE ON should have precedence. (PR #98520)

2024-07-12 Thread Andy Kaylor via cfe-commits
@@ -218,28 +214,31 @@ _Complex float pragma_default_div(_Complex float a, _Complex float b) { // IMPRVD-NEXT: fsub float // IMPRVD-NEXT: fdiv float - // PRMTD: load float, ptr {{.*}} - // PRMTD: fpext float {{.*}} to double - // PRMTD-NEXT: fpext float {{.*}} to double

[clang] [Driver] Ensure ToolChain::LibraryPaths is not empty for non-Darwin (PR #87866)

2024-07-12 Thread Vitaly Buka via cfe-commits
vitalybuka wrote: Also https://lab.llvm.org/buildbot/#/builders/66/builds/1427/steps/13/logs/stdio ``` [20/21] Running compiler_rt regression tests llvm-lit: /b/sanitizer-x86_64-linux/build/llvm-project/compiler-rt/test/lit.common.cfg.py:60: warning: Path reported by clang does not exist: "/b

[clang] [llvm] [SystemZ][z/OS] Update autoconversion functions to improve support for UTF-8 (PR #98652)

2024-07-12 Thread via cfe-commits
https://github.com/cor3ntin edited https://github.com/llvm/llvm-project/pull/98652 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [SystemZ][z/OS] Update autoconversion functions to improve support for UTF-8 (PR #98652)

2024-07-12 Thread via cfe-commits
https://github.com/cor3ntin commented: Thanks for this! I have some concerns over le lack of tests, although I understand this would be difficult. More importantly, I have concerns about the impact on reading non-source files, such as #embed files. Did you test that? https://github.com/llvm/

[clang] [llvm] [SystemZ][z/OS] Update autoconversion functions to improve support for UTF-8 (PR #98652)

2024-07-12 Thread via cfe-commits
@@ -326,7 +326,7 @@ ErrorOr> RealFileSystem::openFileForRead(const Twine &Name) { SmallString<256> RealName, Storage; Expected FDOrErr = sys::fs::openNativeFileForRead( - adjustPath(Name, Storage), sys::fs::OF_None, &RealName); + adjustPath(Name, Storage), sys::fs

[clang] [llvm] [SystemZ][z/OS] Update autoconversion functions to improve support for UTF-8 (PR #98652)

2024-07-12 Thread via cfe-commits
@@ -52,6 +53,12 @@ std::error_code restoreStdHandleAutoConversion(int FD); /// \brief Set the tag information for a file descriptor. std::error_code setFileTag(int FD, int CCSID, bool Text); +// Get the the tag ccsid for a file name or a file descriptor. +ErrorOr<__ccsid_t> ge

[clang] 7d1b6b2 - [Clang][ARM][AArch64] Add branch protection attributes to the defaults. (#83277)

2024-07-12 Thread via cfe-commits
Author: Daniel Kiss Date: 2024-07-12T20:52:56+02:00 New Revision: 7d1b6b2c32f5d403a8eba2c16c6431847041ab87 URL: https://github.com/llvm/llvm-project/commit/7d1b6b2c32f5d403a8eba2c16c6431847041ab87 DIFF: https://github.com/llvm/llvm-project/commit/7d1b6b2c32f5d403a8eba2c16c6431847041ab87.diff L

[clang] [Clang][ARM][AArch64] Add branch protection attributes to the defaults. (PR #83277)

2024-07-12 Thread Daniel Kiss via cfe-commits
https://github.com/DanielKristofKiss closed https://github.com/llvm/llvm-project/pull/83277 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-doc] Support markdown and simplify checks (PR #97684)

2024-07-12 Thread via cfe-commits
https://github.com/PeterChou1 edited https://github.com/llvm/llvm-project/pull/97684 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libunwind] [libunwind] Undefined behaviour pointer arithmetic with null pointer (PR #98648)

2024-07-12 Thread Daniel Kiss via cfe-commits
https://github.com/DanielKristofKiss updated https://github.com/llvm/llvm-project/pull/98648 >From 240add341b2c0a1be3d1ebf21938e70e51669126 Mon Sep 17 00:00:00 2001 From: Daniel Kiss Date: Fri, 12 Jul 2024 15:56:40 +0200 Subject: [PATCH 1/2] [libunwind] Fix ubsan issue --- libunwind/src/Unwin

[libunwind] [libunwind] Undefined behaviour pointer arithmetic with null pointer (PR #98648)

2024-07-12 Thread Daniel Kiss via cfe-commits
@@ -230,8 +230,13 @@ void DwarfFDECache::iterateCacheEntries(void (*func)( } #endif // defined(_LIBUNWIND_SUPPORT_DWARF_UNWIND) - -#define arrayoffsetof(type, index, field) ((size_t)(&((type *)0)[index].field)) +template +__attribute__((no_sanitize("undefined"))) static inlin

[clang] [libcxx] [llvm] Reapply "[Clang] Implement resolution for CWG1835 (#92957)" (PR #98547)

2024-07-12 Thread Mike Hommey via cfe-commits
glandium wrote: The sad part is that MSVC and some old versions of GCC don't like `template` being added here :( https://github.com/llvm/llvm-project/pull/98547 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mai

[clang] The pragma STDC CX_LIMITED_RANGE ON should have precedence. (PR #98520)

2024-07-12 Thread Zahira Ammarguellat via cfe-commits
@@ -218,28 +214,31 @@ _Complex float pragma_default_div(_Complex float a, _Complex float b) { // IMPRVD-NEXT: fsub float // IMPRVD-NEXT: fdiv float - // PRMTD: load float, ptr {{.*}} - // PRMTD: fpext float {{.*}} to double - // PRMTD-NEXT: fpext float {{.*}} to double

[clang] [WebAssembly] Disable running `wasm-opt` on components (PR #98373)

2024-07-12 Thread Alex Crichton via cfe-commits
alexcrichton wrote: @sbc100 oh I can't merge things myself so if you'd be ok merging for me that'd be appreciated https://github.com/llvm/llvm-project/pull/98373 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/ma

[clang] The pragma STDC CX_LIMITED_RANGE ON should have precedence. (PR #98520)

2024-07-12 Thread Zahira Ammarguellat via cfe-commits
https://github.com/zahiraam edited https://github.com/llvm/llvm-project/pull/98520 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] [C23] Fix typeof_unqual for qualified array types (PR #92767)

2024-07-12 Thread Mital Ashok via cfe-commits
https://github.com/MitalAshok updated https://github.com/llvm/llvm-project/pull/92767 >From f87cb4c754a477515746e2ac2f8906b93ccd1fe3 Mon Sep 17 00:00:00 2001 From: Mital Ashok Date: Mon, 20 May 2024 15:58:58 +0100 Subject: [PATCH 1/5] [Clang] [C23] Fix typeof_unqual for qualified array types P

[clang] [analyzer] Don't display the offset value in underflows (PR #98621)

2024-07-12 Thread Donát Nagy via cfe-commits
https://github.com/NagyDonat updated https://github.com/llvm/llvm-project/pull/98621 From 2765bc97d3242d50fd73aedb9e9d38dfdcef814c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Don=C3=A1t=20Nagy?= Date: Fri, 12 Jul 2024 13:57:53 +0200 Subject: [PATCH 1/2] [analyzer] Don't display the offset value in

[clang] [Clang] [C23] Fix typeof_unqual for qualified array types (PR #92767)

2024-07-12 Thread Mital Ashok via cfe-commits
MitalAshok wrote: @AaronBallman was there anything else that needed to be done? Just did a merge because ReleaseNotes was edited but this should be fine https://github.com/llvm/llvm-project/pull/92767 ___ cfe-commits mailing list cfe-commits@lists.llv

[clang] [analyzer] Don't display the offset value in underflows (PR #98621)

2024-07-12 Thread Donát Nagy via cfe-commits
NagyDonat wrote: > I wonder if we could have something in between. I'm thinking of having the > concrete offset as a separate note, instead of having it part of the primary > message. That way after BR selection, we would still deterministically pick > the shortest parh, and also have the offs

[clang] [analyzer] Don't display the offset value in underflows (PR #98621)

2024-07-12 Thread Donát Nagy via cfe-commits
NagyDonat wrote: _(Technical detail: I'll be on vacation during the next week, so I won't see updates on this PR until the 22nd of July. If you want to merge this PR, feel free to do so, but don't forget to adjust the description and the title to accurately reflect the current state of the PR.

[clang] [analyzer] Improve bug report hashing, merge similar reports (PR #98621)

2024-07-12 Thread Donát Nagy via cfe-commits
https://github.com/NagyDonat edited https://github.com/llvm/llvm-project/pull/98621 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [TBAA] Emit distinct TBAA tags for pointers with different depths,types. (PR #76612)

2024-07-12 Thread Florian Hahn via cfe-commits
https://github.com/fhahn closed https://github.com/llvm/llvm-project/pull/76612 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Improve bug report hashing, merge similar reports (PR #98621)

2024-07-12 Thread Donát Nagy via cfe-commits
https://github.com/NagyDonat edited https://github.com/llvm/llvm-project/pull/98621 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [TBAA] Emit int TBAA metadata on FP math libcall expf (PR #96025)

2024-07-12 Thread Eli Friedman via cfe-commits
https://github.com/efriedma-quic commented: >> How does this interact with StrictFP? > I don't think it related to StrictFP, because we only care whether the global > variable errno is modified? which may bring in alias to other global > variable. As now the type of errno is int, so we record

[clang] [TBAA] Emit int TBAA metadata on FP math libcall expf (PR #96025)

2024-07-12 Thread Eli Friedman via cfe-commits
@@ -0,0 +1,43 @@ +// NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py UTC_ARGS: --version 5 +// The test may fail as time out on windows +// REQUIRES: system-linux + +// RUN: %clang -S -O3 -emit-llvm -o - -x c++ %s | FileCheck %s -check-prefixes=CHECK,

[clang] [TBAA] Emit int TBAA metadata on FP math libcall expf (PR #96025)

2024-07-12 Thread Eli Friedman via cfe-commits
https://github.com/efriedma-quic edited https://github.com/llvm/llvm-project/pull/96025 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [TBAA] Emit int TBAA metadata on FP math libcall expf (PR #96025)

2024-07-12 Thread Eli Friedman via cfe-commits
@@ -707,7 +707,34 @@ static RValue emitLibraryCall(CodeGenFunction &CGF, const FunctionDecl *FD, const CallExpr *E, llvm::Constant *calleeValue) { CodeGenFunction::CGFPOptionsRAII FPOptsRAII(CGF, E); CGCallee callee = CGCallee::forDirect(calle

[clang] [llvm] [Clang][Coroutines] Introducing the `[[clang::coro_inplace_task]]` attribute (PR #94693)

2024-07-12 Thread Yuxuan Chen via cfe-commits
https://github.com/yuxuanchen1997 updated https://github.com/llvm/llvm-project/pull/94693 >From 4e4bcdc19268543a8348736dede46d8f8cad0066 Mon Sep 17 00:00:00 2001 From: Yuxuan Chen Date: Tue, 4 Jun 2024 23:22:00 -0700 Subject: [PATCH 1/3] [Clang] Introduce [[clang::coro_inplace_task]] --- clan

[clang] [analyzer] Improve bug report hashing, merge similar reports (PR #98621)

2024-07-12 Thread Donát Nagy via cfe-commits
@@ -373,14 +373,14 @@ static std::optional getConcreteValue(std::optional SV) { } static Messages getPrecedesMsgs(const SubRegion *Region, NonLoc Offset) { - std::string RegName = getRegionName(Region); - SmallString<128> Buf; - llvm::raw_svector_ostream Out(Buf); - Out <

[clang] [analyzer] Improve bug report hashing, merge similar reports (PR #98621)

2024-07-12 Thread Donát Nagy via cfe-commits
https://github.com/NagyDonat edited https://github.com/llvm/llvm-project/pull/98621 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [HLSL] Split out resource class data from resource attribute (PR #98419)

2024-07-12 Thread Justin Bogner via cfe-commits
https://github.com/bogner approved this pull request. https://github.com/llvm/llvm-project/pull/98419 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [TBAA] Emit distinct TBAA tags for pointers with different depths,types. (PR #76612)

2024-07-12 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-ve-ninja` running on `hpce-ve-main` while building `clang` at step 4 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/12/builds/1782 Here is the relevant piece of the build log for the

[clang] 5dc371e - [HLSL] Split out resource class data from resource attribute (#98419)

2024-07-12 Thread via cfe-commits
Author: Joshua Batista Date: 2024-07-12T13:10:34-07:00 New Revision: 5dc371e289584928345f74f560a18a805226b5f8 URL: https://github.com/llvm/llvm-project/commit/5dc371e289584928345f74f560a18a805226b5f8 DIFF: https://github.com/llvm/llvm-project/commit/5dc371e289584928345f74f560a18a805226b5f8.diff

[clang] [HLSL] Split out resource class data from resource attribute (PR #98419)

2024-07-12 Thread Joshua Batista via cfe-commits
https://github.com/bob80905 closed https://github.com/llvm/llvm-project/pull/98419 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] The pragma STDC CX_LIMITED_RANGE ON should have precedence. (PR #98520)

2024-07-12 Thread Zahira Ammarguellat via cfe-commits
https://github.com/zahiraam updated https://github.com/llvm/llvm-project/pull/98520 >From da863999000a6b12f2930247de9df3daf0e5a608 Mon Sep 17 00:00:00 2001 From: Zahira Ammarguellat Date: Thu, 11 Jul 2024 11:54:13 -0700 Subject: [PATCH 1/3] The pragma STDC CX_LIMITED_RANGE ON should have prece

[clang] The pragma STDC CX_LIMITED_RANGE ON should have precedence. (PR #98520)

2024-07-12 Thread Zahira Ammarguellat via cfe-commits
https://github.com/zahiraam ready_for_review https://github.com/llvm/llvm-project/pull/98520 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] The pragma STDC CX_LIMITED_RANGE ON should have precedence. (PR #98520)

2024-07-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-codegen Author: Zahira Ammarguellat (zahiraam) Changes The `pragma STDC CX_LIMITED_RANGE` should have precedence over the command line `-fcomplex-arithmetic`. --- Full diff: https://github.com/llvm/llvm-project/pull/98520.diff 2 Files Affected:

[clang] The pragma STDC CX_LIMITED_RANGE ON should have precedence. (PR #98520)

2024-07-12 Thread Zahira Ammarguellat via cfe-commits
https://github.com/zahiraam updated https://github.com/llvm/llvm-project/pull/98520 >From da863999000a6b12f2930247de9df3daf0e5a608 Mon Sep 17 00:00:00 2001 From: Zahira Ammarguellat Date: Thu, 11 Jul 2024 11:54:13 -0700 Subject: [PATCH 1/4] The pragma STDC CX_LIMITED_RANGE ON should have prece

[clang] [libc] [llvm] [NVPTX] Implement variadic functions using IR lowering (PR #96015)

2024-07-12 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 updated https://github.com/llvm/llvm-project/pull/96015 >From 8bd49caa9fa93fd3d0812e0a4315f8ff4956056a Mon Sep 17 00:00:00 2001 From: Joseph Huber Date: Mon, 17 Jun 2024 15:32:31 -0500 Subject: [PATCH 1/2] [NVPTX] Implement variadic functions using IR lowering Summar

[clang] [libc] [llvm] [NVPTX] Implement variadic functions using IR lowering (PR #96015)

2024-07-12 Thread Joseph Huber via cfe-commits
jhuber6 wrote: Ping, waiting on this so I can land https://github.com/llvm/llvm-project/pull/96369. https://github.com/llvm/llvm-project/pull/96015 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo

[clang] [Driver] Ensure ToolChain::LibraryPaths is not empty for non-Darwin (PR #87866)

2024-07-12 Thread Vitaly Buka via cfe-commits
vitalybuka wrote: > Also > https://lab.llvm.org/buildbot/#/builders/66/builds/1427/steps/13/logs/stdio > > ``` > [20/21] Running compiler_rt regression tests > llvm-lit: > /b/sanitizer-x86_64-linux/build/llvm-project/compiler-rt/test/lit.common.cfg.py:60: > warning: Path reported by clang doe

[clang] [libc] [llvm] [NVPTX] Implement variadic functions using IR lowering (PR #96015)

2024-07-12 Thread Justin Lebar via cfe-commits
jlebar wrote: @Artem-B https://github.com/llvm/llvm-project/pull/96015 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Improve bug report hashing, merge similar reports (PR #98621)

2024-07-12 Thread Balazs Benics via cfe-commits
=?utf-8?q?Donát?= Nagy Message-ID: In-Reply-To: steakhal wrote: > _(Technical detail: I'll be on vacation during the next week, so I won't see > updates on this PR until the 22nd of July. If you want to merge this PR, feel > free to do so.)_ No need to rush. Have a nice one! https://githu

[clang-tools-extra] Fix the docs for the boost/modernize use-ranges checks (PR #98662)

2024-07-12 Thread Nathan James via cfe-commits
njames93 wrote: > Those functions could be listed as an bullet points. > But anyway lest, merge this and check if this fixes issues. Should be good, I actually built the docs this time rather than using a basic rst viewer to test it https://github.com/llvm/llvm-project/pull/98662 _

[clang] [clang] Diagnose use of deprecated template alias (PR #97619)

2024-07-12 Thread Louis Dionne via cfe-commits
ldionne wrote: I just ran the bootstrapping build with this patch applied and ran `check-runtimes`, and nothing failed. This might not be sufficient (perhaps I missed something important), but at first glance it doesn't seem to break anything on our side. https://github.com/llvm/llvm-project/

[clang] [cmake][Fuchsia] armv8m -> armv8m.main (PR #98695)

2024-07-12 Thread via cfe-commits
https://github.com/PiJoules created https://github.com/llvm/llvm-project/pull/98695 armv8m builtins aren't being built because `armv8m` doesn't match any of the arm cpu strings in compiler-rt cmake files. Instead there's `armv8m.main` and `armv8m.base`. We want to use the `armv8m.main` version

[clang] [cmake][Fuchsia] armv8m -> armv8m.main (PR #98695)

2024-07-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: None (PiJoules) Changes armv8m builtins aren't being built because `armv8m` doesn't match any of the arm cpu strings in compiler-rt cmake files. Instead there's `armv8m.main` and `armv8m.base`. We want to use the `armv8m.main` version. -

[clang] [cmake][Fuchsia] armv8m -> armv8m.main (PR #98695)

2024-07-12 Thread via cfe-commits
https://github.com/zeroomega approved this pull request. https://github.com/llvm/llvm-project/pull/98695 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 2ad7b4a - [cmake][Fuchsia] armv8m -> armv8m.main (#98695)

2024-07-12 Thread via cfe-commits
Author: PiJoules Date: 2024-07-12T13:55:25-07:00 New Revision: 2ad7b4af95bc333d8f216915cd1b9d688590dcc5 URL: https://github.com/llvm/llvm-project/commit/2ad7b4af95bc333d8f216915cd1b9d688590dcc5 DIFF: https://github.com/llvm/llvm-project/commit/2ad7b4af95bc333d8f216915cd1b9d688590dcc5.diff LOG:

[clang] [cmake][Fuchsia] armv8m -> armv8m.main (PR #98695)

2024-07-12 Thread via cfe-commits
https://github.com/PiJoules closed https://github.com/llvm/llvm-project/pull/98695 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Diagnose use of deprecated template alias (PR #97619)

2024-07-12 Thread via cfe-commits
premanandrao wrote: > I just ran the bootstrapping build with this patch applied and ran > `check-runtimes`, and nothing failed. This might not be sufficient (perhaps I > missed something important), but at first glance it doesn't seem to break > anything on our side. Thank you SO much! Wo

[clang] [Clang] Allow diagnostics starting with Arm (PR #98091)

2024-07-12 Thread via cfe-commits
amilendra wrote: > > If the diagnostics are downstream-only, does this really need go upstream? > > We try to upstream as much as possible to reduce merge conflicts. @jroelofs Do you have any objections to commiting this? https://github.com/llvm/llvm-project/pull/98091 ___

[clang] [llvm] [PAC][Driver] Implement `-mbranch-protection=pauthabi` option (PR #97237)

2024-07-12 Thread Daniil Kovalev via cfe-commits
https://github.com/kovdan01 updated https://github.com/llvm/llvm-project/pull/97237 >From 3b4b1b1739b810d758e68f30c48b648963cff740 Mon Sep 17 00:00:00 2001 From: Daniil Kovalev Date: Mon, 1 Jul 2024 00:50:21 +0300 Subject: [PATCH 1/6] [PAC][Driver] Implement `-mbranch-protection=pauthabi` opti

[clang] [llvm] [PAC][Driver] Implement `-mbranch-protection=pauthabi` option (PR #97237)

2024-07-12 Thread Daniil Kovalev via cfe-commits
https://github.com/kovdan01 updated https://github.com/llvm/llvm-project/pull/97237 >From 3b4b1b1739b810d758e68f30c48b648963cff740 Mon Sep 17 00:00:00 2001 From: Daniil Kovalev Date: Mon, 1 Jul 2024 00:50:21 +0300 Subject: [PATCH 1/6] [PAC][Driver] Implement `-mbranch-protection=pauthabi` opti

[clang] [llvm] [PAC][Driver] Implement `-mbranch-protection=pauthabi` option (PR #97237)

2024-07-12 Thread Daniil Kovalev via cfe-commits
https://github.com/kovdan01 edited https://github.com/llvm/llvm-project/pull/97237 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [PAC][Driver] Support `pauthtest` ABI for AArch64 Linux triples (PR #97237)

2024-07-12 Thread Daniil Kovalev via cfe-commits
https://github.com/kovdan01 edited https://github.com/llvm/llvm-project/pull/97237 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [PAC][Driver] Support `pauthtest` ABI for AArch64 Linux triples (PR #97237)

2024-07-12 Thread Daniil Kovalev via cfe-commits
https://github.com/kovdan01 closed https://github.com/llvm/llvm-project/pull/97237 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [PAC][Driver] Support `pauthtest` ABI for AArch64 Linux triples (PR #97237)

2024-07-12 Thread Daniil Kovalev via cfe-commits
https://github.com/kovdan01 reopened https://github.com/llvm/llvm-project/pull/97237 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [PAC][Driver] Support `pauthtest` ABI for AArch64 Linux triples (PR #97237)

2024-07-12 Thread Daniil Kovalev via cfe-commits
kovdan01 wrote: @smithp35 I've implemented `pauthtest` ABI support - see new PR description for details and tests for examples. I'm not sure if the implementation is nice and maybe I've put some logic in wrong places while there are better ones, but the result looks matching the requirements d

[clang] [cmake][Fuchsia] armv8m -> armv8m.main (PR #98695)

2024-07-12 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `lldb-aarch64-ubuntu` running on `linaro-lldb-aarch64-ubuntu` while building `clang` at step 6 "test". Full details are available at: https://lab.llvm.org/buildbot/#/builders/59/builds/1495 Here is the relevant piece of the bu

[clang] [libc] [llvm] [NVPTX] Implement variadic functions using IR lowering (PR #96015)

2024-07-12 Thread Artem Belevich via cfe-commits
https://github.com/Artem-B edited https://github.com/llvm/llvm-project/pull/96015 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [libc] [llvm] [NVPTX] Implement variadic functions using IR lowering (PR #96015)

2024-07-12 Thread Artem Belevich via cfe-commits
https://github.com/Artem-B approved this pull request. LGTM with a minor nit. https://github.com/llvm/llvm-project/pull/96015 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [libc] [llvm] [NVPTX] Implement variadic functions using IR lowering (PR #96015)

2024-07-12 Thread Artem Belevich via cfe-commits
@@ -203,8 +203,12 @@ ABIArgInfo NVPTXABIInfo::classifyArgumentType(QualType Ty) const { void NVPTXABIInfo::computeInfo(CGFunctionInfo &FI) const { if (!getCXXABI().classifyReturnType(FI)) FI.getReturnInfo() = classifyReturnType(FI.getReturnType()); + + unsigned Argument

[clang] [Clang] Allow diagnostics starting with Arm (PR #98091)

2024-07-12 Thread Jon Roelofs via cfe-commits
jroelofs wrote: This one looks fairly benign, and while I agree that it is good to upstream things wherever possible, my slight objection rests mostly on two principles: * In general the community doesn't take behavior-changing patches purely on the basis of eliminating merge conflicts for cod

[clang] [clang][test] add TestLanguage.def to specify all tested language versions (PR #94243)

2024-07-12 Thread Julian Schmidt via cfe-commits
https://github.com/5chmidti updated https://github.com/llvm/llvm-project/pull/94243 >From e1a2f731a24840c4c922ead50393ad9ccf342d8c Mon Sep 17 00:00:00 2001 From: Julian Schmidt Date: Sat, 1 Jun 2024 17:38:37 +0200 Subject: [PATCH 1/3] [clang][test] add TestLanguage.def to specify all tested la

[clang] [libc] [llvm] [NVPTX] Implement variadic functions using IR lowering (PR #96015)

2024-07-12 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 updated https://github.com/llvm/llvm-project/pull/96015 >From 698ec5bb3e9247b4b47a99eeac4d0933fc0a59ee Mon Sep 17 00:00:00 2001 From: Joseph Huber Date: Mon, 17 Jun 2024 15:32:31 -0500 Subject: [PATCH] [NVPTX] Implement variadic functions using IR lowering Summary: T

[clang] [clang][test] add TestLanguage.def to specify all tested language versions (PR #94243)

2024-07-12 Thread Julian Schmidt via cfe-commits
@@ -0,0 +1,39 @@ + +//===-- TestLanguage.def - Language Versions for Testing *- C++ -*-===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: A

[clang] [clang][test] add TestLanguage.def to specify all tested language versions (PR #94243)

2024-07-12 Thread Julian Schmidt via cfe-commits
@@ -0,0 +1,39 @@ + +//===-- TestLanguage.def - Language Versions for Testing *- C++ -*-===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: A

[clang] [clang][test] add TestLanguage.def to specify all tested language versions (PR #94243)

2024-07-12 Thread Julian Schmidt via cfe-commits
5chmidti wrote: rebased on trunk https://github.com/llvm/llvm-project/pull/94243 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] Allow specifying pipe syntax for use-ranges checks (PR #98696)

2024-07-12 Thread Nathan James via cfe-commits
https://github.com/njames93 created https://github.com/llvm/llvm-project/pull/98696 Add `UseReversePipe` option to (boost|modernize)-use-ranges checks. This controls whether to create a reverse view using function syntax (`reverse(Range)`) or pipe syntax (`Range | reverse`) >From f709a09351a6

[clang-tools-extra] Allow specifying pipe syntax for use-ranges checks (PR #98696)

2024-07-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra Author: Nathan James (njames93) Changes Add `UseReversePipe` option to (boost|modernize)-use-ranges checks. This controls whether to create a reverse view using function syntax (`reverse(Range)`) or pipe syntax (`Range | reverse`) --

[clang] [libc] [llvm] [NVPTX] Implement variadic functions using IR lowering (PR #96015)

2024-07-12 Thread Joseph Huber via cfe-commits
@@ -203,8 +203,12 @@ ABIArgInfo NVPTXABIInfo::classifyArgumentType(QualType Ty) const { void NVPTXABIInfo::computeInfo(CGFunctionInfo &FI) const { if (!getCXXABI().classifyReturnType(FI)) FI.getReturnInfo() = classifyReturnType(FI.getReturnType()); + + unsigned Argument

[clang-tools-extra] Allow specifying pipe syntax for use-ranges checks (PR #98696)

2024-07-12 Thread via cfe-commits
@@ -170,3 +170,20 @@ Options If `true` (default value) the boost headers are included as system headers with angle brackets (`#include `), otherwise quotes are used (`#include "boost.hpp"`). + +.. option:: UseReversePipe + + When `true` (default `false`), Fixes which

[clang-tools-extra] Allow specifying pipe syntax for use-ranges checks (PR #98696)

2024-07-12 Thread via cfe-commits
@@ -170,3 +170,20 @@ Options If `true` (default value) the boost headers are included as system headers with angle brackets (`#include `), otherwise quotes are used (`#include "boost.hpp"`). + +.. option:: UseReversePipe + + When `true` (default `false`), Fixes which

[clang-tools-extra] Allow specifying pipe syntax for use-ranges checks (PR #98696)

2024-07-12 Thread via cfe-commits
@@ -0,0 +1,100 @@ +#ifndef USE_RANGES_FAKE_STD_H +#define USE_RANGES_FAKE_STD_H +namespace std { + +template class vector { +public: EugeneZelenko wrote: ```suggestion ``` https://github.com/llvm/llvm-project/pull/98696 _

[clang-tools-extra] Allow specifying pipe syntax for use-ranges checks (PR #98696)

2024-07-12 Thread via cfe-commits
@@ -170,3 +170,20 @@ Options If `true` (default value) the boost headers are included as system headers with angle brackets (`#include `), otherwise quotes are used (`#include "boost.hpp"`). + +.. option:: UseReversePipe + + When `true` (default `false`), Fixes which

[clang] [libc] [llvm] [NVPTX] Implement variadic functions using IR lowering (PR #96015)

2024-07-12 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 closed https://github.com/llvm/llvm-project/pull/96015 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 486d00e - [NVPTX] Implement variadic functions using IR lowering (#96015)

2024-07-12 Thread via cfe-commits
Author: Joseph Huber Date: 2024-07-12T17:09:48-05:00 New Revision: 486d00eca6b6ab470e8324b52cdf9f32023c1c9a URL: https://github.com/llvm/llvm-project/commit/486d00eca6b6ab470e8324b52cdf9f32023c1c9a DIFF: https://github.com/llvm/llvm-project/commit/486d00eca6b6ab470e8324b52cdf9f32023c1c9a.diff

[clang-tools-extra] Allow specifying pipe syntax for use-ranges checks (PR #98696)

2024-07-12 Thread Nathan James via cfe-commits
https://github.com/njames93 updated https://github.com/llvm/llvm-project/pull/98696 >From c3431c0bedb23feac279efa43b51740353dcab8d Mon Sep 17 00:00:00 2001 From: Nathan James Date: Fri, 12 Jul 2024 23:04:42 +0100 Subject: [PATCH] Allow specifying pipe syntax for use-ranges checks Add `UseRever

[clang] [HLSL] Rework implicit conversion sequences (PR #96011)

2024-07-12 Thread Chris B via cfe-commits
https://github.com/llvm-beanz updated https://github.com/llvm/llvm-project/pull/96011 >From 84c6a2c9dea0d964db68b282b0236783a46ee292 Mon Sep 17 00:00:00 2001 From: Chris Bieneman Date: Tue, 18 Jun 2024 20:25:57 -0500 Subject: [PATCH] [HLSL] Rework implicit conversion sequences This PR reworks

[clang] [HLSL] Rework implicit conversion sequences (PR #96011)

2024-07-12 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 2ad7b4af95bc333d8f216915cd1b9d688590dcc5 84c6a2c9dea0d964db68b282b0236783a46ee292 --e

[clang] [HLSL] Rework implicit conversion sequences (PR #96011)

2024-07-12 Thread Chris B via cfe-commits
https://github.com/llvm-beanz updated https://github.com/llvm/llvm-project/pull/96011 >From 84c6a2c9dea0d964db68b282b0236783a46ee292 Mon Sep 17 00:00:00 2001 From: Chris Bieneman Date: Tue, 18 Jun 2024 20:25:57 -0500 Subject: [PATCH 1/2] [HLSL] Rework implicit conversion sequences This PR rewo

[clang] [HLSL] Rework implicit conversion sequences (PR #96011)

2024-07-12 Thread Chris B via cfe-commits
https://github.com/llvm-beanz updated https://github.com/llvm/llvm-project/pull/96011 >From 84c6a2c9dea0d964db68b282b0236783a46ee292 Mon Sep 17 00:00:00 2001 From: Chris Bieneman Date: Tue, 18 Jun 2024 20:25:57 -0500 Subject: [PATCH 1/3] [HLSL] Rework implicit conversion sequences This PR rewo

[clang] [AArch64][RISCV] Document option --print-supported-extensions (PR #98698)

2024-07-12 Thread Jonathan Thackray via cfe-commits
https://github.com/jthackray created https://github.com/llvm/llvm-project/pull/98698 Add documentation for the `--print-supported-extensions` option which appears to be missing at present. >From 8071d6f5da6d6a236a558e1488f2d9f1eb51719b Mon Sep 17 00:00:00 2001 From: Jonathan Thackray Date: Fri

[clang] [AArch64][RISCV] Document option --print-supported-extensions (PR #98698)

2024-07-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Jonathan Thackray (jthackray) Changes Add documentation for the `--print-supported-extensions` option which appears to be missing at present. --- Full diff: https://github.com/llvm/llvm-project/pull/98698.diff 1 Files Affected: - (modif

[clang] Implement resource binding type prefix mismatch diagnostic infrastructure (PR #97103)

2024-07-12 Thread Joshua Batista via cfe-commits
https://github.com/bob80905 edited https://github.com/llvm/llvm-project/pull/97103 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Prevent dangling StringRefs (PR #98699)

2024-07-12 Thread Youngsuk Kim via cfe-commits
https://github.com/JOE1994 created https://github.com/llvm/llvm-project/pull/98699 Fix locations where dangling StringRefs are created. * `ConstraintSatisfaction::SubstitutionDiagnostic`: typedef of `std::pair` * `concepts::Requirement::SubstitutionDiagnostic`: struct whose 1st and 3rd data

[clang] [clang] Prevent dangling StringRefs (PR #98699)

2024-07-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Youngsuk Kim (JOE1994) Changes Fix locations where dangling StringRefs are created. * `ConstraintSatisfaction::SubstitutionDiagnostic`: typedef of `std::pair` * `concepts::Requirement::SubstitutionDiagnostic`:

[clang] [clang] Prevent dangling StringRefs (PR #98699)

2024-07-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-modules Author: Youngsuk Kim (JOE1994) Changes Fix locations where dangling StringRefs are created. * `ConstraintSatisfaction::SubstitutionDiagnostic`: typedef of `std::pair` * `concepts::Requirement::SubstitutionDiagn

[clang] [libcxx] [llvm] Reapply "[Clang] Implement resolution for CWG1835 (#92957)" (PR #98547)

2024-07-12 Thread Pranav Kant via cfe-commits
pranavk wrote: Could we perhaps put this behind a flag to help with the transition? Some cases can already be silenced with -Wno-missing-dependent-template-keyword but cases like above in absl are hard errors and there are no workarounds. https://github.com/llvm/llvm-project/pull/98547 __

[clang] Implement resource binding type prefix mismatch diagnostic infrastructure (PR #97103)

2024-07-12 Thread Joshua Batista via cfe-commits
@@ -1,9 +1,10 @@ // RUN: %clang_cc1 -triple dxil-pc-shadermodel6.3-library -x hlsl -o - -fsyntax-only %s -verify -// expected-error@+1 {{invalid resource class specifier 'c' used; expected 'b', 's', 't', or 'u'}} -float a : register(c0, space1); +// valid, The register keywor

<    1   2   3   4   5   >