[clang] [libcxx] [clang & libcxx] constexpr pointer tagging (DO NOT MERGE) (PR #111861)

2024-10-11 Thread Hana Dusíková via cfe-commits
@@ -0,0 +1,517 @@ +// -*- 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

[clang] Clang: Support minimumnum and maximumnum intrinsics (PR #96281)

2024-10-11 Thread YunQiang Su via cfe-commits
https://github.com/wzssyqa updated https://github.com/llvm/llvm-project/pull/96281 >From c08b0a8d34107dc4563c434485ba5f326ab8df93 Mon Sep 17 00:00:00 2001 From: YunQiang Su Date: Fri, 21 Jun 2024 14:28:42 +0800 Subject: [PATCH 1/6] Clang: Support minimumnum and maximumnum intrinsics We just in

[clang] [libcxx] [clang & libcxx] constexpr pointer tagging (DO NOT MERGE) (PR #111861)

2024-10-11 Thread Hana Dusíková via cfe-commits
@@ -0,0 +1,517 @@ +// -*- 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

[clang] Clang: Support minimumnum and maximumnum intrinsics (PR #96281)

2024-10-11 Thread via cfe-commits
@@ -58,7 +58,9 @@ // Check __has_constexpr_builtin #if !__has_constexpr_builtin(__builtin_fmax) || \ - !__has_constexpr_builtin(__builtin_fmin) + !__has_constexpr_builtin(__builtin_fmin) || \ + !__has_constexpr_builtin(__builtin_fmaximum_num) || \ + !__has_con

[clang] [clang] When checking for covariant return types, make sure the poiners or references are to *classes* (PR #111856)

2024-10-11 Thread Boaz Brickner via cfe-commits
https://github.com/bricknerb updated https://github.com/llvm/llvm-project/pull/111856 >From 786d31e2657964e578cd1fdf2006b0fb3b19fab6 Mon Sep 17 00:00:00 2001 From: Boaz Brickner Date: Thu, 10 Oct 2024 15:15:23 + Subject: [PATCH 1/4] [clang] When checking for covariant return types, make su

[clang] [C++20][Modules] Fix crash when function and lambda inside loaded from different modules (PR #109167)

2024-10-11 Thread Dmitry Polukhin via cfe-commits
dmpolukhin wrote: > I've checked #111992 and it does fix the problem, so let's land it? Yes, I would like to create a test case to don't regress this feature in future. I need to reduce libc++ `functional` header to something smaller. https://github.com/llvm/llvm-project/pull/109167 __

[clang-tools-extra] [clang-tidy] Make `P +- BS / sizeof(*P)` opt-outable in `bugprone-sizeof-expression` (PR #111178)

2024-10-11 Thread via cfe-commits
https://github.com/whisperity updated https://github.com/llvm/llvm-project/pull/78 >From 4415aaf903635d11cce7afc232e1768ad32e8592 Mon Sep 17 00:00:00 2001 From: Whisperity Date: Fri, 4 Oct 2024 17:30:41 +0200 Subject: [PATCH 1/2] feat(clang-tidy): Make `P + BS / sizeof(*P)` opt-outable ---

[clang] Clang: Support minimumnum and maximumnum intrinsics (PR #96281)

2024-10-11 Thread Matt Arsenault via cfe-commits
@@ -15314,6 +15314,32 @@ bool FloatExprEvaluator::VisitCallExpr(const CallExpr *E) { Result = RHS; return true; } + + case Builtin::BI__builtin_fmaximum_num: + case Builtin::BI__builtin_fmaximum_numf: + case Builtin::BI__builtin_fmaximum_numl: + case Builtin::B

[clang] [llvm] [clang-format] Add CI check confirming ClangFormatStyleOptions.rst is up-to-date. (PR #111513)

2024-10-11 Thread Iuri Chaer via cfe-commits
ichaer wrote: > IMO it would be more helpful to add the build target to > lib/Format/CMakeLists.txt instead. For example, ``` $ ninja FormatTests ``` Moving the target from `clang/docs/CMakeLists.txt` to `lib/Format/CMakeLists.txt` sounds great to me, but the testing part of the thing relies

[clang] [llvm] [SystemZ][z/OS] Add new openFileForReadBinary function, and pass IsText parameter to getBufferForFile (PR #111723)

2024-10-11 Thread kadir çetinkaya via cfe-commits
https://github.com/kadircet approved this pull request. thanks for bearing with me! https://github.com/llvm/llvm-project/pull/111723 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Reland: [clang] Finish implementation of P0522 (PR #111711)

2024-10-11 Thread Mikhail Goncharov via cfe-commits
metaflow wrote: I am going to revert this, sorry https://github.com/llvm/llvm-project/pull/111711 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [mlir] [polly] [NFC] Rename `Intrinsic::getDeclaration` to `getOrInsertDeclaration` (PR #111752)

2024-10-11 Thread Rahul Joshi via cfe-commits
https://github.com/jurahul closed https://github.com/llvm/llvm-project/pull/111752 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] When checking for covariant return types, make sure the poiners or references are to *classes* (PR #111856)

2024-10-11 Thread Boaz Brickner via cfe-commits
https://github.com/bricknerb updated https://github.com/llvm/llvm-project/pull/111856 >From 786d31e2657964e578cd1fdf2006b0fb3b19fab6 Mon Sep 17 00:00:00 2001 From: Boaz Brickner Date: Thu, 10 Oct 2024 15:15:23 + Subject: [PATCH 1/3] [clang] When checking for covariant return types, make su

[clang] Reland: [clang] Finish implementation of P0522 (PR #111711)

2024-10-11 Thread Mikhail Goncharov via cfe-commits
metaflow wrote: oh, that might become messy quickly. Do you know if 4dadf42c1a74dd4e37db9ffd6fbb3027f59751a7 and 224519b08945637a85e9798c78286643288f7b77 depend on this PR and should also be reverted? https://github.com/llvm/llvm-project/pull/111711 ___

[clang] [clang] Ignore inline namespace for `hasName` (PR #109147)

2024-10-11 Thread Alejandro Álvarez Ayllón via cfe-commits
https://github.com/alejandro-alvarez-sonarsource updated https://github.com/llvm/llvm-project/pull/109147 From d369d26aad940299eb98ff0c322614043144c558 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alejandro=20=C3=81lvarez=20Ayll=C3=B3n?= Date: Mon, 24 Jul 2023 13:56:29 +0200 Subject: [PATCH 1/7] [

[clang] Clang: Support minimumnum and maximumnum intrinsics (PR #96281)

2024-10-11 Thread YunQiang Su via cfe-commits
@@ -475,6 +475,12 @@ SYMBOL(fmaxl, None, ) SYMBOL(fmin, None, ) SYMBOL(fminf, None, ) SYMBOL(fminl, None, ) +SYMBOL(fmaximum_num, None, ) wzssyqa wrote: Oh, yes. It is autogenerated from https://github.com/PeterFeicht/cppreference-doc/releases. If we need to

[clang] [llvm] [AArch64] Introduce new armv9.6 features (PR #111677)

2024-10-11 Thread via cfe-commits
@@ -1510,12 +1524,16 @@ TEST(TargetParserTest, AArch64ArchExtFeature) { {"rdm", "nordm", "+rdm", "-rdm"}, {"sve", "nosve", "+sve", "-sve"}, {"sve-b16b16", "nosve-b16b16", "+sve-b16b16", "-sve-b16b16"}, + {"sve-bfscale", "nosve-bfscale", "+sve-bfscale", "-

[clang] [llvm] [AArch64] Introduce new armv9.6 features (PR #111677)

2024-10-11 Thread via cfe-commits
@@ -1413,6 +1424,9 @@ TEST(TargetParserTest, AArch64ExtensionFeatures) { EXPECT_TRUE(llvm::is_contained(Features, "+tlbiw")); EXPECT_TRUE(llvm::is_contained(Features, "+jsconv")); EXPECT_TRUE(llvm::is_contained(Features, "+complxnum")); + EXPECT_TRUE(llvm::is_contained(F

[clang] [llvm] [AArch64] Introduce new armv9.6 features (PR #111677)

2024-10-11 Thread via cfe-commits
@@ -1791,22 +1822,52 @@ AArch64ExtensionDependenciesBaseArchTestParams {"sve2-bitperm", "nosve2"}, {}, {"sve2", "sve2-bitperm"}}, -{AArch64::ARMV8A, {"nosve2", "sve2-aes"}, {"sve2", "sve2-aes"}, {}}, -{AArch64::ARMV8A, {"sve2-aes", "no

[clang] [clang] LazyOffsetPtr: Use native pointer width (PR #111995)

2024-10-11 Thread A. Wilcox via cfe-commits
https://github.com/awilfox created https://github.com/llvm/llvm-project/pull/111995 On big-endian systems, narrow casting will read the higher bits of the value. LazyOffsetPtr's `getAddressOfPointer` returns the address-of `Ptr` which was unconditionally a 64-bit value. On 32-bit big endian

[clang] [llvm] [AArch64] Introduce new armv9.6 features (PR #111677)

2024-10-11 Thread via cfe-commits
Lukacma wrote: In the end, we are going to go forward with adding Armv9.6-A to all feature descriptions. https://github.com/llvm/llvm-project/pull/111677 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/li

[clang] [clang] LazyOffsetPtr: Use native pointer width (PR #111995)

2024-10-11 Thread via cfe-commits
github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it

[clang] [clang] LazyOffsetPtr: Use native pointer width (PR #111995)

2024-10-11 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: A. Wilcox (awilfox) Changes On big-endian systems, narrow casting will read the higher bits of the value. LazyOffsetPtr's `getAddressOfPointer` returns the address-of `Ptr` which was unconditionally a 64-bit value. On 32-bit big endian

[clang] [llvm] [mlir] [polly] [NFC] Rename `Intrinsic::getDeclaration` to `getOrInsertDeclaration` (PR #111752)

2024-10-11 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `cross-project-tests-sie-ubuntu` running on `doug-worker-1a` while building `clang,llvm,mlir,polly` at step 5 "build-unified-tree". Full details are available at: https://lab.llvm.org/buildbot/#/builders/181/builds/6731 Here

[clang] [llvm] [AArch64] Introduce new armv9.6 features (PR #111677)

2024-10-11 Thread via cfe-commits
@@ -522,6 +522,39 @@ def FeatureTLBIW : ExtensionWithMArch<"tlbiw", "TLBIW", "FEAT_TLBIW", // Armv9.6 Architecture Extensions //===--===// +def FeatureCMPBR : ExtensionWithMArch<"cmpbr", "CMPBR", "FEAT_CMPB

[clang] [llvm] [AArch64] Introduce new armv9.6 features (PR #111677)

2024-10-11 Thread via cfe-commits
@@ -522,6 +522,39 @@ def FeatureTLBIW : ExtensionWithMArch<"tlbiw", "TLBIW", "FEAT_TLBIW", // Armv9.6 Architecture Extensions //===--===// +def FeatureCMPBR : ExtensionWithMArch<"cmpbr", "CMPBR", "FEAT_CMPB

[clang] [llvm] [mlir] [polly] [NFC] Rename `Intrinsic::getDeclaration` to `getOrInsertDeclaration` (PR #111752)

2024-10-11 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `cross-project-tests-sie-ubuntu-dwarf5` running on `doug-worker-1b` while building `clang,llvm,mlir,polly` at step 5 "build-unified-tree". Full details are available at: https://lab.llvm.org/buildbot/#/builders/163/builds/7028

[clang] [llvm] [AArch64] Introduce new armv9.6 features (PR #111677)

2024-10-11 Thread via cfe-commits
https://github.com/Lukacma updated https://github.com/llvm/llvm-project/pull/111677 >From efabe14befb5c35ae35826d79af0b64756cb3e9b Mon Sep 17 00:00:00 2001 From: Marian Lukac Date: Wed, 9 Oct 2024 10:52:42 + Subject: [PATCH 1/2] [AArch64] Introduce new armv9.6 features --- .../print-suppo

[clang] [llvm] [AArch64] Introduce new armv9.6 features (PR #111677)

2024-10-11 Thread via cfe-commits
@@ -522,6 +522,39 @@ def FeatureTLBIW : ExtensionWithMArch<"tlbiw", "TLBIW", "FEAT_TLBIW", // Armv9.6 Architecture Extensions //===--===// +def FeatureCMPBR : ExtensionWithMArch<"cmpbr", "CMPBR", "FEAT_CMPB

[clang] [clang] LazyOffsetPtr: Use native pointer width (PR #111995)

2024-10-11 Thread A. Wilcox via cfe-commits
awilfox wrote: This patch has been tested on 32-bit PowerPC, 64-bit PowerPC, and 64-bit x86_64 (ensuring it doesn't change any behaviour on LE systems) - the entire Clang test suite passed on all three platforms. https://github.com/llvm/llvm-project/pull/111995

[libcxx] [libcxxabi] [libunwind] [runtimes] Always define cxx_shared, cxx_static & other targets (PR #80007)

2024-10-11 Thread Mark Murray via cfe-commits
MarkMurrayARM wrote: NM - fixed in our builds. https://github.com/llvm/llvm-project/pull/80007 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [mlir] [polly] [NFC] Rename `Intrinsic::getDeclaration` to `getOrInsertDeclaration` (PR #111752)

2024-10-11 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,llvm,mlir,polly` at step 4 "build". Full details are available at: https://lab.llvm.org/buildbot/#/builders/59/builds/6436 Here is the relev

[clang] c8554e1 - Turn `-Wdeprecated-literal-operator` on by default (#111027)

2024-10-11 Thread via cfe-commits
Author: Erich Keane Date: 2024-10-11T06:10:03-07:00 New Revision: c8554e13eec048180d003af2aa7b2cc8498d4fba URL: https://github.com/llvm/llvm-project/commit/c8554e13eec048180d003af2aa7b2cc8498d4fba DIFF: https://github.com/llvm/llvm-project/commit/c8554e13eec048180d003af2aa7b2cc8498d4fba.diff L

[clang] Turn `-Wdeprecated-literal-operator` on by default (PR #111027)

2024-10-11 Thread Erich Keane via cfe-commits
https://github.com/erichkeane closed https://github.com/llvm/llvm-project/pull/111027 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] fix access checking about function overloading (PR #107768)

2024-10-11 Thread Zhikai Zeng via cfe-commits
Backl1ght wrote: Sorry for the late response, I will continue to look into this issue this sunday. https://github.com/llvm/llvm-project/pull/107768 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo

[clang] [CLANG]Add Scalable vectors for mfloat8_t (PR #101644)

2024-10-11 Thread via cfe-commits
https://github.com/CarolineConcatto updated https://github.com/llvm/llvm-project/pull/101644 >From 7134302c7e1054021af36a207dbfd0c40c9e8c51 Mon Sep 17 00:00:00 2001 From: Caroline Concatto Date: Fri, 2 Aug 2024 08:47:18 + Subject: [PATCH 1/2] [CLANG]Add Scalable vectors for mfloat8_t This

[clang] [llvm] [AArch64] Introduce new armv9.6 features (PR #111677)

2024-10-11 Thread via cfe-commits
https://github.com/Lukacma updated https://github.com/llvm/llvm-project/pull/111677 >From efabe14befb5c35ae35826d79af0b64756cb3e9b Mon Sep 17 00:00:00 2001 From: Marian Lukac Date: Wed, 9 Oct 2024 10:52:42 + Subject: [PATCH 1/3] [AArch64] Introduce new armv9.6 features --- .../print-suppo

[clang] [flang] [libc] [libcxx] [libcxxabi] [libunwind] [lldb] [llvm] [mlir] [clang][frontend] Add support for attribute plugins for statement attributes (PR #110334)

2024-10-11 Thread Aaron Ballman via cfe-commits
Valentin Clement =?utf-8?b?KOODkOODrOODsw=?=,Renato Golin Message-ID: In-Reply-To: https://github.com/AaronBallman approved this pull request. LGTM, though we should probably add a release note to clang/docs/ReleaseNotes.rst so users know that the plugin system got a bit more functionality.

[clang] [llvm] [mlir] [polly] [NFC] Rename `Intrinsic::getDeclaration` to `getOrInsertDeclaration` (PR #111752)

2024-10-11 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `bolt-x86_64-ubuntu-nfc` running on `bolt-worker` while building `clang,llvm,mlir,polly` at step 8 "test-build-bolt-check-bolt". Full details are available at: https://lab.llvm.org/buildbot/#/builders/92/builds/7899 Here is

[clang] [CLANG]Add Scalable vectors for mfloat8_t (PR #101644)

2024-10-11 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 159d694c05500a656775f4cbd6931dae9aab290a 111faea1619953a76a1acda070c61559c00678ca --e

[clang] [HIP] fix host min/max in header (PR #82956)

2024-10-11 Thread Yaxun Liu via cfe-commits
yxsamliu wrote: yet another usage of mixed signed/unsigned min https://github.com/ROCm/hipBLASLt/issues/1227 https://github.com/llvm/llvm-project/pull/82956 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman

[clang] [CLANG]Add Scalable vectors for mfloat8_t (PR #101644)

2024-10-11 Thread via cfe-commits
CarolineConcatto wrote: Hi @paulwalker-arm, I believe I addressed all the comments in the code. I updated almost all of them to have a test for the SVMfloat8_t. The ones that are not updated are the ones that test FIXED SVE types. The mangling needs scalar type support, BuiltinType::MFloat8,

[clang] [CLANG]Add Scalable vectors for mfloat8_t (PR #101644)

2024-10-11 Thread Paul Walker via cfe-commits
@@ -69,7 +69,7 @@ #ifndef SVE_VECTOR_TYPE_BFLOAT #define SVE_VECTOR_TYPE_BFLOAT(Name, MangledName, Id, SingletonId, NumEls, ElBits, NF) \ - SVE_VECTOR_TYPE_DETAILS(Name, MangledName, Id, SingletonId, NumEls, ElBits, NF, false, false, true) + SVE_VECTOR_TYPE_DETAILS(Name, M

[clang] [CLANG]Add Scalable vectors for mfloat8_t (PR #101644)

2024-10-11 Thread Paul Walker via cfe-commits
@@ -4419,6 +4419,13 @@ QualType ASTContext::getScalableVectorType(QualType EltTy, unsigned NumElts, EltTySize == ElBits && NumElts == (NumEls * NF) && NumFields == 1) { \ return SingletonId; \ } +#define

[clang] [clang-tools-extra] [clang-tidy] bugprone-implicit-widening ignores unsigned consts (PR #101073)

2024-10-11 Thread Chris Warner via cfe-commits
cwarner-8702 wrote: Thanks @AaronBallman! > I don't think we want constant expression evaluation to fail based on an > input parameter. It will already fail if the inputs as signed and expression overflows, so this seems like a case that all callers will have to deal with anyway. > I think a

[clang] [clang] Fix segmentation fault caused by stack overflow on deeply nested expressions (PR #111701)

2024-10-11 Thread Reid Kleckner via cfe-commits
@@ -0,0 +1,1013 @@ +// RUN: %clang_cc1 %s -emit-llvm -o - -Wstack-exhausted -verify + +class AClass { +public: + AClass() {} + AClass &foo() { return *this; } +}; + +void test_bar() { + AClass a; + // expected-warning@* {{stack nearly exhausted; compilation time may suffer, a

[clang] [llvm] [SystemZ][z/OS] Add new openFileForReadBinary function, and pass IsText parameter to getBufferForFile (PR #111723)

2024-10-11 Thread Abhina Sree via cfe-commits
https://github.com/abhina-sree updated https://github.com/llvm/llvm-project/pull/111723 >From c1676e48a587e10ba54c28e99192fd5e6a36f72e Mon Sep 17 00:00:00 2001 From: Abhina Sreeskantharajan Date: Wed, 9 Oct 2024 13:23:41 -0400 Subject: [PATCH 1/3] [SystemZ][z/OS] Add new openFileForReadBinary f

[clang] Add clang/docs/FunctionEffectAnalysis.rst. (PR #109855)

2024-10-11 Thread Doug Wyatt via cfe-commits
https://github.com/dougsonos updated https://github.com/llvm/llvm-project/pull/109855 >From 085965b324efde41168c5d51db3a368578d3458f Mon Sep 17 00:00:00 2001 From: Doug Wyatt Date: Mon, 23 Sep 2024 14:44:32 -0700 Subject: [PATCH 1/5] Add clang/docs/FunctionEffectAnalysis.rst. --- clang/docs/F

[clang] [llvm] [SystemZ][z/OS] Add new openFileForReadBinary function, and pass IsText parameter to getBufferForFile (PR #111723)

2024-10-11 Thread Abhina Sree via cfe-commits
abhina-sree wrote: @cor3ntin, I've added support for #embed handling now, I've added a z/OS only testcase clang/test/Preprocessor/embed_zos.c which is a copy of clang/test/Preprocessor/embed_art.c except it changes the file tag to be non-ascii which is now working with the latest commit. http

[clang] [lld] [llvm] [mlir] [IR] Introduce `T` to `DataLayout` to represent flat address space if a target supports it (PR #108786)

2024-10-11 Thread Shilei Tian via cfe-commits
shiltian wrote: > The LangRef doesn't need to know why it's undesirable. It's like the n field `n` field? The following? ``` n::... This specifies a set of native integer widths for the target CPU in bits. For example, it might contain n32 for 32-bit PowerPC, n32:64 for PowerPC 64, or n8:16:3

[clang] [Clang] Add explicit visibility symbol macros (PR #108276)

2024-10-11 Thread Thomas Fransham via cfe-commits
@@ -0,0 +1,67 @@ +//===-- clang/Support/Compiler.h - Compiler abstraction support -*- 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: Apa

[clang] Make PCH's respect any VFS specified. (PR #106577)

2024-10-11 Thread Neil Henning via cfe-commits
sheredom wrote: > > We tried setting -working-directory=Z:/working, but the VFS requires that > > this is a real path. > > What error are you seeing if it's not? Is it trying to set it too early > before the VFS is created or something? The VFS doesn't actually override the set current workin

[clang] clang: Fix hipstdpar test relying on default target (PR #111975)

2024-10-11 Thread Sean Perry via cfe-commits
https://github.com/perry-ca approved this pull request. This works on z/OS. Thanks for fixing it. https://github.com/llvm/llvm-project/pull/111975 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/

[clang] [clang][CGOpenMPRuntime] Avoid Type::getPointerTo() (NFC) (PR #112017)

2024-10-11 Thread Nikita Popov via cfe-commits
@@ -1713,25 +1713,15 @@ llvm::Function *CGOpenMPRuntime::emitThreadPrivateVarDefinition( if (!Ctor && !Dtor) return nullptr; -llvm::Type *CopyCtorTyArgs[] = {CGM.VoidPtrTy, CGM.VoidPtrTy}; -auto *CopyCtorTy = llvm::FunctionType::get(CGM.VoidPtrTy, CopyCtorTy

[clang] [clang] LazyOffsetPtr: Use native pointer width (PR #111995)

2024-10-11 Thread A. Wilcox via cfe-commits
awilfox wrote: I feel like the messages being highlighted in the CI formatter need work anyway. I was hoping the community would have a suggestion on better wording. However, I will be glad to reformat those lines in that manner if this wording should stay. https://github.com/llvm/llvm-proj

[clang] [Clang] Add a flag to include GPU startup files (PR #112025)

2024-10-11 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 updated https://github.com/llvm/llvm-project/pull/112025 >From a014a06d5b06bb8c94b4d86636fb57f342e184db Mon Sep 17 00:00:00 2001 From: Joseph Huber Date: Fri, 11 Oct 2024 12:21:49 -0500 Subject: [PATCH 1/3] [Clang] Add a flag to include GPU startup files Summary: The

[clang] [Clang] Add a flag to include GPU startup files (PR #112025)

2024-10-11 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 edited https://github.com/llvm/llvm-project/pull/112025 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Make PCH's respect any VFS specified. (PR #106577)

2024-10-11 Thread Ben Langmuir via cfe-commits
benlangmuir wrote: @sheredom I think I'm missing something. What's the actual failure? Setting `-working-directory` causes `FileManager` to prepend to paths the configured directory, so the VFS lookups would then be absolute paths not relative. I see the driver checks if the path in `-working

[clang] [Clang] [Sema] Don't crash on unexpanded pack in invalid block literal (PR #110762)

2024-10-11 Thread via cfe-commits
https://github.com/Sirraide updated https://github.com/llvm/llvm-project/pull/110762 >From 9c073cc3145bf6961b565516aea0e8d0c3fca0d7 Mon Sep 17 00:00:00 2001 From: Sirraide Date: Thu, 10 Oct 2024 13:08:20 +0200 Subject: [PATCH 1/5] Undo previous changes, keeping the release note and tests ---

[clang] [llvm] [CUDA] Add support for CUDA-12.6 and sm_100 (PR #112028)

2024-10-11 Thread Artem Belevich via cfe-commits
https://github.com/Artem-B updated https://github.com/llvm/llvm-project/pull/112028 >From 5dac14aab180fd965d996b47cf983b8c462fe703 Mon Sep 17 00:00:00 2001 From: Sergey Kozub Date: Tue, 2 Jul 2024 02:44:56 -0700 Subject: [PATCH] [CUDA] Add support for CUDA-12.6 and sm_100 --- clang/docs/Relea

[clang] 48bda00 - [Clang] [Sema] Don't crash on unexpanded pack in invalid block literal (#110762)

2024-10-11 Thread via cfe-commits
Author: Sirraide Date: 2024-10-11T20:03:43+02:00 New Revision: 48bda00b281a432d6de5e5e5dde6c5d66b992ac8 URL: https://github.com/llvm/llvm-project/commit/48bda00b281a432d6de5e5e5dde6c5d66b992ac8 DIFF: https://github.com/llvm/llvm-project/commit/48bda00b281a432d6de5e5e5dde6c5d66b992ac8.diff LOG:

[clang] [Clang] [Sema] Don't crash on unexpanded pack in invalid block literal (PR #110762)

2024-10-11 Thread via cfe-commits
https://github.com/Sirraide closed https://github.com/llvm/llvm-project/pull/110762 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] [Sema] Don't crash on unexpanded pack in invalid block literal (PR #110762)

2024-10-11 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 2aa1dbf9c92e3b27954ba9166c927616625073e8 daf41fb0502e54e4ab5c63ed2227b7be88297c74 --e

[clang] Reapply "[Clang][Sema] Refactor collection of multi-level template argument lists (#106585, #111173)" (PR #111852)

2024-10-11 Thread Krystian Stasiowski via cfe-commits
https://github.com/sdkrystian closed https://github.com/llvm/llvm-project/pull/111852 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] [Sema] Don't crash on unexpanded pack in invalid block literal (PR #110762)

2024-10-11 Thread via cfe-commits
Sirraide wrote: > C/C++ code formatter, clang-format found issues in your code ... I don’t think I changed anything about that include? Or did it just reorder them because I added one? :thinking: https://github.com/llvm/llvm-project/pull/110762 ___

[clang] [clang] Fix segmentation fault caused by stack overflow on deeply nested expressions (PR #111701)

2024-10-11 Thread Ilya Biryukov via cfe-commits
https://github.com/ilya-biryukov edited https://github.com/llvm/llvm-project/pull/111701 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Fix segmentation fault caused by stack overflow on deeply nested expressions (PR #111701)

2024-10-11 Thread Ilya Biryukov via cfe-commits
@@ -0,0 +1,1013 @@ +// RUN: %clang_cc1 %s -emit-llvm -o - -Wstack-exhausted -verify + +class AClass { +public: + AClass() {} + AClass &foo() { return *this; } +}; + +void test_bar() { + AClass a; + // expected-warning@* {{stack nearly exhausted; compilation time may suffer, a

[clang] [clang] Fix segmentation fault caused by stack overflow on deeply nested expressions (PR #111701)

2024-10-11 Thread Ilya Biryukov via cfe-commits
https://github.com/ilya-biryukov edited https://github.com/llvm/llvm-project/pull/111701 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][analyzer] Improve test and documentation in cstring NotNullTerminated checker (PR #112019)

2024-10-11 Thread Balázs Kéri via cfe-commits
https://github.com/balazske created https://github.com/llvm/llvm-project/pull/112019 CStringChecker has a sub-checker alpha.unix.cstring.NotNullTerminated which checks for invalid objects passed to string functions. The checker and its name are not exact and more functions could be checked, th

[clang] clang: Fix hipstdpar test relying on default target (PR #111975)

2024-10-11 Thread Alex Voicu via cfe-commits
AlexVlx wrote: @arsenm what are you actually trying to fix and what do you expect this to do? https://github.com/llvm/llvm-project/pull/111975 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c

[clang] [clang][analyzer] Improve test and documentation in cstring NotNullTerminated checker (PR #112019)

2024-10-11 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-static-analyzer-1 Author: Balázs Kéri (balazske) Changes CStringChecker has a sub-checker alpha.unix.cstring.NotNullTerminated which checks for invalid objects passed to string functions. The checker and its name are not exact and more functions

[clang] [clang][analyzer] Improve test and documentation in cstring NotNullTerminated checker (PR #112019)

2024-10-11 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Balázs Kéri (balazske) Changes CStringChecker has a sub-checker alpha.unix.cstring.NotNullTerminated which checks for invalid objects passed to string functions. The checker and its name are not exact and more functions could be checked,

[clang] [Modules] Add -cc1 -flate-module-map-file to load module maps after PCMs (PR #88893)

2024-10-11 Thread Ilya Biryukov via cfe-commits
https://github.com/ilya-biryukov closed https://github.com/llvm/llvm-project/pull/88893 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Modules] Add -cc1 -flate-module-map-file to load module maps after PCMs (PR #88893)

2024-10-11 Thread Ilya Biryukov via cfe-commits
ilya-biryukov wrote: The comments on this thread bugged me, so I managed to find a way to achieve the same effect without adding any new flags. #112015 should give the deduplication that I want to get here without introducing any new flags, through a tweak to the mechanism of non-affecting sou

[clang-tools-extra] [clangd] Simplify code with *Map::operator[] (NFC) (PR #111939)

2024-10-11 Thread Kazu Hirata via cfe-commits
https://github.com/kazutakahirata closed https://github.com/llvm/llvm-project/pull/111939 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] clang: Fix hipstdpar test relying on default target (PR #111975)

2024-10-11 Thread Shilei Tian via cfe-commits
@@ -1,21 +1,17 @@ -// REQUIRES: x86-registered-target -// REQUIRES: amdgpu-registered-target -// REQUIRES: system-linux shiltian wrote: I'm not sure if the removal of `system-linux` would cause issue. https://github.com/llvm/llvm-project/pull/111975

[clang] [clang][CGOpenMPRuntime] Avoid Type::getPointerTo() (NFC) (PR #112017)

2024-10-11 Thread Youngsuk Kim via cfe-commits
https://github.com/JOE1994 created https://github.com/llvm/llvm-project/pull/112017 `llvm::Type::getPointerTo()` is to be deprecated & removed soon. >From 542112f283387db666c54422c714e901bb898c84 Mon Sep 17 00:00:00 2001 From: Youngsuk Kim Date: Fri, 11 Oct 2024 10:06:57 -0500 Subject: [PATCH]

[clang] [clang][CGOpenMPRuntime] Avoid Type::getPointerTo() (NFC) (PR #112017)

2024-10-11 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-codegen Author: Youngsuk Kim (JOE1994) Changes `llvm::Type::getPointerTo()` is to be deprecated & removed soon. --- Full diff: https://github.com/llvm/llvm-project/pull/112017.diff 1 Files Affected: - (modified) clang/lib/CodeGen/CGOpenMPRuntim

[clang] [lld] [llvm] [mlir] [IR] Introduce `T` to `DataLayout` to represent flat address space if a target supports it (PR #108786)

2024-10-11 Thread Matt Arsenault via cfe-commits
arsenm wrote: > > The LangRef doesn't need to know why it's undesirable. It's like the n field > > `n` field? The following? > Yes. It's an optimization hint https://github.com/llvm/llvm-project/pull/108786 ___ cfe-commits mailing list cfe-commits@

[clang] clang: Fix hipstdpar test relying on default target (PR #111975)

2024-10-11 Thread Matt Arsenault via cfe-commits
@@ -1,21 +1,17 @@ -// REQUIRES: x86-registered-target -// REQUIRES: amdgpu-registered-target -// REQUIRES: system-linux arsenm wrote: This is a pile of workarounds, there's no reason any of these tests should be host dependent https://github.com/llvm/llvm-proj

[clang] [clang] Fix segmentation fault caused by stack overflow on deeply nested expressions (PR #111701)

2024-10-11 Thread Ilya Biryukov via cfe-commits
@@ -0,0 +1,1013 @@ +// RUN: %clang_cc1 %s -emit-llvm -o - -Wstack-exhausted -verify + +class AClass { +public: + AClass() {} + AClass &foo() { return *this; } +}; + +void test_bar() { + AClass a; + // expected-warning@* {{stack nearly exhausted; compilation time may suffer, a

[clang] [llvm] [mlir] [polly] [NFC] Rename `Intrinsic::getDeclaration` to `getOrInsertDeclaration` (PR #111752)

2024-10-11 Thread Daniel Chen via cfe-commits
DanielCChen wrote: I see. I don't have strong preference although it would be nice to use some function that exists. https://github.com/llvm/llvm-project/pull/111752 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-

[clang] [C++20][Modules] Load function body from the module that gives canonical decl (PR #111992)

2024-10-11 Thread Dmitry Polukhin via cfe-commits
https://github.com/dmpolukhin updated https://github.com/llvm/llvm-project/pull/111992 >From 0d02e79ced894769cfbea45a121670e933e2c886 Mon Sep 17 00:00:00 2001 From: Dmitry Polukhin Date: Fri, 11 Oct 2024 05:35:18 -0700 Subject: [PATCH 1/2] [C++20][Modules] Load function body from the module tha

[clang] [llvm] [mlir] [polly] [NFC] Rename `Intrinsic::getDeclaration` to `getOrInsertDeclaration` (PR #111752)

2024-10-11 Thread Rahul Joshi via cfe-commits
jurahul wrote: Makes sense, thanks. On Fri, Oct 11, 2024 at 9:49 AM Nikita Popov ***@***.***> wrote: > test_demangle should definitely *not* be updated when renaming functions. > > — > Reply to this email directly, view it on GitHub >

[clang] Add isTrivial() and isTriviallyCopyable() AST matchers (PR #90634)

2024-10-11 Thread via cfe-commits
@@ -5444,6 +5444,43 @@ AST_MATCHER(FunctionDecl, isDefaulted) { return Node.isDefaulted(); } +/// Matches trivial methods and types. higher-performance wrote: Ah I see, yeah. I don't think we want to support that since the other matchers don't do that eith

[clang] 5b25c31 - [OpenACC] Implement loop 'gang' clause. (#112006)

2024-10-11 Thread via cfe-commits
Author: Erich Keane Date: 2024-10-11T09:05:19-07:00 New Revision: 5b25c31351ad1b10a3819411379b3258869c1e1b URL: https://github.com/llvm/llvm-project/commit/5b25c31351ad1b10a3819411379b3258869c1e1b DIFF: https://github.com/llvm/llvm-project/commit/5b25c31351ad1b10a3819411379b3258869c1e1b.diff L

[clang] [llvm] [SystemZ][z/OS] Add new openFileForReadBinary function, and pass IsText parameter to getBufferForFile (PR #111723)

2024-10-11 Thread Abhina Sree via cfe-commits
https://github.com/abhina-sree updated https://github.com/llvm/llvm-project/pull/111723 >From c1676e48a587e10ba54c28e99192fd5e6a36f72e Mon Sep 17 00:00:00 2001 From: Abhina Sreeskantharajan Date: Wed, 9 Oct 2024 13:23:41 -0400 Subject: [PATCH 1/3] [SystemZ][z/OS] Add new openFileForReadBinary f

[clang] [OpenACC] Implement loop 'gang' clause. (PR #112006)

2024-10-11 Thread Erich Keane via cfe-commits
https://github.com/erichkeane closed https://github.com/llvm/llvm-project/pull/112006 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [SystemZ][z/OS] Add new openFileForReadBinary function, and pass IsText parameter to getBufferForFile (PR #111723)

2024-10-11 Thread Abhina Sree via cfe-commits
https://github.com/abhina-sree updated https://github.com/llvm/llvm-project/pull/111723 >From c1676e48a587e10ba54c28e99192fd5e6a36f72e Mon Sep 17 00:00:00 2001 From: Abhina Sreeskantharajan Date: Wed, 9 Oct 2024 13:23:41 -0400 Subject: [PATCH 1/3] [SystemZ][z/OS] Add new openFileForReadBinary f

[clang-tools-extra] 3f156ef - [clangd] Simplify code with *Map::operator[] (NFC) (#111939)

2024-10-11 Thread via cfe-commits
Author: Kazu Hirata Date: 2024-10-11T09:00:39-07:00 New Revision: 3f156ef4cc333bcf9389addd1e0054b654501d7e URL: https://github.com/llvm/llvm-project/commit/3f156ef4cc333bcf9389addd1e0054b654501d7e DIFF: https://github.com/llvm/llvm-project/commit/3f156ef4cc333bcf9389addd1e0054b654501d7e.diff L

[clang] [llvm] [mlir] [polly] [NFC] Rename `Intrinsic::getDeclaration` to `getOrInsertDeclaration` (PR #111752)

2024-10-11 Thread Daniel Chen via cfe-commits
DanielCChen wrote: @jurahul It seems this PR is missing a test file `libcxxabi/test/test_demangle.pass.cpp`. It has a line of ``` {"_ZN4llvm9Intrinsic14getDeclarationEPNS_6ModuleENS0_2IDEPPKNS_4TypeEj", "llvm::Intrinsic::getDeclaration(llvm::Module*, llvm::Intrinsic::ID, llvm::Type const**,

[clang] [llvm] [mlir] [polly] [NFC] Rename `Intrinsic::getDeclaration` to `getOrInsertDeclaration` (PR #111752)

2024-10-11 Thread Rahul Joshi via cfe-commits
jurahul wrote: Yes I purposely did not change it as it just seems to test mangling and using this function as an example. I can change it if it makes sense. On Fri, Oct 11, 2024 at 9:10 AM Daniel Chen ***@***.***> wrote: > @jurahul > It seems this PR is missing a te

[clang] clang: Fix hipstdpar test relying on default target (PR #111975)

2024-10-11 Thread Matt Arsenault via cfe-commits
arsenm wrote: > @arsenm what are you actually trying to fix and what do you expect this to do? Fix not running tests except on linux. We should have maximum host test coverage, and this test has no reason to depend on the host. All it needs is the explicit target instead of relying on the defa

[clang] [Clang] Add explicit visibility symbol macros (PR #108276)

2024-10-11 Thread Aaron Ballman via cfe-commits
@@ -0,0 +1,67 @@ +//===-- clang/Support/Compiler.h - Compiler abstraction support -*- 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: Apa

[clang] [Fuchsia][cmake][NFC] Indent nested loops in Fuchsia-stage2.cmake (PR #112020)

2024-10-11 Thread Paul Kirth via cfe-commits
https://github.com/ilovepi created https://github.com/llvm/llvm-project/pull/112020 None >From 07141c444024adaf80dc6fe3cc5378de6d641883 Mon Sep 17 00:00:00 2001 From: Paul Kirth Date: Fri, 11 Oct 2024 16:36:37 + Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=

[clang] [Fuchsia][cmake][NFC] Indent nested loops in Fuchsia-stage2.cmake (PR #112020)

2024-10-11 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Paul Kirth (ilovepi) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/112020.diff 1 Files Affected: - (modified) clang/cmake/caches/Fuchsia-stage2.cmake (+36-36) ``diff diff --git a/clang/cmake/caches/Fuchsia

[clang] clang: Fix hipstdpar test relying on default target (PR #111975)

2024-10-11 Thread Matt Arsenault via cfe-commits
arsenm wrote: Window bot passed, which was the important bit. Linux failed on a different test entirely https://github.com/llvm/llvm-project/pull/111975 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/li

[clang] Make PCH's respect any VFS specified. (PR #106577)

2024-10-11 Thread Ben Langmuir via cfe-commits
benlangmuir wrote: > We tried setting -working-directory=Z:/working, but the VFS requires that > this is a real path. What error are you seeing if it's not? Is it trying to set it too early before the VFS is created or something? https://github.com/llvm/llvm-project/pull/106577 __

[clang] [Clang] [Sema] Don't crash on unexpanded pack in invalid block literal (PR #110762)

2024-10-11 Thread John McCall via cfe-commits
https://github.com/rjmccall approved this pull request. Nice, this looks really clean. Thanks for taking this on! https://github.com/llvm/llvm-project/pull/110762 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/m

[clang] [Clang] fix overload resolution for object parameters with top-level cv-qualifiers in member functions (PR #110435)

2024-10-11 Thread Oleksandr T. via cfe-commits
@@ -1422,8 +1422,14 @@ static bool IsOverloadOrOverrideImpl(Sema &SemaRef, FunctionDecl *New, // the implicit object parameter are of the same type. auto NormalizeQualifiers = [&](const CXXMethodDecl *M, Qualifiers Q) { -if (M->isExplicitObjectMemberFunction()) +i

<    1   2   3   4   5   6   >