https://github.com/davemgreen approved this pull request.
Thanks, LGTM
https://github.com/llvm/llvm-project/pull/132368
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1677,7 +1676,8 @@ being treated as convergent when it's safe.
__device__ int example(void) {
float x;
-[[clang::noconvergent]] x = bar(x);
+[[clang::noconvergent]] x = bar(x); // no effect on convergence
+[[clang::noconvergent]] { asm volatile ("nop"); }
jroelofs wrote:
> Hi @jroelofs @DanielKristofKiss. I am not sure how to detect on other
> platforms. Suggestions for apple and windows welcome.
The place to look for feature tests on Apple platforms is here:
https://github.com/apple-oss-distributions/xnu/blob/main/osfmk/arm/cpu_capabilities_pu
@@ -33,7 +33,7 @@ enum CPUFeatures {
FEAT_FP,
FEAT_SIMD,
FEAT_CRC,
- RESERVED_FEAT_SHA1, // previously used and now ABI legacy
+ FEAT_CSSC,
jroelofs wrote:
Is re-using this index okay re: the ABI legacy thing?
https://github.com/llvm/llvm-project/pull
https://github.com/Sunday111 updated
https://github.com/llvm/llvm-project/pull/132443
>From 9a88ff05af21a5d5357780293f2d011ffe457041 Mon Sep 17 00:00:00 2001
From: Kostiantyn Lazukin
Date: Thu, 20 Mar 2025 16:54:06 +
Subject: [PATCH] [Driver] Search for compiler-rt libraries in sysroot/lib
Author: Matheus Izvekov
Date: 2025-03-24T13:18:07-03:00
New Revision: d447c6e9b73a43f865e4900a8374b242ce4c4bf6
URL:
https://github.com/llvm/llvm-project/commit/d447c6e9b73a43f865e4900a8374b242ce4c4bf6
DIFF:
https://github.com/llvm/llvm-project/commit/d447c6e9b73a43f865e4900a8374b242ce4c4bf6.dif
@@ -872,6 +883,16 @@ def ProcessorFeatures {
list Carmel = [HasV8_2aOps, FeatureNEON, FeatureSHA2,
FeatureAES,
FeatureFullFP16, FeatureCRC, FeatureLSE,
FeatureRAS, FeatureRDM,
FeatureFPARMv8];
+ li
https://github.com/mizvekov updated
https://github.com/llvm/llvm-project/pull/132626
>From 6c1a554cfdb7028fccae5f99863c36118fa0226a Mon Sep 17 00:00:00 2001
From: Matheus Izvekov
Date: Sat, 22 Mar 2025 17:29:16 -0300
Subject: [PATCH 1/3] Revert "[Clang] Distinguish expanding-pack-in-place cases
Author: Fraser Cormack
Date: 2025-03-24T16:21:59Z
New Revision: 7e22b0903150c7f722b5290155d251b50d839c3a
URL:
https://github.com/llvm/llvm-project/commit/7e22b0903150c7f722b5290155d251b50d839c3a
DIFF:
https://github.com/llvm/llvm-project/commit/7e22b0903150c7f722b5290155d251b50d839c3a.diff
LOG
@@ -33,7 +33,7 @@ enum CPUFeatures {
FEAT_FP,
FEAT_SIMD,
FEAT_CRC,
- RESERVED_FEAT_SHA1, // previously used and now ABI legacy
+ FEAT_CSSC,
labrinea wrote:
Yeap, we only need to make sure GNU is doing the same cc @Wilco1
https://github.com/llvm/llvm-p
rorth wrote:
Usually the SPARC-only `clang` will reject any non-`sparc*` triple as
unsupported. However, there are cases where they still work; I never
understood when this works and when it doesn't.
https://github.com/llvm/llvm-project/pull/132723
https://github.com/bcardosolopes approved this pull request.
LGTM after nit
https://github.com/llvm/llvm-project/pull/132468
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/132782
We can't pass the AtomicType along to ASTContext::getFloatTypeSemantics.
>From 88ce8a55d9e8fea57ff71989f6a5cd2fe9b505ec Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Mon, 24 Mar 2025 18:13
@@ -23,3 +22,67 @@ void test(A &a, B &b) {
const A &&ar10 = static_cast(xvalue());
const A &&ar11 = static_cast(xvalue());
}
+
+struct C : private A { // expected-note 4 {{declared private here}}
+C&& that();
+
+void f() {
+(void)static_cast(*this);
+
@@ -143,6 +147,114 @@ class CIRBaseBuilderTy : public mlir::OpBuilder {
return createCast(loc, cir::CastKind::bitcast, src, newTy);
}
+ mlir::Value createBinop(mlir::Value lhs, cir::BinOpKind kind,
+ const llvm::APInt &rhs) {
+return create(
https://github.com/bcardosolopes edited
https://github.com/llvm/llvm-project/pull/132420
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/quic-garvgupt updated
https://github.com/llvm/llvm-project/pull/121829
>From e07a4cd4e0ff77f74b66695923bc998904c14746 Mon Sep 17 00:00:00 2001
From: Garvit Gupta
Date: Fri, 13 Dec 2024 05:31:56 -0800
Subject: [PATCH] [Driver] Teach Barmetal toolchain about GCC installation
T
tarunprabhu wrote:
> > If you are soliciting reviews, you could also use the "Reviewers" box on
> > the top right of this page
>
> I would, but I do not currently have the right permissions to use the box.
> Hence the ccs - I need someone else to do it.
Huh. I didn't realize that one needed s
Author: Carlos Galvez
Date: 2025-03-24T20:47:57+01:00
New Revision: 0fb4ef40b12d08e4a4966eeac6a395b669d63237
URL:
https://github.com/llvm/llvm-project/commit/0fb4ef40b12d08e4a4966eeac6a395b669d63237
DIFF:
https://github.com/llvm/llvm-project/commit/0fb4ef40b12d08e4a4966eeac6a395b669d63237.diff
spall wrote:
> > > For example, for `abs`, it still depends on the _HLSL_16BIT_AVAILABILITY
> > > availability attribute. Does this PR intend to keep abs overloads using
> > > half "unexposed"? Or should that overload for abs be exposed too?
> >
> >
> > Oh I see what you are referring to. Tha
https://github.com/llvm-beanz commented:
this is not NFC, so we should verify that we can call these intrinsics with
`half` values even if 16-bit types aren't enabled, and that they properly
codegen to 32-bit variants.
https://github.com/llvm/llvm-project/pull/132804
__
bob80905 wrote:
> > For example, for `abs`, it still depends on the _HLSL_16BIT_AVAILABILITY
> > availability attribute. Does this PR intend to keep abs overloads using
> > half "unexposed"? Or should that overload for abs be exposed too?
>
> Oh I see what you are referring to. That might be m
mrkajetanp wrote:
> If you are soliciting reviews, you could also use the "Reviewers" box on the
> top right of this page
I would, but I do not currently have the right permissions to use the box.
Hence the ccs - I need someone else to do it.
https://github.com/llvm/llvm-project/pull/132801
https://github.com/quic-garvgupt edited
https://github.com/llvm/llvm-project/pull/121829
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jan Svoboda (jansvoboda11)
Changes
This makes it so that `CompilerInvocation` can be the only entity that manages
ownership of `HeaderSearchOptions`, making it possible to implement
copy-on-write semantics.
---
Patch is 31.81 KiB, trunc
https://github.com/spall edited https://github.com/llvm/llvm-project/pull/132804
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4749,6 +4753,15 @@ class Sema final : public SemaBase {
CXXRecordDecl *getStdBadAlloc() const;
EnumDecl *getStdAlignValT() const;
+ ClassTemplateDecl *getStdTypeIdentity() const;
+ std::optional InstantiateSpecializedTypeIdentity(QualType Subject);
o
efriedma-quic wrote:
https://llvm.org/docs/GitHub.html#backporting-fixes-to-the-release-branches
https://llvm.org/docs/HowToReleaseLLVM.html#release-patch-rules .
https://github.com/llvm/llvm-project/pull/126382
___
cfe-commits mailing list
cfe-commit
Midar wrote:
Thanks, that looks like it'll have to wait for 21.
https://github.com/llvm/llvm-project/pull/126382
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
@llvm/pr-subscribers-clang
Author: Nico Weber (nico)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/132834.diff
3 Files Affected:
- (modified) clang/lib/Driver/ToolChains/Darwin.cpp (+41)
- (modified) clang/lib/Driver
@@ -2997,6 +2997,7 @@ static void RenderFloatingPointOptions(const ToolChain
&TC, const Driver &D,
LangOptions::ComplexRangeKind Range = LangOptions::ComplexRangeKind::CX_None;
std::string ComplexRangeStr = "";
std::string GccRangeComplexOption = "";
+ std::string LastC
@@ -0,0 +1,17 @@
+// RUN: %clang_cc1 -std=c99 -fsyntax-only -verify %s
pinskia wrote:
> This feels like a pretty bare bones set of tests, I think we can do better.
>
> For example, it has been discussed that there should not be side effects,
> let's verify that
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-modules
Author: Michael Spencer (Bigcheese)
Changes
Instead of eagerly populating the `clang::ModuleMap` when looking up a module
by name, this patch changes `HeaderSearch` to only load the modules that are
actually use
@@ -51,63 +51,40 @@ sub silent_system {
# Compiler command setup.
##===--===##
-# Search in the PATH if the compiler exists
-sub SearchInPath {
-my $file = shift;
-foreach my $dir (split (':', $ENV{PA
Pierre-vh wrote:
@shiltian PSDB passed, is it enough to land?
https://github.com/llvm/llvm-project/pull/128509
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -143,6 +147,114 @@ class CIRBaseBuilderTy : public mlir::OpBuilder {
return createCast(loc, cir::CastKind::bitcast, src, newTy);
}
+ mlir::Value createBinop(mlir::Value lhs, cir::BinOpKind kind,
+ const llvm::APInt &rhs) {
+return create(
https://github.com/mydeveloperday approved this pull request.
https://github.com/llvm/llvm-project/pull/132519
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -826,6 +826,129 @@ def ForOp : CIR_Op<"for", [LoopOpInterface,
NoRegionArguments]> {
}];
}
+//===--===//
+// BinOp
+//===--===//
+
+//
https://github.com/shiltian approved this pull request.
https://github.com/llvm/llvm-project/pull/128509
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/131942
>From 321ac988a49489d910bf8ba90a28d05db853cc0d Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Tue, 18 Mar 2025 13:19:24 -0700
Subject: [PATCH] [IRBuilder] Add new overload for CreateIntrinsic
Add a new `Creat
https://github.com/sarnex updated
https://github.com/llvm/llvm-project/pull/126956
>From 044158c176316e6c9f44ce1487ff631c8a0d15c4 Mon Sep 17 00:00:00 2001
From: "Sarnie, Nick"
Date: Tue, 4 Mar 2025 11:16:34 -0800
Subject: [PATCH 1/4] [clang] Add isOffloadingTarget function to LangOpts
Signed-o
https://github.com/frasercrmck created
https://github.com/llvm/llvm-project/pull/132758
None
>From 8e2c04878b87a7da3929836df2d20c4c4c701312 Mon Sep 17 00:00:00 2001
From: Fraser Cormack
Date: Mon, 24 Mar 2025 15:28:20 +
Subject: [PATCH] [libclc] Add missing license headers to source IR fil
https://github.com/bob80905 approved this pull request.
https://github.com/llvm/llvm-project/pull/131032
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bogner approved this pull request.
https://github.com/llvm/llvm-project/pull/131032
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Paul Schwabauer (koplas)
Changes
The File ID is incorrectly calculated, resulting in an out-of-bounds access.
The test code is more complex because the File fetching only happens in
specific scenarios.
---
Full diff: https://github.com/l
mrkajetanp wrote:
cc @tblah @DavidTruby
https://github.com/llvm/llvm-project/pull/132801
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Kajetan Puchalski (mrkajetanp)
Changes
Add -f[no-]slp-vectorize to the flang driver.
Add corresponding -fvectorize-slp to the flang frontend.
---
Full diff: https://github.com/llvm/llvm-project/pull/132801.diff
6 Files Affected:
https://github.com/mrkajetanp created
https://github.com/llvm/llvm-project/pull/132801
Add -f[no-]slp-vectorize to the flang driver.
Add corresponding -fvectorize-slp to the flang frontend.
>From d9b2a86d01688b69b8f116f6b52caeec8cd6e756 Mon Sep 17 00:00:00 2001
From: Kajetan Puchalski
Date: Mo
https://github.com/koplas created
https://github.com/llvm/llvm-project/pull/132802
The File ID is incorrectly calculated, resulting in an out-of-bounds access.
The test code is more complex because the File fetching only happens in
specific scenarios.
>From cad802f9c91f12b8db26bc63eef13097474
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
@@ -0,0 +1,534 @@
+#!/usr/bin/env python3
+
+"""generate_unsupported_in_drivermode.py
+
+This script generates Lit regression test files that validate that options are
only exposed to intended driver modes.
+
+The options and driver modes are parsed from Options.td, whose path sh
@@ -0,0 +1,534 @@
+#!/usr/bin/env python3
+
+"""generate_unsupported_in_drivermode.py
+
+This script generates Lit regression test files that validate that options are
only exposed to intended driver modes.
+
+The options and driver modes are parsed from Options.td, whose path sh
@@ -0,0 +1,534 @@
+#!/usr/bin/env python3
+
+"""generate_unsupported_in_drivermode.py
+
+This script generates Lit regression test files that validate that options are
only exposed to intended driver modes.
+
+The options and driver modes are parsed from Options.td, whose path sh
@@ -0,0 +1,534 @@
+#!/usr/bin/env python3
+
+"""generate_unsupported_in_drivermode.py
+
+This script generates Lit regression test files that validate that options are
only exposed to intended driver modes.
+
+The options and driver modes are parsed from Options.td, whose path sh
https://github.com/ojhunt commented:
> *[Reviewable](https://reviewable.io/reviews/llvm/llvm-project/113510)*
> status: 0 of 54 files reviewed, 119 unresolved discussions (waiting on
> @AaronBallman, @Bigcheese, @ChuanqiXu9, @cor3ntin, @efriedma-quic,
> @erichkeane, @mizvekov, @ogiroux, @Sir
https://github.com/nico updated https://github.com/llvm/llvm-project/pull/132834
>From ec8faf518af6384203c776fdd9928ef06e9b5be7 Mon Sep 17 00:00:00 2001
From: Nico Weber
Date: Mon, 24 Mar 2025 15:56:49 -0400
Subject: [PATCH 1/2] [clang] Upstream Mach-O ptrauth driver defaults
---
clang/lib/Dri
@@ -1277,19 +1278,27 @@ QualType
Sema::CheckNonTypeTemplateParameterType(TypeSourceInfo *&TSI,
TSI = SubstAutoTypeSourceInfoDependent(TSI);
}
- return CheckNonTypeTemplateParameterType(TSI->getType(), Loc);
+ return CheckNonTypeTemplateParameterType(TSI->getType(), Lo
@@ -0,0 +1,534 @@
+#!/usr/bin/env python3
+
+"""generate_unsupported_in_drivermode.py
+
+This script generates Lit regression test files that validate that options are
only exposed to intended driver modes.
+
+The options and driver modes are parsed from Options.td, whose path sh
Author: Helena Kotas
Date: 2025-03-24T14:56:05-07:00
New Revision: 9a82f742b4976650827674e58c3c2b0b412b56f0
URL:
https://github.com/llvm/llvm-project/commit/9a82f742b4976650827674e58c3c2b0b412b56f0
DIFF:
https://github.com/llvm/llvm-project/commit/9a82f742b4976650827674e58c3c2b0b412b56f0.diff
https://github.com/hulxv updated
https://github.com/llvm/llvm-project/pull/131280
>From bf9bd4156cb7f652c9cf0477f537e5c58b470448 Mon Sep 17 00:00:00 2001
From: hulxv
Date: Fri, 14 Mar 2025 07:39:15 +0200
Subject: [PATCH 01/12] [clang-doc] [feat] add `--repository-line-prefix`
argument (fix #59
@@ -177,14 +177,45 @@
// RUN: %clang -### -target x86_64 -ffast-math -fcomplex-arithmetic=basic -c
%s 2>&1 \
// RUN: | FileCheck --check-prefix=BASIC %s
-// BASIC: -complex-range=basic
-// FULL: -complex-range=full
-// PRMTD: -complex-range=promoted
-// BASIC-NOT: -complex-
dwblaikie wrote:
> The link from the class to the specific vtable even seems mildly incorrect,
> in that during object construction the vtable will go through several
> different values, not just one.
Not sure I follow this - the object is only of the type, in some sense, when it
is pointing
bob80905 wrote:
> > > > For example, for `abs`, it still depends on the
> > > > _HLSL_16BIT_AVAILABILITY availability attribute. Does this PR intend to
> > > > keep abs overloads using half "unexposed"? Or should that overload for
> > > > abs be exposed too?
> > >
> > >
> > > Oh I see what y
@@ -0,0 +1,81 @@
+//===--===//
+//
+// 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: Apac
https://github.com/hulxv updated
https://github.com/llvm/llvm-project/pull/131280
>From bf9bd4156cb7f652c9cf0477f537e5c58b470448 Mon Sep 17 00:00:00 2001
From: hulxv
Date: Fri, 14 Mar 2025 07:39:15 +0200
Subject: [PATCH 01/11] [clang-doc] [feat] add `--repository-line-prefix`
argument (fix #59
https://github.com/choikwa updated
https://github.com/llvm/llvm-project/pull/129347
>From 1a6f7f27fb0017f45543ff8cc09c49bdfd4ebfd8 Mon Sep 17 00:00:00 2001
From: Kevin Choi
Date: Fri, 28 Feb 2025 16:52:03 -0600
Subject: [PATCH] [AMDGPU][clang] provide device implementation for
__builtin_logb a
https://github.com/jhuber6 edited
https://github.com/llvm/llvm-project/pull/129347
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/132868
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yxsamliu created
https://github.com/llvm/llvm-project/pull/132869
Currently HIP still uses offload bundler for non-rdc mode for the new offload
driver.
This patch switches to use offload wrapper for non-device-only non-rdc mode
when new offload driver is enabled.
This make
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Paul Kirth (ilovepi)
Changes
Many of the test files had an inconsistent formatting. This patch ran
clang-format over them using the project's .clang-format file, with
column limit = 0, to prevent test directives from being split
spall wrote:
> For example, for `abs`, it still depends on the _HLSL_16BIT_AVAILABILITY
> availability attribute. Does this PR intend to keep abs overloads using half
> "unexposed"? Or should that overload for abs be exposed too?
I'm not sure I understand the question, I think abs is the same
@@ -110,20 +110,76 @@ static std::string computeBaseSysRoot(const Driver &D,
bool IncludeTriple) {
return std::string(SysRootDir);
}
+std::string BareMetal::computeSysRoot() const {
+ if (!SysRoot.empty())
+return SysRoot;
+
+ std::string SysRoot = getDriver().SysRoot
@@ -293,7 +294,8 @@ void CIRGenFunction::startFunction(GlobalDecl gd, QualType
returnType,
mlir::Value addrVal =
emitAlloca(cast(paramVar)->getName(),
- convertType(paramVar->getType()), paramLoc, alignment);
+ convertType(param
https://github.com/tbaederr edited
https://github.com/llvm/llvm-project/pull/132775
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/farzonl approved this pull request.
https://github.com/llvm/llvm-project/pull/132804
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,10 @@
+! RUN: %flang -### -S -fslp-vectorize %s 2>&1 | FileCheck
-check-prefix=CHECK-SLP-VECTORIZE %s
+! RUN: %flang -### -S -fno-slp-vectorize %s 2>&1 | FileCheck
-check-prefix=CHECK-NO-SLP-VECTORIZE %s
+! RUN: %flang -### -S -O1 %s 2>&1 | FileCheck
-check-prefix=CH
https://github.com/shiltian created
https://github.com/llvm/llvm-project/pull/132814
None
>From c1f916c5cb19c1a816737dbb5cf21e5ac1636de3 Mon Sep 17 00:00:00 2001
From: Shilei Tian
Date: Mon, 24 Mar 2025 15:11:55 -0400
Subject: [PATCH] [AMDGPU] Remove outdated COV6 warning
---
clang/include/c
@@ -0,0 +1,81 @@
+//===--===//
+//
+// 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: Apac
https://github.com/bob80905 approved this pull request.
Talked offline, LGTM
https://github.com/llvm/llvm-project/pull/132804
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
bob80905 wrote:
> > > this is not NFC, so we should verify that we can call these intrinsics
> > > with `half` values even if 16-bit types aren't enabled, and that they
> > > properly codegen to 32-bit varia
> >
> >
> > > > > > > For example, for `abs`, it still depends on the
> > > > > > >
@@ -0,0 +1,472 @@
+#!/usr/bin/env python3
+
+"""generate_unsupported_in_drivermode.py
+
+This script generates Lit regression test files that validate that options are
only exposed to intended driver modes.
+
+The options and driver modes are parsed from Options.td, whose path sh
https://github.com/ziqingluo-90 updated
https://github.com/llvm/llvm-project/pull/131374
>From fdc265eeca36dc877f56389d8dba39b517e6 Mon Sep 17 00:00:00 2001
From: Ziqing Luo
Date: Fri, 14 Mar 2025 11:13:46 -0700
Subject: [PATCH 1/3] [NFC][Static Analyzer] Rename `NotNullConstraint` &
`NotN
@@ -1165,7 +1165,7 @@ ProgramStateRef
StdLibraryFunctionsChecker::NotNullConstraint::apply(
return State->assume(L, CannotBeNull);
}
-void StdLibraryFunctionsChecker::NotNullConstraint::describe(
+void StdLibraryFunctionsChecker::NullnessConstraint::describe(
Descripti
Author: Aaron Ballman
Date: 2025-03-19T07:49:16-04:00
New Revision: 449cdfacc07bd365d49a637196ce8f8067cf845c
URL:
https://github.com/llvm/llvm-project/commit/449cdfacc07bd365d49a637196ce8f8067cf845c
DIFF:
https://github.com/llvm/llvm-project/commit/449cdfacc07bd365d49a637196ce8f8067cf845c.diff
ilovepi wrote:
> @ilovepi I am not familiar with PR, so accidentally closed the previous PR.
> This is a new one. Sorry for that.
It's fine. IIRC you can reopen one pretty easily, but 🤷 Its no problem.
https://github.com/llvm/llvm-project/pull/132510
__
Author: Owen Pan
Date: 2025-03-24T19:18:13-07:00
New Revision: da7f1564a82e431dfbbf5b8d0bd53c64c5ca6af9
URL:
https://github.com/llvm/llvm-project/commit/da7f1564a82e431dfbbf5b8d0bd53c64c5ca6af9
DIFF:
https://github.com/llvm/llvm-project/commit/da7f1564a82e431dfbbf5b8d0bd53c64c5ca6af9.diff
LOG:
ilovepi wrote:
Ah, I see you deleted your repository, then yeah, that would make it hard to
reopen.
https://github.com/llvm/llvm-project/pull/132510
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinf
ilovepi wrote:
>I am not familiar with PR, so accidentally closed the previous PR
>https://github.com/llvm/llvm-project/pull/132360. This is a new one.
You can remove this part from the description. It isn't relevant to the rest of
the project, and doesn't belong in the commit log.
https://gi
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Yaxun (Sam) Liu (yxsamliu)
Changes
Currently HIP still uses offload bundler for non-rdc mode for the new offload
driver.
This patch switches to use offload wrapper for non-device-only non-rdc mode
when new offload driver is enable
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Yaxun (Sam) Liu (yxsamliu)
Changes
Currently HIP still uses offload bundler for non-rdc mode for the new offload
driver.
This patch switches to use offload wrapper for non-device-only non-rdc mode
when new offload driver is enabled.
Thi
llvmbot wrote:
@llvm/pr-subscribers-backend-amdgpu
@llvm/pr-subscribers-clang
Author: Jinsong Ji (jsji)
Changes
Fix error
llvm\clang\tools\amdgpu-arch\AMDGPUArchByHIP.cpp(102,29): error: result of
comparison of constant 18446744073709551615 with expression of type 'unsigned
int' is alwa
https://github.com/jhuber6 approved this pull request.
https://github.com/llvm/llvm-project/pull/132868
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4456,6 +4456,12 @@ void Parser::ParseDeclarationSpecifiers(
isInvalid = DS.setFunctionSpecNoreturn(Loc, PrevSpec, DiagID);
break;
+case tok::kw__Export:
+ // If we find kw__Export, it is being applied to a var or function
+ // This will be handled
hulxv wrote:
@ilovepi
About `writeFileDefiniation`, what's your opinion about making it similar to
the markdown generator like this?
```cpp
static void writeFileDefinition(const ClangDocContext &CDCtx, const Location
&L);
```
I think it's more better
https://github.com/llvm/llvm-project/pull/
https://github.com/tarunprabhu requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/132801
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4032,11 +4032,14 @@ def : Flag<["-"], "ftree-vectorize">, Alias;
def : Flag<["-"], "fno-tree-vectorize">, Alias;
}
+let Visibility = [ClangOption, FlangOption] in {
tarunprabhu wrote:
I think we should merge this with the visibility block for `fvectorize`
@@ -0,0 +1,10 @@
+! RUN: %flang -### -S -fslp-vectorize %s 2>&1 | FileCheck
-check-prefix=CHECK-SLP-VECTORIZE %s
+! RUN: %flang -### -S -fno-slp-vectorize %s 2>&1 | FileCheck
-check-prefix=CHECK-NO-SLP-VECTORIZE %s
+! RUN: %flang -### -S -O1 %s 2>&1 | FileCheck
-check-prefix=CH
@@ -161,6 +161,14 @@ void Flang::addCodegenOptions(const ArgList &Args,
options::OPT_fno_vectorize, enableVec))
CmdArgs.push_back("-vectorize-loops");
+ // -fslp-vectorize is enabled based on the optimization level selected.
+ bool EnableSLPVec = shoul
llvm-beanz wrote:
> Finish the work of #81782
Maybe instead:
> Apply pattern of #81782 to intrinsics added in #95999.
Also probably worth having a description of the problem this solves in the
description so that it ends up in the final commit message.
https://github.com/llvm/llvm-project/pu
spall wrote:
> Are there any test changes we should consider here? If not could you prefix
> `[NFC]` to the title?
It looked to me like the tests were all correct based on what was done in the
reference PR
https://github.com/llvm/llvm-project/pull/132804
__
1 - 100 of 486 matches
Mail list logo
