llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Hirofumi Nakamura (hnakamura5)
Changes
Adds the support for tokens that have forms like unary operators.
- bang operators: `!name`
- cond operator: `!cond`
- numeric literals: `+1`, `-1`
cond operator are one of bang operators but
https://github.com/hnakamura5 edited
https://github.com/llvm/llvm-project/pull/78996
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hnakamura5 created
https://github.com/llvm/llvm-project/pull/78996
Adds the support for tokens that have forms like unary operators.
- bang operators: `!name`
- cond operator: `!cond`
- numeric literals: `+1`, `-1`
cond operator are one of bang operators but is distinguished
AaronBallman wrote:
> So my takeaways here are:
>
> 1. Tests that ensure we don't crash anymore are important for us.
>
> 2. `-verify` is an acceptable way to write such tests.
>
>
> Is this correct?
Yup, that's it exactly!
https://github.com/llvm/llvm-project/pull/78898
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
shiltian wrote:
This breaks the build on macOS because the linker doesn't support `--long-plt`.
Please fix it or revert the patch.
https://github.com/llvm/llvm-project/pull/78959
___
cfe-commits
@@ -351,6 +354,16 @@ struct BufferizationOptions {
/// used.
UnknownTypeConverterFn unknownTypeConverterFn = nullptr;
+ // Use during type conversion to determine the memory space for memref based
+ // on the originanl tensor type
+ GetMemorySpaceFn getMemorySpaceFn =
Endilll wrote:
So my takeaways here are:
1. Tests that ensure we don't crash anymore are important for us.
2. `-verify` is an acceptable way to write such tests.
Is this correct?
https://github.com/llvm/llvm-project/pull/78898
___
cfe-commits mailing
https://github.com/zahiraam updated
https://github.com/llvm/llvm-project/pull/78330
>From 8f8917528e30d2ba67f669cfd1a893bc85c21121 Mon Sep 17 00:00:00 2001
From: Ammarguellat
Date: Tue, 16 Jan 2024 11:24:03 -0800
Subject: [PATCH 1/6] Fixed a bug in Smith's algorithm and made sure last
option
https://github.com/labrinea updated
https://github.com/llvm/llvm-project/pull/78788
>From 425bf555fcc2c2ca2fca72822618702ac9278be6 Mon Sep 17 00:00:00 2001
From: Alexandros Lamprineas
Date: Fri, 19 Jan 2024 21:07:46 +
Subject: [PATCH] [AArch64][FMV] Support feature MOPS in Function Multi
https://github.com/AaronBallman commented:
Yeah, these changes really need to come with test coverage. We allow landing
changes with no test coverage when it's an NFC change or when the testing would
require heroic efforts. Otherwise, we expect tests that are as good as you can
reasonably
@@ -14030,6 +14034,9 @@ void
Sema::DefineImplicitDefaultConstructor(SourceLocation CurrentLocation,
CXXRecordDecl *ClassDecl = Constructor->getParent();
assert(ClassDecl && "DefineImplicitDefaultConstructor - invalid
constructor");
+ if (ClassDecl->isInvalidDecl()) {
+
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/78898
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -5990,6 +5990,10 @@ void Sema::ActOnDefaultCtorInitializers(Decl *CDtorDecl)
{
if (CXXConstructorDecl *Constructor
= dyn_cast(CDtorDecl)) {
+if (CXXRecordDecl *ClassDecl = Constructor->getParent();
+!ClassDecl || ClassDecl->isInvalidDecl()) {
+
https://github.com/daniel-grumberg closed
https://github.com/llvm/llvm-project/pull/78584
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Daniel Grumberg
Date: 2024-01-22T15:41:29Z
New Revision: c5532124dc0ffc892ded0b093e92244977e748f8
URL:
https://github.com/llvm/llvm-project/commit/c5532124dc0ffc892ded0b093e92244977e748f8
DIFF:
https://github.com/llvm/llvm-project/commit/c5532124dc0ffc892ded0b093e92244977e748f8.diff
https://github.com/pogo59 closed https://github.com/llvm/llvm-project/pull/78613
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Paul T Robinson
Date: 2024-01-22T07:38:33-08:00
New Revision: d9cb37c9045bb0b3692d6faaeac43150a26e42e4
URL:
https://github.com/llvm/llvm-project/commit/d9cb37c9045bb0b3692d6faaeac43150a26e42e4
DIFF:
https://github.com/daniel-grumberg closed
https://github.com/llvm/llvm-project/pull/77451
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Daniel Grumberg
Date: 2024-01-22T15:32:57Z
New Revision: 69fedaf830f8a2df4751a3c20189b7299daf88ae
URL:
https://github.com/llvm/llvm-project/commit/69fedaf830f8a2df4751a3c20189b7299daf88ae
DIFF:
https://github.com/llvm/llvm-project/commit/69fedaf830f8a2df4751a3c20189b7299daf88ae.diff
https://github.com/daniel-grumberg updated
https://github.com/llvm/llvm-project/pull/77451
>From 8ff189e707a909f5228bce2042812a45a98d1e6c Mon Sep 17 00:00:00 2001
From: Daniel Grumberg
Date: Tue, 9 Jan 2024 12:06:14 +
Subject: [PATCH] [clang][ExtractAPI] Add support C unions in non 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 fa6025e25b5754e8cf39169e3a7085b57ea35de5
b3f8e9d1ce4f9a88dc6b57f2862e928a465aecf8 --
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Lucas Duarte Prates (pratlucas)
Changes
This introduces the Armv9.5-A architecture version to the Arm backend,
following on from the existing implementation for AArch64 targets.
Mode details about the Armv9.5-A architecture version can be
llvmbot wrote:
@llvm/pr-subscribers-backend-arm
Author: Lucas Duarte Prates (pratlucas)
Changes
This introduces the Armv9.5-A architecture version to the Arm backend,
following on from the existing implementation for AArch64 targets.
Mode details about the Armv9.5-A architecture version
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Lucas Duarte Prates (pratlucas)
Changes
This introduces the Armv9.5-A architecture version to the Arm backend,
following on from the existing implementation for AArch64 targets.
Mode details about the Armv9.5-A architecture version
https://github.com/pratlucas created
https://github.com/llvm/llvm-project/pull/78994
This introduces the Armv9.5-A architecture version to the Arm backend,
following on from the existing implementation for AArch64 targets.
Mode details about the Armv9.5-A architecture version can be found at:
manbearian wrote:
@nicolasvasilache do you mind taking a look please when you have a chance?
https://github.com/llvm/llvm-project/pull/78484
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -504,6 +504,15 @@ static VPValue *createScalarIVSteps(VPlan , const
InductionDescriptor ,
HeaderVPBB->insert(BaseIV->getDefiningRecipe(), IP);
}
+ VPTypeAnalysis TypeInfo(SE.getContext());
+ if (TypeInfo.inferScalarType(BaseIV) != TypeInfo.inferScalarType(Step)) {
@@ -1469,6 +1464,47 @@ void VPReplicateRecipe::print(raw_ostream , const
Twine ,
}
#endif
+Value *VPScalarCastRecipe ::generate(VPTransformState , unsigned Part) {
+ assert(vputils::onlyFirstLaneUsed(this) &&
+ "Codegen only implemented for first lane.");
+ switch
@@ -1469,6 +1465,47 @@ void VPReplicateRecipe::print(raw_ostream , const
Twine ,
}
#endif
+Value *VPScalarCastRecipe ::generate(VPTransformState , unsigned Part) {
+ assert(vputils::onlyFirstLaneUsed(this) &&
+ "Codegen only implemented for first lane only.");
+
@@ -2297,6 +2297,7 @@ emitTransformedIndex(IRBuilderBase , Value *Index,
Value *StartValue,
? B.CreateSExtOrTrunc(Index, StepTy)
: B.CreateCast(Instruction::SIToFP, Index, StepTy);
if (CastedIndex != Index) {
+
https://github.com/PiotrZSL closed
https://github.com/llvm/llvm-project/pull/72037
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Piotr Zegar
Date: 2024-01-22T16:17:33+01:00
New Revision: 6a80e56ad0c7ae0adb8c4fb3f88eab7643566f41
URL:
https://github.com/llvm/llvm-project/commit/6a80e56ad0c7ae0adb8c4fb3f88eab7643566f41
DIFF:
https://github.com/llvm/llvm-project/commit/6a80e56ad0c7ae0adb8c4fb3f88eab7643566f41.diff
https://github.com/AaronBallman approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/78330
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -31,3 +31,27 @@
// RUN: %clang -fveclib=Accelerate %s -nodefaultlibs -target
arm64-apple-ios8.0.0 -### 2>&1 | FileCheck
--check-prefix=CHECK-LINK-NODEFAULTLIBS %s
// CHECK-LINK-NODEFAULTLIBS-NOT: "-framework" "Accelerate"
+
+
+/* Verify that the correct vector library is
https://github.com/MDevereau closed
https://github.com/llvm/llvm-project/pull/77947
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Matthew Devereau
Date: 2024-01-22T15:11:49Z
New Revision: 6ba62f4f251763745d39194ecd33ebaf7a739059
URL:
https://github.com/llvm/llvm-project/commit/6ba62f4f251763745d39194ecd33ebaf7a739059
DIFF:
https://github.com/llvm/llvm-project/commit/6ba62f4f251763745d39194ecd33ebaf7a739059.diff
Author: erichkeane
Date: 2024-01-22T07:09:22-08:00
New Revision: 0d8e333a7e756fc6ab07305ba1bbce916f510b30
URL:
https://github.com/llvm/llvm-project/commit/0d8e333a7e756fc6ab07305ba1bbce916f510b30
DIFF:
https://github.com/llvm/llvm-project/commit/0d8e333a7e756fc6ab07305ba1bbce916f510b30.diff
https://github.com/erichkeane approved this pull request.
Makes sense to me.
https://github.com/llvm/llvm-project/pull/78991
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
I did review everything else, we seem to be otherwise in sync
https://github.com/llvm/llvm-project/pull/78991
___
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: cor3ntin (cor3ntin)
Changes
* Set `__cpp_auto_cast`, as per https://github.com/cplusplus/CWG/issues/281
* Support `__has_extension(cxx_generalized_nttp)` in C++20 as the feature isn't
stable enough for a feature test macro
* Support
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/78991
* Set `__cpp_auto_cast`, as per https://github.com/cplusplus/CWG/issues/281
* Support `__has_extension(cxx_generalized_nttp)` in C++20 as the feature isn't
stable enough for a feature test macro
* Support
https://github.com/sam-mccall closed
https://github.com/llvm/llvm-project/pull/78714
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Sam McCall
Date: 2024-01-22T16:03:37+01:00
New Revision: 3de5d8e1254977c8812412a159da4185c9853fd0
URL:
https://github.com/llvm/llvm-project/commit/3de5d8e1254977c8812412a159da4185c9853fd0
DIFF:
https://github.com/llvm/llvm-project/commit/3de5d8e1254977c8812412a159da4185c9853fd0.diff
https://github.com/alexey-bataev updated
https://github.com/llvm/llvm-project/pull/77529
>From 7440ee8ba235fd871af0999f66d5d6130456400b Mon Sep 17 00:00:00 2001
From: Alexey Bataev
Date: Tue, 9 Jan 2024 21:43:31 +
Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=
https://github.com/jhuber6 edited
https://github.com/llvm/llvm-project/pull/78333
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -6291,9 +6291,19 @@ ExprResult Sema::BuildCXXDefaultArgExpr(SourceLocation
CallLoc,
ImmediateCallVisitor V(getASTContext());
if (!NestedDefaultChecking)
V.TraverseDecl(Param);
-if (V.HasImmediateCalls) {
-
https://github.com/jhuber6 commented:
You should add a test that checks the output of `-ccc-print-phases` and
`-ccc-print-bindings`.
https://github.com/llvm/llvm-project/pull/78333
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy
Message-ID:
In-Reply-To:
https://github.com/NagyDonat updated
https://github.com/llvm/llvm-project/pull/78315
>From c75c05c6e894a46797913c5bdccb240cbcc01ae9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Don=C3=A1t=20Nagy?=
Date: Tue, 12 Dec 2023
https://github.com/yxsamliu approved this pull request.
https://github.com/llvm/llvm-project/pull/78333
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -238,3 +239,236 @@ void init_capture_init_list() {
// CHECK: call {{.*}}dtor
// CHECK: }
}
+
+namespace P2718R0 {
yronglin wrote:
agree, I have compiled this case, Currently, clang will not extending lifetime
of temporaries in this case.
@@ -2312,12 +2312,31 @@ Parser::DeclGroupPtrTy
Parser::ParseDeclGroup(ParsingDeclSpec ,
bool IsForRangeLoop = false;
if (TryConsumeToken(tok::colon, FRI->ColonLoc)) {
IsForRangeLoop = true;
+ EnterExpressionEvaluationContext ForRangeInitContext(
+
@@ -1357,6 +1363,19 @@ class Sema final {
// VLAs).
bool InConditionallyConstantEvaluateContext = false;
+/// Whether we are currently in a context in which temporaries must be
+/// lifetime-extended (Eg. in a for-range initializer).
+bool
@@ -9959,6 +9976,18 @@ class Sema final {
return currentEvaluationContext().isImmediateFunctionContext();
}
+ bool isInLifetimeExtendingContext() const {
+assert(!ExprEvalContexts.empty() &&
+ "Must be in an expression evaluation context");
+return
https://github.com/yxsamliu approved this pull request.
https://github.com/llvm/llvm-project/pull/78915
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sudonatalie closed
https://github.com/llvm/llvm-project/pull/78611
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Natalie Chouinard
Date: 2024-01-22T09:54:54-05:00
New Revision: ce519b59b77551d1e9698de17367385a3dbf1352
URL:
https://github.com/llvm/llvm-project/commit/ce519b59b77551d1e9698de17367385a3dbf1352
DIFF:
Fznamznon wrote:
> Because we
>
yxsamliu wrote:
LGTM for amdgpu
https://github.com/llvm/llvm-project/pull/78759
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mjklemm wrote:
> Maybe we should merge a deprecation warning on -lFortran_main before the LLVM
> release, then make these changes only after the release has branched?
That sounds like a good idea for now. Then, we can buy a bit of time for the
RFC and to make decision how we want to deal
https://github.com/balazske approved this pull request.
OK, change can be merged before the simplification.
https://github.com/llvm/llvm-project/pull/78693
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/erichkeane commented:
I don't have a good idea here... I don't really know what it means to
'relocate' a volatile.
https://github.com/llvm/llvm-project/pull/77092
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/kparzysz closed
https://github.com/llvm/llvm-project/pull/77761
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
thesamesam wrote:
Note that on Gentoo, this isn't right either, we have e.g.:
```
/usr/lib/gcc/x86_64-pc-linux-gnu/14/include/g++-v14/bits/new_allocator.h
/usr/lib/gcc/x86_64-pc-linux-gnu/14/include/g++-v14/ext/new_allocator.h
```
https://github.com/llvm/llvm-project/pull/78534
Endilll wrote:
> why can't we add -verify test? Yes, it will be checking errors that the patch
> didn't touch, but it is what mostly people do when adding clang tests and it
> will be +N test cases in a regular test base which not only ensure your
> change is correct, but the future ones too.
@@ -3051,6 +3051,7 @@ bool Sema::SubstDefaultArgument(
// default argument expression appears.
ContextRAII SavedContext(*this, FD);
std::unique_ptr LIS;
+auto NewTemplateArgs = TemplateArgs;
erichkeane wrote:
Don't use auto here.
@@ -3063,11 +3064,18 @@ bool Sema::SubstDefaultArgument(
/*ForDefinition*/ false);
if (addInstantiatedParametersToScope(FD, PatternFD, *LIS, TemplateArgs))
return true;
+ if (FD->isOutOfLine()) {
+auto *CurrentTemplateArgumentList =
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/78400
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane commented:
Also needs a release note.
https://github.com/llvm/llvm-project/pull/78400
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1098,7 +1098,16 @@ void VerifyDiagnosticConsumer::CheckDiagnostics() {
// Produce an error if no expected-* directives could be found in the
// source file(s) processed.
if (Status == HasNoDirectives) {
-
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/72892
>From dcdb669e8d15ddfed5725efff9d9951306368f7f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Mon, 20 Nov 2023 11:53:40 +0100
Subject: [PATCH] [clang][Interp] Add inline descriptor to global
@@ -0,0 +1,14 @@
+# REQUIRES: x86
+# RUN: mkdir -p %t
+# RUN: llvm-mc -filetype=obj -triple=x86_64-apple-darwin %p/Inputs/libhello.s
-o %t/hello.o
+# RUN: llvm-mc -filetype=obj -triple=x86_64-apple-darwin %s -o %t/main.o
+# RUN: %lld -L %t %t/main.o %t/hello.o -o %t/a.out
@@ -133,12 +195,19 @@ computeOffset(ProgramStateRef State, SValBuilder ,
SVal Location) {
return std::nullopt;
}
-// TODO: once the constraint manager is smart enough to handle non simplified
-// symbolic expressions remove this function. Note that this can not be used in
https://github.com/mariusz-sikora-at-amd edited
https://github.com/llvm/llvm-project/pull/78414
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -626,11 +629,82 @@ class Cvt_PK_F32_F8_Pat;
-foreach Index = [0, -1] in {
- def : Cvt_PK_F32_F8_Pat;
- def : Cvt_PK_F32_F8_Pat;
+let SubtargetPredicate = isGFX9Only in {
+ foreach Index = [0, -1] in {
+def : Cvt_PK_F32_F8_Pat;
+def : Cvt_PK_F32_F8_Pat;
+ }
+}
+
+
https://github.com/kmclaughlin-arm approved this pull request.
https://github.com/llvm/llvm-project/pull/78961
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jsji wrote:
Oh, Thanks, will give it a try, haven't pulled latest this morning.
https://github.com/llvm/llvm-project/pull/78359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jhuber6 wrote:
> FYI. There is a failure in liner-wrapper.c in
> https://buildkite.com/llvm-project/github-pull-requests/builds/30337#018d1aaa-8225-4630-a5f0-527d1c7c129d
>
> ```
> # note: command had no output on stdout or stderr
> | # error: command failed with exit status: 1
> | #
jsji wrote:
FYI. There is a failure in liner-wrapper.c in
https://buildkite.com/llvm-project/github-pull-requests/builds/30337#018d1aaa-8225-4630-a5f0-527d1c7c129d
```
# note: command had no output on stdout or stderr
| # error: command failed with exit status: 1
| # executed command:
Jan =?utf-8?q?Kokemüller?= ,
Jan =?utf-8?q?Kokemüller?= ,
Jan =?utf-8?q?Kokemüller?= ,
Jan =?utf-8?q?Kokemüller?= ,
Jan =?utf-8?q?Kokemüller?= ,
Jan =?utf-8?q?Kokemüller?= ,
Jan =?utf-8?q?Kokemüller?= ,
Jan =?utf-8?q?Kokemüller?= ,
Jan =?utf-8?q?Kokemüller?= ,
Jan =?utf-8?q?Kokemüller?= ,
Jan
https://github.com/gchatelet closed
https://github.com/llvm/llvm-project/pull/78588
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/5chmidti approved this pull request.
https://github.com/llvm/llvm-project/pull/72037
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/legrosbuffle approved this pull request.
https://github.com/llvm/llvm-project/pull/78588
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -535,92 +472,178 @@ struct FPRep : public
FPRepBase {
// - Quiet Not a Number
// - Unnormal
// This can be reduced to the following logic:
-if (exp_bits() == encode(BiasedExponent::BITS_ALL_ONES()))
+if (exp_bits() == encode(BiasedExp::BITS_ALL_ONES()))
https://github.com/gchatelet updated
https://github.com/llvm/llvm-project/pull/78588
>From 49ba96c8aa51fb56a5bf96a1e97fef48bcc42f09 Mon Sep 17 00:00:00 2001
From: Guillaume Chatelet
Date: Thu, 18 Jan 2024 14:42:54 +
Subject: [PATCH 1/3] [libc] `FPRep` builders return `FPRep` instead of raw
Fznamznon wrote:
> I tried my best to explain that it's not an issues of prettiness.
Well, ok, perhaps checking that there is actually no crash in clang can be
tricky. But what I meant is, since there is no good way to check exit code, why
can't we add `-verify` test? Yes, it will be checking
@@ -33,7 +33,66 @@ using namespace taint;
using llvm::formatv;
namespace {
-enum OOB_Kind { OOB_Precedes, OOB_Exceeds, OOB_Taint };
+class StateUpdateReporter {
+ const SubRegion *Reg;
+ NonLoc ByteOffsetVal;
+ std::optional ElementType = std::nullopt;
+ std::optional
mbrkusanin wrote:
> > Correct, some of these instructions use opsel[1] which in LLVM in stored in
> > src1_modifiers so a dummy src1 is used.
>
> Why can't we just use `SRCMODS.OP_SEL_1` with src0?
That could work. We would have to make custom encoding classes then since
OP_SEL_1 would have
@@ -626,11 +629,82 @@ class Cvt_PK_F32_F8_Pat;
-foreach Index = [0, -1] in {
- def : Cvt_PK_F32_F8_Pat;
- def : Cvt_PK_F32_F8_Pat;
+let SubtargetPredicate = isGFX9Only in {
+ foreach Index = [0, -1] in {
+def : Cvt_PK_F32_F8_Pat;
+def : Cvt_PK_F32_F8_Pat;
+ }
+}
+
+
https://github.com/wangpc-pp updated
https://github.com/llvm/llvm-project/pull/78970
>From 8cc71cb7ddb2e6691d31138ae2ef683a0690e171 Mon Sep 17 00:00:00 2001
From: wangpc
Date: Mon, 22 Jan 2024 21:11:42 +0800
Subject: [PATCH 1/2] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
https://github.com/ilinpv approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/78788
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -783,6 +783,28 @@ void tools::addLTOOptions(const ToolChain ,
const ArgList ,
"-generate-arange-section"));
}
+ // Pass vector library arguments to LTO.
+ Arg *ArgVecLib = Args.getLastArg(options::OPT_fveclib);
+ if (ArgVecLib
@@ -783,6 +783,28 @@ void tools::addLTOOptions(const ToolChain ,
const ArgList ,
"-generate-arange-section"));
}
+ // Pass vector library arguments to LTO.
+ Arg *ArgVecLib = Args.getLastArg(options::OPT_fveclib);
+ if (ArgVecLib
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
Author: Matthew Devereau (MDevereau)
Changes
---
Patch is 63.70 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/78961.diff
68 Files Affected:
- (modified) clang/lib/Headers/CMakeLists.txt
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Matthew Devereau (MDevereau)
Changes
---
Patch is 63.70 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/78961.diff
68 Files Affected:
- (modified) clang/lib/Headers/CMakeLists.txt (+3-3)
-
https://github.com/MDevereau ready_for_review
https://github.com/llvm/llvm-project/pull/78961
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/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 fa6025e25b5754e8cf39169e3a7085b57ea35de5
8cc71cb7ddb2e6691d31138ae2ef683a0690e171 --
wangpc-pp wrote:
> Can we split the CodeGen part out of this patch?
MC part is https://github.com/llvm/llvm-project/pull/78970.
This PR will be the CodeGen part and I will update this PR later.
https://github.com/llvm/llvm-project/pull/77424
___
llvmbot wrote:
@llvm/pr-subscribers-mc
@llvm/pr-subscribers-llvm-support
Author: Wang Pengcheng (wangpc-pp)
Changes
`A` extension has been split into two parts: Zaamo (Atomic Memory
Operations) and Zalrsc (Load-Reserved/Store-Conditional). See also
https://github.com/wangpc-pp created
https://github.com/llvm/llvm-project/pull/78970
`A` extension has been split into two parts: Zaamo (Atomic Memory
Operations) and Zalrsc (Load-Reserved/Store-Conditional). See also
https://github.com/riscv/riscv-zaamo-zalrsc.
This patch adds the MC support.
https://github.com/labrinea edited
https://github.com/llvm/llvm-project/pull/78788
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
401 - 500 of 673 matches
Mail list logo