https://github.com/iagarwa updated
https://github.com/llvm/llvm-project/pull/88736
>From 821c61fb4905b491176e00ea9ed322aad04c98e3 Mon Sep 17 00:00:00 2001
From: Isha Agarwal
Date: Mon, 15 Apr 2024 06:22:34 -0700
Subject: [PATCH 1/2] [X86][test] Added extra cet tests
Updated cet test to:
-Check
@@ -1964,11 +1964,25 @@ Instruction *InstCombinerImpl::visitFPToSI(FPToSIInst
&FI) {
}
Instruction *InstCombinerImpl::visitUIToFP(CastInst &CI) {
- return commonCastTransforms(CI);
+ if (Instruction *R = commonCastTransforms(CI))
+return R;
+ if (!CI.hasNonNeg() && isK
https://github.com/cor3ntin commented:
Thanks for spotting that. I think there are further cases that I'm not sure we
protect against
https://github.com/llvm/llvm-project/pull/88974
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/88974
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,4 @@
+// RUN: %clang_cc1 -std=c++23 -fsyntax-only -verify %s
+
+auto x0 = requires (this int) { true; }; // expected-error {{a requires clause
cannot have an explicit object parameter}}
+auto x1 = requires (int, this int) { true; }; // expected-error {{a requires
clau
cor3ntin wrote:
@AMP999 Feel free to merge!
https://github.com/llvm/llvm-project/pull/87132
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vgvassilev created
https://github.com/llvm/llvm-project/pull/89031
Clang's CodeGen is designed to work with a single llvm::Module. In many cases
for convenience various CodeGen parts have a reference to the llvm::Module
(TheModule or Module) which does not change when a new
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Vassil Vassilev (vgvassilev)
Changes
Clang's CodeGen is designed to work with a single llvm::Module. In many cases
for convenience various CodeGen parts have a reference to the llvm::Module
(TheModule or Module) which does not change when
@@ -73,7 +74,15 @@ llvm::Error
IncrementalExecutor::addModule(PartialTranslationUnit &PTU) {
Jit->getMainJITDylib().createResourceTracker();
ResourceTrackers[&PTU] = RT;
- return Jit->addIRModule(RT, {std::move(PTU.TheModule), TSCtx});
vgvassilev wro
Stefan =?utf-8?q?Gr=C3=A4nitz?=
Message-ID:
In-Reply-To:
@@ -14,7 +14,7 @@ struct A { int a; A(int a) : a(a) {} virtual ~A(); };
// PartialTranslationUnit.
inline A::~A() { printf("~A(%d)\n", a); }
-// Create one instance with new and delete it.
+// Create one instance with
https://github.com/vgvassilev updated
https://github.com/llvm/llvm-project/pull/89031
>From d396330fd004db50b3808e491cb45847888bb651 Mon Sep 17 00:00:00 2001
From: Vassil Vassilev
Date: Wed, 17 Apr 2024 07:12:40 +
Subject: [PATCH] [clang-repl] Clone the llvm::Modules to avoid invalid memory
https://github.com/vgvassilev edited
https://github.com/llvm/llvm-project/pull/89031
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -275,13 +278,15 @@ void DeclPrinter::prettyPrintAttributes(const Decl *D,
if (Pos != AttrPosAsWritten::Left)
Out << ' ';
A->printPretty(Out, Policy);
+ hasPrinted = true;
if (Pos == AttrPosAsWritten::Left)
Out
https://github.com/vgvassilev updated
https://github.com/llvm/llvm-project/pull/88600
>From 9b2bb9068cbefcfffd0931fbbee46b1a0f536a4f Mon Sep 17 00:00:00 2001
From: Vassil Vassilev
Date: Sat, 13 Apr 2024 06:39:34 +
Subject: [PATCH 1/2] Fix the double space and double attribute printing of th
https://github.com/owenca created
https://github.com/llvm/llvm-project/pull/89033
Fixes #78789.
>From 0e2d91d20a1c318868f2e791d859a047b83f0277 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Wed, 17 Apr 2024 00:39:36 -0700
Subject: [PATCH] [clang-format] Annotate ampamp after new/delete as
Bina
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Owen Pan (owenca)
Changes
Fixes #78789.
---
Full diff: https://github.com/llvm/llvm-project/pull/89033.diff
2 Files Affected:
- (modified) clang/lib/Format/TokenAnnotator.cpp (+2)
- (modified) clang/unittests/Format/TokenAnnotat
https://github.com/Meinersbur ready_for_review
https://github.com/llvm/llvm-project/pull/88932
___
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: Michael Kruse (Meinersbur)
Changes
The help text was not updated in #87360.
Clang is also mentioned for the diagnostic warnings reference, which mostly
applies to C/C++/Obj-C, not Fortran. #81726 already tried to fix this,
and I don't kn
llvmbot wrote:
@llvm/pr-subscribers-flang-driver
Author: Michael Kruse (Meinersbur)
Changes
The help text was not updated in #87360.
Clang is also mentioned for the diagnostic warnings reference, which mostly
applies to C/C++/Obj-C, not Fortran. #81726 already tried to fix this,
and I d
https://github.com/banach-space approved this pull request.
Nice, thank you! LGTM
https://github.com/llvm/llvm-project/pull/88932
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/martinboehme updated
https://github.com/llvm/llvm-project/pull/88865
>From 69f444532a9dd1da4c8018684fbf24edacfc91fd Mon Sep 17 00:00:00 2001
From: Martin Braenne
Date: Tue, 16 Apr 2024 09:52:35 +
Subject: [PATCH] [clang][dataflow] Refactor `PropagateResultObject()` with a
martinboehme wrote:
Ready for review.
https://github.com/llvm/llvm-project/pull/88865
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -807,8 +807,12 @@ def gcc_install_dir_EQ : Joined<["--"],
"gcc-install-dir=">,
"Note: executables (e.g. ld) used by the compiler are not overridden by the
selected GCC installation">;
def gcc_toolchain : Joined<["--"], "gcc-toolchain=">, Flags<[NoXarchOption]>,
Visibil
DavidSpickett wrote:
> Clang is also mentioned for the diagnostic warnings reference, which mostly
> applies to C/C++/Obj-C, not Fortran.
> https://github.com/llvm/llvm-project/pull/81726 already tried to fix this,
> and I don't know a better solution.
Do you mean that this PR fixes this, or
MikeWeller wrote:
FYI I don't have merge privs, not sure if I need to ping anybody. Then again I
still see "This workflow requires approval from a maintainer", possibly because
this is my first contribution?
https://github.com/llvm/llvm-project/pull/88138
__
@@ -286,6 +286,10 @@ Changes in existing checks
check by resolving fix-it overlaps in template code by disregarding implicit
instances.
+- Improved :doc:`bugprone-forwarding-reference-overload
+ `
+ check to not flag deleted constructors which are unable to hide anything
@@ -67,57 +67,42 @@ enum CPUFeatures {
FEAT_FP,
FEAT_SIMD,
FEAT_CRC,
- FEAT_SHA1,
FEAT_SHA2,
FEAT_SHA3,
FEAT_AES,
- FEAT_PMULL,
FEAT_FP16,
- FEAT_DIT,
FEAT_DPB,
FEAT_DPB2,
FEAT_JSCVT,
FEAT_FCMA,
FEAT_RCPC,
FEAT_RCPC2,
FEAT_FRINTTS,
- F
@@ -1099,6 +1099,25 @@ static bool upgradeIntrinsicFunction1(Function *F,
Function *&NewFn,
return true;
}
+ ID = StringSwitch(Name)
+ .StartsWith("splice.", Intrinsic::vector_splice)
+ .StartsWith("reverse.", Intrinsic::vector_r
@@ -18779,13 +18779,13 @@ This is an overloaded intrinsic.
::
- declare <2 x i8> @llvm.experimental.vector.reverse.v2i8(<2 x i8> %a)
- declare
@llvm.experimental.vector.reverse.nxv4i32( %a)
+ declare <2 x i8> @llvm.vector.reverse.v2i8(<2 x i8> %a)
+ decl
@@ -18914,13 +18914,13 @@ This is an overloaded intrinsic.
::
- declare <2 x double> @llvm.experimental.vector.splice.v2f64(<2 x double>
%vec1, <2 x double> %vec2, i32 %imm)
- declare
@llvm.experimental.vector.splice.nxv4i32( %vec1, %vec2, i32 %imm)
+ decla
mgabka wrote:
> > vector.revert
>
> I think should be vector.reverse?
Will fix the commit message when merging the PR.
https://github.com/llvm/llvm-project/pull/88748
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-
@@ -1,13 +1,22 @@
// RUN: %clang_cc1 -E -triple i386 -dM -o - -fcf-protection=return %s |
FileCheck %s --check-prefix=RETURN
// RUN: %clang_cc1 -E -triple i386 -dM -o - -fcf-protection=branch %s |
FileCheck %s --check-prefix=BRANCH
// RUN: %clang_cc1 -E -triple i386 -dM -o -
@@ -1,13 +1,22 @@
// RUN: %clang_cc1 -E -triple i386 -dM -o - -fcf-protection=return %s |
FileCheck %s --check-prefix=RETURN
// RUN: %clang_cc1 -E -triple i386 -dM -o - -fcf-protection=branch %s |
FileCheck %s --check-prefix=BRANCH
// RUN: %clang_cc1 -E -triple i386 -dM -o -
ilya-biryukov wrote:
> Have you tried changing the behavior of the existing `-fmodule-map-file=` to
> load the file after we load module files? If so, what kinds of things do you
> see breaking or changing? If we can avoid it, I think it would be better to
> only have a single mode here, and l
https://github.com/mahtohappy created
https://github.com/llvm/llvm-project/pull/89036
When in-place new-ing a local variable of an array of trivial type, the
generated code calls 'memset' with the correct size of the array, earlier it
was generating size (squared of the typedef array + size).
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (mahtohappy)
Changes
When in-place new-ing a local variable of an array of trivial type, the
generated code calls 'memset' with the correct size of the array, earlier it
was generating size (squared of the typedef array + size).
The
Keenuts wrote:
Thanks all!
Agree with Bogner, let's unblock the tests first.
As for the address space for globals, this isn't something we have looked into
yet, so I'd be in favor of keeping the same behavior as the SPIRN flavor until
we have a reason to diverge (as in "thought about this issue
banach-space wrote:
> > Clang is also mentioned for the diagnostic warnings reference, which mostly
> > applies to C/C++/Obj-C, not Fortran. #81726 already tried to fix this, and
> > I don't know a better solution.
>
> Do you mean that this PR fixes this, or that you noticed this problem while
ilya-biryukov wrote:
There's quite a few failures:
https://gist.github.com/ilya-biryukov/380d84dfe53f839f449231eb9a2dd46c
The logic that sets up module structure from module maps before starting to
load PCMs definitely needs to be reworked (and I'm not sure if it can be in the
first place). I
c-rhodes wrote:
Thanks for patch. We use `interleave2` in MLIR so it'll need updating there as
well.
I don't know what the policy is for promoting intrinsics from experimental to
first-class or if it's documented anywhere (?), but I would expect this to be
accompanied with an RFC / announceme
Meinersbur wrote:
> Do you mean that this PR fixes this, or that you noticed this problem while
> working on this?
I noticed this when working on the patch, .i.e that
https://flang.llvm.org/docs/FlangCommandLineReference.html still mentioned
Clang in several places and since it was common, I
arsenm wrote:
> I don't know what the policy is for promoting intrinsics from experimental to
> first-class or if it's documented anywhere (?), but I would expect this to be
> accompanied with an RFC / announcement on Discourse.
I don't remember any intrinsic ever making the move out of experi
nikic wrote:
> > I don't know what the policy is for promoting intrinsics from experimental
> > to first-class or if it's documented anywhere (?), but I would expect this
> > to be accompanied with an RFC / announcement on Discourse.
>
> I don't remember any intrinsic ever making the move out
@@ -807,8 +807,12 @@ def gcc_install_dir_EQ : Joined<["--"],
"gcc-install-dir=">,
"Note: executables (e.g. ld) used by the compiler are not overridden by the
selected GCC installation">;
def gcc_toolchain : Joined<["--"], "gcc-toolchain=">, Flags<[NoXarchOption]>,
Visibil
https://github.com/mgabka edited https://github.com/llvm/llvm-project/pull/88748
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mgabka wrote:
I was actually following this patch https://reviews.llvm.org/D127976 which
promoted vector.insert/extract and there was no RFC for it.
My understanding is that these intrinsics are so heavily used in LLVM now so
there is no need to call them experimental, moreover the auto upgrade
c-rhodes wrote:
> I was actually following this patch https://reviews.llvm.org/D127976 which
> promoted vector.insert/extract and there was no RFC for it. My understanding
> is that these intrinsics are so heavily used in LLVM now so there is no need
> to call them experimental, moreover the a
https://github.com/yronglin updated
https://github.com/llvm/llvm-project/pull/88666
>From 8d48a0bd1cf15b9cf00bc294912b674b5f94a11c Mon Sep 17 00:00:00 2001
From: yronglin
Date: Mon, 15 Apr 2024 00:36:06 +0800
Subject: [PATCH 1/4] [Clang] Allow the value of unroll count to be zero in
'#pragma
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/89041
No reason for this to not be one. This gets rid of a few const_casts.
>From 2edf794268aa825391a37053bb5908e362ad62a6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Wed, 17 Apr 2024 11:02:20
https://github.com/yronglin updated
https://github.com/llvm/llvm-project/pull/88666
>From 8d48a0bd1cf15b9cf00bc294912b674b5f94a11c Mon Sep 17 00:00:00 2001
From: yronglin
Date: Mon, 15 Apr 2024 00:36:06 +0800
Subject: [PATCH 1/4] [Clang] Allow the value of unroll count to be zero in
'#pragma
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-codegen
Author: Timm Baeder (tbaederr)
Changes
No reason for this to not be one. This gets rid of a few const_casts.
---
Full diff: https://github.com/llvm/llvm-project/pull/89041.diff
1 Files Affected:
- (modified) c
DavidSpickett wrote:
> The docbrief strings already use %Program/GlobalDocumentation.Program which
> seems to work and made me think that HelpTextForVariants was actually
> unnecessary. Would you like me to introduce DocBriefForVariants?
IIRC it was unnessecary for the documentation because it
https://github.com/DavidSpickett approved this pull request.
The changes themselves LGTM, thanks for fixing this.
https://github.com/llvm/llvm-project/pull/88932
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
@@ -3903,7 +3904,14 @@ bool Sema::CheckLoopHintExpr(Expr *E, SourceLocation
Loc) {
if (R.isInvalid())
return true;
- bool ValueIsPositive = ValueAPS.isStrictlyPositive();
+ // GCC allows the value of unroll count to be 0.
+ // https://gcc.gnu.org/onlinedocs/gcc/Loop-
Nathan =?utf-8?q?Gauër?=
Message-ID:
In-Reply-To:
@@ -1109,6 +1124,10 @@ void CodeGenFunction::EmitForStmt(const ForStmt &S,
llvm::BasicBlock *CondBlock = CondDest.getBlock();
EmitBlock(CondBlock);
+ if (getTarget().getTriple().isSPIRVLogical())
arsenm
Nathan =?utf-8?q?Gauër?=
Message-ID:
In-Reply-To:
@@ -4824,6 +4824,9 @@ llvm::CallInst
*CodeGenFunction::EmitRuntimeCall(llvm::FunctionCallee callee,
llvm::CallInst *call = Builder.CreateCall(
callee, args, getBundlesForFunclet(callee.getCallee()), name);
call->se
@@ -154,11 +154,20 @@ llvm::Value
*CodeGen::emitRoundPointerUpToAlignment(CodeGenFunction &CGF,
llvm::Value *Ptr,
CharUnits Align) {
// OverflowArgArea = (OverflowArgArea
@@ -154,11 +154,20 @@ llvm::Value
*CodeGen::emitRoundPointerUpToAlignment(CodeGenFunction &CGF,
llvm::Value *Ptr,
CharUnits Align) {
// OverflowArgArea = (OverflowArgArea
https://github.com/frasercrmck updated
https://github.com/llvm/llvm-project/pull/88922
>From bff3b97b599e1c9529a95ead9d61f9ecc9cad129 Mon Sep 17 00:00:00 2001
From: Fraser Cormack
Date: Tue, 16 Apr 2024 15:30:20 +0100
Subject: [PATCH 1/5] [libclc] Convert llvm-spirv to imported executable
This
https://github.com/mahtohappy updated
https://github.com/llvm/llvm-project/pull/89036
>From 56c2b4f70735a6ef3b80cb8461a88ea51f2d02d7 Mon Sep 17 00:00:00 2001
From: mahtohappy
Date: Tue, 16 Apr 2024 17:48:45 +0530
Subject: [PATCH 1/4] [Clang][Sema] placement new initializes typedef array
with c
HerrCai0907 wrote:
I can help you to merge it. 😄
https://github.com/llvm/llvm-project/pull/88138
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mahtohappy updated
https://github.com/llvm/llvm-project/pull/89036
>From 56c2b4f70735a6ef3b80cb8461a88ea51f2d02d7 Mon Sep 17 00:00:00 2001
From: mahtohappy
Date: Tue, 16 Apr 2024 17:48:45 +0530
Subject: [PATCH 1/4] [Clang][Sema] placement new initializes typedef array
with c
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 a71565d75e23fc28076aa5bf1c5cf4432623afc5
bab66e16f29367fa44002d9d31c328607c734354 --
@@ -115,7 +115,13 @@ void AMDGPUABIInfo::computeInfo(CGFunctionInfo &FI) const {
Address AMDGPUABIInfo::EmitVAArg(CodeGenFunction &CGF, Address VAListAddr,
QualType Ty) const {
- llvm_unreachable("AMDGPU does not support varargs");
+ const bo
@@ -0,0 +1,43 @@
+//===- ExpandVariadics.h - expand variadic functions *- 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
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
HerrCai0907 wrote:
please fix format issue.`git-clang-format` is helpful.
https://github.com/llvm/llvm-project/pull/88138
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: NagyDonat
Date: 2024-04-17T12:52:23+02:00
New Revision: 792d437b56adfb3416daf8105942d4899fb82763
URL:
https://github.com/llvm/llvm-project/commit/792d437b56adfb3416daf8105942d4899fb82763
DIFF:
https://github.com/llvm/llvm-project/commit/792d437b56adfb3416daf8105942d4899fb82763.diff
LOG
https://github.com/NagyDonat closed
https://github.com/llvm/llvm-project/pull/88912
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -115,7 +115,13 @@ void AMDGPUABIInfo::computeInfo(CGFunctionInfo &FI) const {
Address AMDGPUABIInfo::EmitVAArg(CodeGenFunction &CGF, Address VAListAddr,
QualType Ty) const {
- llvm_unreachable("AMDGPU does not support varargs");
+ const bo
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/89048
We always capitalize bitfield as "BitField".
>From 071305b588337841fbd2dbb929487c105a289d4b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Wed, 17 Apr 2024 12:53:49 +0200
Subject: [PATCH] [c
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
We always capitalize bitfield as "BitField".
---
Patch is 31.15 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/89048.diff
25 Files Affected:
- (modified) clang
@@ -115,7 +115,13 @@ void AMDGPUABIInfo::computeInfo(CGFunctionInfo &FI) const {
Address AMDGPUABIInfo::EmitVAArg(CodeGenFunction &CGF, Address VAListAddr,
QualType Ty) const {
- llvm_unreachable("AMDGPU does not support varargs");
+ const bo
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -115,7 +115,13 @@ void AMDGPUABIInfo::computeInfo(CGFunctionInfo &FI) const {
Address AMDGPUABIInfo::EmitVAArg(CodeGenFunction &CGF, Address VAListAddr,
QualType Ty) const {
- llvm_unreachable("AMDGPU does not support varargs");
+ const bo
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
JonChesterfield wrote:
Drive by fixes to some of Matt's comments. Caught a missing line in a .def file
for NVPTX through luck due to the enum rename which means the
Other/new-pm-thinlto-postlink-samplepgo-defaults.ll style tests need to be
patched again - leaving that for now as I want to chec
@@ -0,0 +1,1056 @@
+//===-- ExpandVariadicsPass.cpp *- 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: Apach
https://github.com/Sirraide approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/89048
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1,3 +1,6 @@
-__kernel void foo(int *i) {
+// RUN: %clang -emit-llvm -S -o - %s | FileCheck %s
frasercrmck wrote:
Yes, I was hoping to explore in that direction. I've kind of just copied the
skeleton of the LIT tests we have downstream, but as you say, this i
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/89036
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
NagyDonat wrote:
Uh oh, those niebloids are really deep black magic :skull_and_crossbones: I
didn't know about them previously, so thanks for mentioning them.
https://github.com/llvm/llvm-project/pull/88913
___
cfe-commits mailing list
cfe-commits@lis
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/89048
>From 5e4b24933180f012573785fa5747bf37efd26fad Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Wed, 17 Apr 2024 12:53:49 +0200
Subject: [PATCH] [clang][NFC] Fix FieldDecl::isUnnamedBitfield()
Author: NagyDonat
Date: 2024-04-17T13:26:51+02:00
New Revision: 06eedffe0d2782922e63cc25cb927f4acdaf7b30
URL:
https://github.com/llvm/llvm-project/commit/06eedffe0d2782922e63cc25cb927f4acdaf7b30
DIFF:
https://github.com/llvm/llvm-project/commit/06eedffe0d2782922e63cc25cb927f4acdaf7b30.diff
LOG
https://github.com/NagyDonat closed
https://github.com/llvm/llvm-project/pull/88913
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/JinjinLi868 created
https://github.com/llvm/llvm-project/pull/89051
Data type conversion between fp16 and bf16 will generate fptrunc and fpextend
nodes, but they are actually bitcast nodes.
>From 02c11a9db49dd34839feb8329cfb8dbe4bc45763 Mon Sep 17 00:00:00 2001
From: Jinjin
https://github.com/martinboehme created
https://github.com/llvm/llvm-project/pull/89052
This class no longer serves any purpose; see also the discussion here:
https://reviews.llvm.org/D155204#inline-1503204
A lot of existing tests in TransferTest.cpp check for the existence of
`RecordValue`s. S
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
1 - 100 of 435 matches
Mail list logo
