[clang] [Clang] Fix parsing of reversible type traits in template arguments (PR #95969)

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

[clang] 1fe406f - [Clang] Fix parsing of reversible type traits in template arguments (#95969)

2024-07-12 Thread via cfe-commits
Author: cor3ntin Date: 2024-07-13T08:53:02+02:00 New Revision: 1fe406fffe11dad0457a4d214ce67bf492196145 URL: https://github.com/llvm/llvm-project/commit/1fe406fffe11dad0457a4d214ce67bf492196145 DIFF: https://github.com/llvm/llvm-project/commit/1fe406fffe11dad0457a4d214ce67bf492196145.diff LOG:

[clang] [llvm] [Coverage][MC/DC] Show uncoverable and unreachable conditions (PR #94137)

2024-07-12 Thread via cfe-commits
https://github.com/Lambdaris updated https://github.com/llvm/llvm-project/pull/94137 >From 23669f721d056d6e4961e2572e438e5c1f91cab1 Mon Sep 17 00:00:00 2001 From: Lambdaris Date: Sun, 2 Jun 2024 10:19:31 +0800 Subject: [PATCH 1/3] [coverage] Mark branches with either counter is zero as folded

[clang] [llvm] [Coverage][MC/DC] Show uncoverable and unreachable conditions (PR #94137)

2024-07-12 Thread via cfe-commits
Lambdaris wrote: Add an option "--mcdc-exclude" to control which conditions can be excluded from coverage. By default it's "constant,unreachable". https://github.com/llvm/llvm-project/pull/94137 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [llvm] [Coverage][MC/DC] Show uncoverable and unreachable conditions (PR #94137)

2024-07-12 Thread via cfe-commits
https://github.com/Lambdaris updated https://github.com/llvm/llvm-project/pull/94137 >From 23669f721d056d6e4961e2572e438e5c1f91cab1 Mon Sep 17 00:00:00 2001 From: Lambdaris Date: Sun, 2 Jun 2024 10:19:31 +0800 Subject: [PATCH 1/3] [coverage] Mark branches with either counter is zero as folded

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

2024-07-12 Thread Shafik Yaghmour via cfe-commits
https://github.com/shafik requested changes to this pull request. It looks like we are using the same pattern that fix is using in `ASTStmtReader::VisitRequiresExpr` which gives me some comfort but I am not happy that we have a relatively complex and easy to get wrong operation repeated in thr

[clang] [clang] Limit alignment for emitted vectors (PR #98629)

2024-07-12 Thread Eli Friedman via cfe-commits
@@ -133,6 +133,7 @@ struct TransferrableTargetInfo { unsigned short SuitableAlign; unsigned short NewAlign; unsigned MaxVectorAlign; efriedma-quic wrote: It's used in the same ASTContext code that's modified by this patch. Probably fine to adjust it fo

[clang] [clang] Limit alignment for emitted vectors (PR #98629)

2024-07-12 Thread Eli Friedman via cfe-commits
efriedma-quic wrote: The size of ArgFlagsTy is, as far as I can tell, basically irrelevant: it's only used as part of computing the calling convention in isel. I don't think anyone will notice if it's a few bytes larger. The non-byval alignment limit was last adjusted in https://reviews.llvm

[clang] 66e6df2 - [clang][Interp] Fix one-past-end pointers going back into the block

2024-07-12 Thread Timm Bäder via cfe-commits
Author: Timm Bäder Date: 2024-07-13T07:25:47+02:00 New Revision: 66e6df22b5a509c16e50364d72b1a40bacaea91a URL: https://github.com/llvm/llvm-project/commit/66e6df22b5a509c16e50364d72b1a40bacaea91a DIFF: https://github.com/llvm/llvm-project/commit/66e6df22b5a509c16e50364d72b1a40bacaea91a.diff LO

[clang] 634128b - [clang][Interp][NFC] Remove some unused includes

2024-07-12 Thread Timm Bäder via cfe-commits
Author: Timm Bäder Date: 2024-07-13T07:25:47+02:00 New Revision: 634128be4610a4d85d92935109ed7761f0e06af1 URL: https://github.com/llvm/llvm-project/commit/634128be4610a4d85d92935109ed7761f0e06af1 DIFF: https://github.com/llvm/llvm-project/commit/634128be4610a4d85d92935109ed7761f0e06af1.diff LO

[clang] 7d59020 - [clang][Interp][NFC] Remove unused include

2024-07-12 Thread Timm Bäder via cfe-commits
Author: Timm Bäder Date: 2024-07-13T07:25:47+02:00 New Revision: 7d5902025d20e00b1865d02dfe514fb35259ae2a URL: https://github.com/llvm/llvm-project/commit/7d5902025d20e00b1865d02dfe514fb35259ae2a DIFF: https://github.com/llvm/llvm-project/commit/7d5902025d20e00b1865d02dfe514fb35259ae2a.diff LO

[clang] [SPARC][clang] Add -m(no-)v8plus flags handling (PR #98713)

2024-07-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-driver Author: Koakuma (koachan) Changes Implement handling for `-m(no-)v8plus` flags to allow the user to switch between V8 and V8+ mode with 32-bit code. Currently it only toggles the V9 feature bit, ABI and codegen changes will be done in fut

[clang] [SPARC][clang] Add -m(no-)v8plus flags handling (PR #98713)

2024-07-12 Thread via cfe-commits
https://github.com/koachan created https://github.com/llvm/llvm-project/pull/98713 Implement handling for `-m(no-)v8plus` flags to allow the user to switch between V8 and V8+ mode with 32-bit code. Currently it only toggles the V9 feature bit, ABI and codegen changes will be done in future pa

[clang] [clang][Sema] Add support for OpenBSD's syslog format attribute (PR #97366)

2024-07-12 Thread Brad Smith via cfe-commits
brad0 wrote: > Hmm, this function also seems to exist on Linux, though. Additionally, from > my `man syslog` docs: I'm not sure what you mean by that in relation to this addition of a format attribute. > > The remaining arguments are a format, as in `printf(3)`, and any arguments > > require

[clang] [clang] Fix crash in concept deprecation (PR #98622)

2024-07-12 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov requested changes to this pull request. See previous comments, I think this is primarily an error recovery issue on the decltype, we shouldn't let it escape an undeduced auto, as that would confuse further analysis. https://github.com/llvm/llvm-project/pull/98622 __

[clang] [compiler-rt] [libcxx] [cmake] Add hexagon-linux cmake cache files (PR #98712)

2024-07-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Brian Cain (androm3da) Changes These can be used to create a fully-bootstrapped toolchain to target hexagon {baremetal,linux} with scripts like the ones in https://github.com/quic/toolchain_for_hexagon --- Full diff: https://github.com/l

[clang] [compiler-rt] [libcxx] [cmake] Add hexagon-linux cmake cache files (PR #98712)

2024-07-12 Thread Brian Cain via cfe-commits
https://github.com/androm3da created https://github.com/llvm/llvm-project/pull/98712 These can be used to create a fully-bootstrapped toolchain to target hexagon {baremetal,linux} with scripts like the ones in https://github.com/quic/toolchain_for_hexagon >From 3cb0ce53c66815496ac0b5296ee36aa

[clang] [clang] Fix crash in concept deprecation (PR #98622)

2024-07-12 Thread Matheus Izvekov via cfe-commits
@@ -7416,10 +7416,11 @@ NamedDecl *Sema::ActOnVariableDeclarator( tryToFixVariablyModifiedVarType(TInfo, R, D.getIdentifierLoc(), /*DiagID=*/0); - if (const AutoType *AutoT = R->getAs()) -CheckConstrainedAuto( -AutoT, -

[clang] [clang] Fix crash in concept deprecation (PR #98622)

2024-07-12 Thread Matheus Izvekov via cfe-commits
@@ -7416,10 +7416,11 @@ NamedDecl *Sema::ActOnVariableDeclarator( tryToFixVariablyModifiedVarType(TInfo, R, D.getIdentifierLoc(), /*DiagID=*/0); - if (const AutoType *AutoT = R->getAs()) -CheckConstrainedAuto( -AutoT, -

[clang] [Clang][NFCI] Remove records of unsatisfied atomic expressions in ConstraintSatisfaction (PR #98654)

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

[clang] fb19649 - [Clang][NFCI] Remove records of unsatisfied atomic expressions in ConstraintSatisfaction (#98654)

2024-07-12 Thread via cfe-commits
Author: Younan Zhang Date: 2024-07-13T11:49:21+08:00 New Revision: fb19649535d3d31a748409542947d09291643204 URL: https://github.com/llvm/llvm-project/commit/fb19649535d3d31a748409542947d09291643204 DIFF: https://github.com/llvm/llvm-project/commit/fb19649535d3d31a748409542947d09291643204.diff

[clang] [Clang][NFCI] Remove records of unsatisfied atomic expressions in ConstraintSatisfaction (PR #98654)

2024-07-12 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov approved this pull request. LGTM, I agree that we currently don't use it, so we might as well remove it. But the overall situation is all wrong, we shouldn't be storing fully formed diagnostic strings in the AST, we should be storing them as PartialDiagnostics. But

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

2024-07-12 Thread Matheus Izvekov via cfe-commits
https://github.com/mizvekov approved this pull request. LGTM. If it does fix the mentioned issue, please add a release note. https://github.com/llvm/llvm-project/pull/98699 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org

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

2024-07-12 Thread Shafik Yaghmour via cfe-commits
shafik wrote: > The sad part is that MSVC and some old versions of GCC don't like `template` > being added here :( It looks like the description provides a workaround for gcc but a bug report against MSVC should be filed if we don' have one already. https://github.com/llvm/llvm-project/pull/9

[clang] [X86_32][C++] fix 0 sized struct case in vaarg. (PR #86388)

2024-07-12 Thread Longsheng Mou via cfe-commits
CoTinker wrote: Inconsistency here is inevitable. Such struct `SuperEmpty { struct{ int a[0];} b;}` in c++ mode can not be ignored in i386 for that c++ fields are never empty, and its TypeInfo.Width = TypeInfo.Align = 0. https://github.com/llvm/llvm-project/blob/594989918bf9e178f3b7c5e200d700d0

[clang] [Clang] Don't crash if input file is not a module. (PR #98439)

2024-07-12 Thread Shafik Yaghmour via cfe-commits
shafik wrote: I don't think the build error is related to this change. https://github.com/llvm/llvm-project/pull/98439 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Don't crash if input file is not a module. (PR #98439)

2024-07-12 Thread Shafik Yaghmour via cfe-commits
https://github.com/shafik commented: Thank you for this fix! https://github.com/llvm/llvm-project/pull/98439 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] fix cast failures by adjusting the resolution of record declaration contexts to handle semantic and lexical distinctions (PR #96228)

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

[clang] [test] Precommit tests for PR96025, NFC (PR #98704)

2024-07-12 Thread via cfe-commits
vfdff wrote: require Linux for this test as it time out on Windows ? https://github.com/llvm/llvm-project/pull/98704 ___ 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 via cfe-commits
vfdff wrote: hi @efriedma-quic I find the case will timeout even without this changes ,so would it reasonable to add **require Linux for this test** now ? https://buildkite.com/llvm-project/github-pull-requests/builds/80879#0190a9c3-b007-4daa-9656-0c8f00601ee9 https://github.com/llvm/llvm-p

[clang] [clang] Add option to opt out of the missing_dependent_template_keyword diagnostic (PR #98613)

2024-07-12 Thread Pranav Kant via cfe-commits
pranavk wrote: This doesn't seem to handle all the cases? See https://github.com/abseil/abseil-cpp/pull/1711 for absl build failures that are not silenced by using this option. https://github.com/llvm/llvm-project/pull/98613 ___ cfe-commits mailing l

[clang] [clang] Limit alignment for emitted vectors (PR #98629)

2024-07-12 Thread Phoebe Wang via cfe-commits
@@ -133,6 +133,7 @@ struct TransferrableTargetInfo { unsigned short SuitableAlign; unsigned short NewAlign; unsigned MaxVectorAlign; phoebewang wrote: What's this used for? Can we limit it to 2^14? https://github.com/llvm/llvm-project/pull/98629 ___

[clang] [clang] Limit alignment for emitted vectors (PR #98629)

2024-07-12 Thread Phoebe Wang via cfe-commits
phoebewang wrote: > > max aligment [...] LLVM Verifier accepts is 2^14 > > The verifier limit in most places is 2^32. It looks like 2^14 is specifically > for ByVal. And there isn't really any good reason for that limit; it was > arbitrarily chosen in > [66011c1](https://github.com/llvm/llvm-

[clang] [test] Precommit tests for PR96025, NFC (PR #98704)

2024-07-12 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Allen (vfdff) Changes Check whether timeout exists in the Windows operating system. --- Full diff: https://github.com/llvm/llvm-project/pull/98704.diff 1 Files Affected: - (added) clang/test/CodeGen/math-libcalls-tbaa.cpp (+38) `

[clang] [test] Precommit tests for PR96025, NFC (PR #98704)

2024-07-12 Thread via cfe-commits
https://github.com/vfdff created https://github.com/llvm/llvm-project/pull/98704 Check whether timeout exists in the Windows operating system. >From 1b8503202474c15aa46fc4cc9151003a8d3ca336 Mon Sep 17 00:00:00 2001 From: zhongyunde 00443407 Date: Fri, 12 Jul 2024 21:09:46 -0400 Subject: [PATCH]

[clang] [C++20] [Modules] [Itanium ABI] Generate the vtable in the module unit of dynamic classes (PR #75912)

2024-07-12 Thread Matheus Izvekov via cfe-commits
mizvekov wrote: > Clarifying a couple of things here I think we need the distinction between simple NFC, for trivial, non-controversial and non-surprising changes like some renames, small whitespace / formatting change, documentation changes, and complex NFC, which requires some thinking and

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

2024-07-12 Thread Krystian Stasiowski via cfe-commits
sdkrystian wrote: @pranavk see #98613 https://github.com/llvm/llvm-project/pull/98547 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

2024-07-12 Thread Xiang Li 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

[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] [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] 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

[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] [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] [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 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] 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] [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] [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] [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] [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 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] [HLSL] Rework implicit conversion sequences This PR reworks

[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] 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] [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-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-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
@@ -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
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-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] [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] [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] [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
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] [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] [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] [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
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] [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] [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] [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
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 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] 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] 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] [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] [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] [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] 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/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] [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/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] [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-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] [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] [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] [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 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] [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] 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] 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 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 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] [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] 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] [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

  1   2   3   4   5   >