https://github.com/dmasloff edited
https://github.com/llvm/llvm-project/pull/106145
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zmodem wrote:
> > Please let me know if you have a fix for that. I'll land revert in the
> > morning by PDT.
>
> Thank you for flagging this up and opening this @vitalybuka - I'm taking a
> look now to see if there's an easy enough fix to save reverting.
No need to save reverting --- revertin
https://github.com/zmodem closed
https://github.com/llvm/llvm-project/pull/106343
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zyn0217 edited
https://github.com/llvm/llvm-project/pull/102857
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`openmp-offload-amdgpu-runtime` running on `omp-vega20-0` while building
`clang` at step 7 "Add check check-offload".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/30/builds/4934
Here is the relevan
https://github.com/4vtomat created
https://github.com/llvm/llvm-project/pull/106359
The C intrinsic spec is ratified:
https://github.com/riscv-non-isa/rvv-intrinsic-doc/pull/234.
>From 59157100eac32981821eb15bce55d58f5f964bac Mon Sep 17 00:00:00 2001
From: Brandon Wu
Date: Wed, 28 Aug 2024 0
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Brandon Wu (4vtomat)
Changes
The C intrinsic spec is ratified:
https://github.com/riscv-non-isa/rvv-intrinsic-doc/pull/234.
---
Patch is 81.53 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/1
https://github.com/whisperity edited
https://github.com/llvm/llvm-project/pull/106061
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/whisperity edited
https://github.com/llvm/llvm-project/pull/106061
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zyn0217 ready_for_review
https://github.com/llvm/llvm-project/pull/102857
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wangpc-pp commented:
Do we still need this?
```
def Experimental
: SubtargetFeature<"experimental", "HasExperimental",
"true", "Experimental intrinsics">;
```
https://github.com/llvm/llvm-project/pull/106359
4vtomat wrote:
> Do we still need this?
>
> ```
> def Experimental
>: SubtargetFeature<"experimental", "HasExperimental",
> "true", "Experimental intrinsics">;
> ```
I guess we do in case there is any other extension that encounter the same
situation lol~
https://git
tmatheson-arm wrote:
This looks like the right direction to go.
https://github.com/llvm/llvm-project/pull/106304
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Endilll created
https://github.com/llvm/llvm-project/pull/106360
This patch updates `make_cxx_dr_status` script to use the same spoiler-like way
to hide additional details that `cxx_status.html` uses. This gives implemented
yet unresolved DRs new but very familiar look:
![s9
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Vlad Serebrennikov (Endilll)
Changes
This patch updates `make_cxx_dr_status` script to use the same spoiler-like way
to hide additional details that `cxx_status.html` uses. This gives implemented
yet unresolved DRs new but very familiar l
@@ -101,6 +101,26 @@ static_assert(!__is_layout_compatible(StructWithAnonUnion,
StructWithAnonUnion3)
#endif
} // namespace cwg2759
+#if __cplusplus >= 202002L
+namespace cwg2770 { // cwg2770: 20
+template
+struct B {
+ static_assert(sizeof(T) == 1);
+ using type = int;
+};
Author: Reagan
Date: 2024-08-28T12:54:28+02:00
New Revision: 15405b32b1cdbefab9ce1b1f301a51ae25404037
URL:
https://github.com/llvm/llvm-project/commit/15405b32b1cdbefab9ce1b1f301a51ae25404037
DIFF:
https://github.com/llvm/llvm-project/commit/15405b32b1cdbefab9ce1b1f301a51ae25404037.diff
LOG: [
https://github.com/Sirraide closed
https://github.com/llvm/llvm-project/pull/66593
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -568,6 +568,80 @@ namespace cwg137 { // cwg137: yes
const volatile int *cvqcv = static_cast(cvp);
}
+namespace cwg138 { // cwg138: partial
+namespace example1 {
+void foo(); // #cwg138-ex1-foo
+namespace A {
+ using example1::foo; // #cwg138-ex1-using
+ class X {
+s
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/106366
We have type information for them now, so we can do this.
>From 670598d602949cee9bc8cbfe563563aa699d8505 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Wed, 28 Aug 2024 12:58:29 +0200
Subje
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
We have type information for them now, so we can do this.
---
Full diff: https://github.com/llvm/llvm-project/pull/106366.diff
2 Files Affected:
- (modified) clang/lib/AST/ByteCode/Interp.h (+1-1)
- (modi
@@ -101,6 +101,26 @@ static_assert(!__is_layout_compatible(StructWithAnonUnion,
StructWithAnonUnion3)
#endif
} // namespace cwg2759
+#if __cplusplus >= 202002L
+namespace cwg2770 { // cwg2770: 20
+template
+struct B {
+ static_assert(sizeof(T) == 1);
+ using type = int;
+};
https://github.com/nebulark created
https://github.com/llvm/llvm-project/pull/106369
This change moves the flattening of commandline arguments for LF_BUILDINFO into
the compiler frontend. This way other compiler frontends that use LLVM can do
that flattening in a way that makes sense for them.
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
Sirraide wrote:
(Unassigning myself because I’m not familiar enough w/ the details of
substitution to review this in a meaningful way).
That said, I think this is still missing a release note isn’t it? :)
https://github.com/llvm/llvm-project/pull/102857
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (nebulark)
Changes
This change moves the flattening of commandline arguments for LF_BUILDINFO into
the compiler frontend. This way other compiler frontends that use LLVM can do
that flattening in a way that makes sense for them. E.g.
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: None (nebulark)
Changes
This change moves the flattening of commandline arguments for LF_BUILDINFO into
the compiler frontend. This way other compiler frontends that use LLVM can do
that flattening in a way that makes sense for th
llvmbot wrote:
@llvm/pr-subscribers-debuginfo
Author: None (nebulark)
Changes
This change moves the flattening of commandline arguments for LF_BUILDINFO into
the compiler frontend. This way other compiler frontends that use LLVM can do
that flattening in a way that makes sense for them.
zyn0217 wrote:
> (Unassigning myself because I’m not familiar enough w/ the details of
> substitution to review this in a meaningful way).
No problem :)
> That said, I think this is still missing a release note isn’t it? :)
Yeah, I realized it the moment after I pushed the last commit, and I
@@ -1957,15 +1945,13 @@ multiclass VCMLA_ROTS {
def : SOpInst<"vcmla" # ROT # "_lane", "...qI", "Q" # type, Op<(call
"vcmla" # ROT, $p0, $p1,
(bitcast $p0, (dup_typed laneqty , (call "vget_lane", (bitcast
lanety, $p2), $p3>>;
-let isLaneQ = 1 in {
-
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-backend-aarch64
Author: Franklin (FLZ101)
Changes
References:
* Arm Neoverse N3 Software Optimization Guide
* Arm A64 Instruction Set for A-profile architecture
---
Patch is 1.64 MiB, truncated to 20.00 KiB below, full versi
Xazax-hun wrote:
I have no concerns with moving forward here, my understanding is that the
blockers have been resolved. Moreover, we are early in the development cycle
for the next release so we still have a lot of time to get more experience with
this check once it is move out of alpha. But I
usx95 wrote:
What is the reason for changing from `llvm::TimeTraceProfilerEntry*` to
`std::shared_ptr`? We are also adding significant
devirtualization cost with the added different event types.
Can we do something like this:
```
enum class TimeTraceEventType { DurationEvent, InstantEvent };
st
dtcxzyw wrote:
Ping.
https://github.com/llvm/llvm-project/pull/104741
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/106321
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide requested changes to this pull request.
I’m pretty sure there are some fundamental problems with the current approach
wrt how getting source locations for the attribute is being handled.
The general approach for the rest of it seems fine from a cursory look, though.
@@ -2438,6 +2438,22 @@ bool Type::isIncompleteType(NamedDecl **Def) const {
}
}
+bool Type::isIncompletableIncompleteType() const {
+ if (!isIncompleteType())
+return false;
+
+ // Forward declarations of structs, classes, enums, and unions could be later
+ // complet
@@ -186,4 +218,370 @@ bool Sema::CheckCountedByAttrOnField(FieldDecl *FD, Expr
*E, bool CountInBytes,
return false;
}
+SourceRange Sema::BoundsSafetySourceRangeFor(const CountAttributedType *CATy) {
+ // Note: This implementation relies on `CountAttributedType` being uniqu
@@ -186,4 +218,370 @@ bool Sema::CheckCountedByAttrOnField(FieldDecl *FD, Expr
*E, bool CountInBytes,
return false;
}
+SourceRange Sema::BoundsSafetySourceRangeFor(const CountAttributedType *CATy) {
+ // Note: This implementation relies on `CountAttributedType` being uniqu
@@ -186,4 +218,370 @@ bool Sema::CheckCountedByAttrOnField(FieldDecl *FD, Expr
*E, bool CountInBytes,
return false;
}
+SourceRange Sema::BoundsSafetySourceRangeFor(const CountAttributedType *CATy) {
Sirraide wrote:
This entire function seems like a very ba
@@ -3862,6 +3878,27 @@ CountAttributedType::CountAttributedType(
DeclSlot[i] = CoupledDecls[i];
}
+StringRef CountAttributedType::GetAttributeName(bool WithMacroPrefix) const {
Sirraide wrote:
Instead of doing this, it would be better to get a hold of the
@@ -8192,6 +8192,14 @@ ExprResult InitializationSequence::Perform(Sema &S,
Kind.getRange().getEnd());
} else {
CurInit = new (S.Context) ImplicitValueInitExpr(Step->Type);
+// Note the return value isn't used to return early
+// to pres
@@ -2440,6 +2440,26 @@ class alignas(TypeAlignment) Type : public
ExtQualsTypeCommonBase {
return !isFunctionType();
}
+ /// \returns True if the type is incomplete and it is also a type that
+ /// cannot be completed by a later type definition.
+ ///
+ /// E.g. For
@@ -186,4 +218,370 @@ bool Sema::CheckCountedByAttrOnField(FieldDecl *FD, Expr
*E, bool CountInBytes,
return false;
}
+SourceRange Sema::BoundsSafetySourceRangeFor(const CountAttributedType *CATy) {
+ // Note: This implementation relies on `CountAttributedType` being uniqu
@@ -186,4 +218,370 @@ bool Sema::CheckCountedByAttrOnField(FieldDecl *FD, Expr
*E, bool CountInBytes,
return false;
}
+SourceRange Sema::BoundsSafetySourceRangeFor(const CountAttributedType *CATy) {
+ // Note: This implementation relies on `CountAttributedType` being uniqu
@@ -186,4 +218,370 @@ bool Sema::CheckCountedByAttrOnField(FieldDecl *FD, Expr
*E, bool CountInBytes,
return false;
}
+SourceRange Sema::BoundsSafetySourceRangeFor(const CountAttributedType *CATy) {
+ // Note: This implementation relies on `CountAttributedType` being uniqu
@@ -8192,6 +8192,14 @@ ExprResult InitializationSequence::Perform(Sema &S,
Kind.getRange().getEnd());
} else {
CurInit = new (S.Context) ImplicitValueInitExpr(Step->Type);
+// Note the return value isn't used to return early
+// to pres
@@ -8238,6 +8246,15 @@ ExprResult InitializationSequence::Perform(Sema &S,
}
}
+ // Note the return value isn't used to return early so that additional
+ // diagnostics can be emitted and to preserve the AST as best as possible
+ // even though an
@@ -186,4 +218,370 @@ bool Sema::CheckCountedByAttrOnField(FieldDecl *FD, Expr
*E, bool CountInBytes,
return false;
}
+SourceRange Sema::BoundsSafetySourceRangeFor(const CountAttributedType *CATy) {
+ // Note: This implementation relies on `CountAttributedType` being uniqu
@@ -186,4 +218,370 @@ bool Sema::CheckCountedByAttrOnField(FieldDecl *FD, Expr
*E, bool CountInBytes,
return false;
}
+SourceRange Sema::BoundsSafetySourceRangeFor(const CountAttributedType *CATy) {
+ // Note: This implementation relies on `CountAttributedType` being uniqu
krasimirgg wrote:
It appears that this introduced a regression in code like this:
```
int d = 0;
int c(int i) { return i; }
void a() { double b = ((double)c(0)) * d / (double)0; }
```
After this patch, the last line gets formatted as:
```
void a() { double b = ((double)c(0))*d / (double)0; }
```
@@ -380,6 +380,9 @@ ARM_CPU_NAME("neoverse-n1", ARMV8_2A,
FK_CRYPTO_NEON_FP_ARMV8, false,
ARM_CPU_NAME("neoverse-n2", ARMV9A, FK_NEON_FP_ARMV8, false,
(ARM::AEK_BF16 | ARM::AEK_DOTPROD | ARM::AEK_FP16FML |
ARM::AEK_I8MM | ARM::AEK_RAS | ARM::AEK_SB )
prj- wrote:
What's your version?
```
$ clang-format-20 gh.txt
void a() { double b = ((double)c(0)) * d / (double)0; }
```
https://github.com/llvm/llvm-project/pull/102261
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cg
@@ -0,0 +1,12 @@
+// RUN: %clang_cc1 -std=c++20 %s -triple %itanium_abi_triple -emit-llvm -o - |
FileCheck %s
+
+template class S>
+void create_unique()
+ requires (S{0}, true) {}
+
+template struct A {
+ constexpr A(Fn) {};
+};
+
+template void create_unique();
+// CHECK: @_
krasimirgg wrote:
Ohh sorry, I had started with a stale clang-format commit. This is a non-issue
at HEAD.
https://github.com/llvm/llvm-project/pull/102261
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/
https://github.com/NagyDonat edited
https://github.com/llvm/llvm-project/pull/106061
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -285,6 +288,50 @@ void SizeofExpressionCheck::registerMatchers(MatchFinder
*Finder) {
hasRHS(ignoringParenImpCasts(SizeOfExpr.bind("sizeof-ptr-div-expr"
.bind("sizeof-in-ptr-arithmetic-div"),
this);
+
+ // SEI CERT ARR39-C. Do not add or subt
https://github.com/NagyDonat approved this pull request.
LGTM, I didn't spot anything problematic.
https://github.com/llvm/llvm-project/pull/106061
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
https://github.com/NagyDonat edited
https://github.com/llvm/llvm-project/pull/106061
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/106036
>From 67e70c3abc2527d2f35abdc10ce2710dd9ebb918 Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Mon, 26 Aug 2024 07:37:04 +0300
Subject: [PATCH] [Clang] restrict use of attribute names reserved by the C++
st
https://github.com/ivanaivanovska updated
https://github.com/llvm/llvm-project/pull/103039
>From 6d3e4470dfe21f8f3832a2d38b4c4327aa7422a3 Mon Sep 17 00:00:00 2001
From: Ivana Ivanovska
Date: Tue, 13 Aug 2024 10:30:34 +
Subject: [PATCH 1/2] Added instant events and marking defered templated
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/105727
>From d8bed3f4db8056a6afa9bd7eae5d4a8361f83086 Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Thu, 22 Aug 2024 23:25:31 +0300
Subject: [PATCH 1/2] [Clang] prevent assertion failure when converting vectors
@@ -908,9 +936,9 @@ class Analyzer {
static bool isSafeBuiltinFunction(const FunctionDecl *FD) {
unsigned BuiltinID = FD->getBuiltinID();
switch (BuiltinID) {
- case 0: // not builtin
+ case 0: // Not builtin.
return false;
- default: //
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/99656
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -210,24 +210,88 @@ void nb18(void (^block)() [[clang::nonblocking]])
[[clang::nonblocking]]
}
// Builtin functions
-void nb18a() [[clang::nonblocking]] {
+void nb19() [[clang::nonblocking]] {
__builtin_assume(1);
void *ptr = __builtin_malloc(1); // expected
@@ -1187,13 +1234,261 @@ Analyzer::AnalysisMap::~AnalysisMap() {
namespace clang {
-void performEffectAnalysis(Sema &S, TranslationUnitDecl *TU) {
- if (S.hasUncompilableErrorOccurred() || S.Diags.getIgnoreAllWarnings())
-// exit if having uncompilable errors or ignoring
@@ -1,44 +1,67 @@
-//=== EffectAnalysis.cpp - Sema warnings for function effects
-===//
+//=== SemaFunctionEffects.cpp - Sema handling of function effects
-===//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
// See ht
@@ -210,24 +210,88 @@ void nb18(void (^block)() [[clang::nonblocking]])
[[clang::nonblocking]]
}
// Builtin functions
-void nb18a() [[clang::nonblocking]] {
+void nb19() [[clang::nonblocking]] {
__builtin_assume(1);
void *ptr = __builtin_malloc(1); // expected
https://github.com/Sirraide commented:
I think this is in pretty good shape at this point. I’ve pointed out a few more
things I noticed just now, but I think at this point I’d like another set of
eyes to come and take a look at this in case I missed something.
https://github.com/llvm/llvm-proj
@@ -0,0 +1,33 @@
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx11 -pedantic -std=c++11 %s
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx14 -pedantic -std=c++14 %s
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx17 -pedantic -std=c++17 %s
+// RUN: %clang_cc1 -
@@ -0,0 +1,33 @@
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx11 -pedantic -std=c++11 %s
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx14 -pedantic -std=c++14 %s
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx17 -pedantic -std=c++17 %s
+// RUN: %clang_cc1 -
@@ -0,0 +1,33 @@
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx11 -pedantic -std=c++11 %s
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx14 -pedantic -std=c++14 %s
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx17 -pedantic -std=c++17 %s
+// RUN: %clang_cc1 -
https://github.com/yronglin updated
https://github.com/llvm/llvm-project/pull/105996
>From 0e4c511107f76da085a8019cf2eca78c3a5a0754 Mon Sep 17 00:00:00 2001
From: yronglin
Date: Mon, 26 Aug 2024 02:09:31 +0800
Subject: [PATCH 1/6] [Clang][Interp] Implement constexpr vector unary
operators +, -
@@ -177,6 +177,26 @@ static bool isLanguageDefinedBuiltin(const SourceManager
&SourceMgr,
return false;
}
+static bool isReservedAttrName(Preprocessor &PP, IdentifierInfo *II) {
philnik777 wrote:
`-nostdlib` doesn't imply that you don't have a standard lib
@@ -0,0 +1,33 @@
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx11 -pedantic -std=c++11 %s
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx14 -pedantic -std=c++14 %s
+// RUN: %clang_cc1 -fsyntax-only -verify=expected,cxx17 -pedantic -std=c++17 %s
+// RUN: %clang_cc1 -
https://github.com/usx95 edited https://github.com/llvm/llvm-project/pull/106277
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,12 @@
+// RUN: %clang_cc1 -std=c++20 %s -triple %itanium_abi_triple -emit-llvm -o - |
FileCheck %s
+
+template class S>
+void create_unique()
+ requires (S{0}, true) {}
+
+template struct A {
+ constexpr A(Fn) {};
+};
+
+template void create_unique();
+// CHECK: @_
cor3ntin wrote:
> > Can you also add a changelog entry? Thanks!
>
> I've never seen them used in LLVM. Has that changed recently?
Checkout `clang/docs/ReleaseNotes.rst`
https://github.com/llvm/llvm-project/pull/106353
___
cfe-commits mailing list
cf
https://github.com/Endilll closed
https://github.com/llvm/llvm-project/pull/106360
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -5312,6 +5314,120 @@ bool Compiler::VisitComplexUnaryOperator(const
UnaryOperator *E) {
return true;
}
+template
+bool Compiler::VisitVectorUnaryOperator(const UnaryOperator *E) {
+ const Expr *SubExpr = E->getSubExpr();
+ assert(SubExpr->getType()->isVectorType());
+
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 c992690179eb5de6efe47d5c8f3a23f2302723f2
adf8f04eac4ff6e71c985bb991f68c2238a3462f --e
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 96e1320a9aa8fd010b02fc6751da801c48725a02
b3d7b528a422f7c44b535c480acbd74f03555b4d --e
0xzre wrote:
> FYI the tests are failing
@cor3ntin The solution here #79748 doesn't pass the test too. Kinda expected
not to pass the test clang
https://github.com/llvm/llvm-project/pull/102878
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/pratlucas updated
https://github.com/llvm/llvm-project/pull/106391
>From b3d7b528a422f7c44b535c480acbd74f03555b4d Mon Sep 17 00:00:00 2001
From: Lucas Prates
Date: Wed, 28 Aug 2024 13:58:57 +0100
Subject: [PATCH 1/2] [Driver] Add -mbranch-protection to ARM and AArch64
multil
https://github.com/momchil-velikov updated
https://github.com/llvm/llvm-project/pull/99041
>From 87f1a5aa2215a9fbc1bde7905b2fd5e8d5ff859a Mon Sep 17 00:00:00 2001
From: Momchil Velikov
Date: Mon, 15 Jul 2024 17:50:43 +0100
Subject: [PATCH 1/2] [AArch64] Implement NEON vamin/vamax intrinsics
Th
https://github.com/ostannard approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/106391
___
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/106395
Ignore the multiplication overflow but report the 0 denominator.
>From 7b1cfea20ae76a7b04ae63bf6d6082488c17b2e3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Wed, 28 Aug 2024 15:50:10 +020
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
Ignore the multiplication overflow but report the 0 denominator.
---
Full diff: https://github.com/llvm/llvm-project/pull/106395.diff
2 Files Affected:
- (modified) clang/lib/AST/ByteCode/Interp.h (+9-2)
https://github.com/dklimkin created
https://github.com/llvm/llvm-project/pull/106398
None
>From edeb17fdfb5ef6ce074fbdeec35d3e28b71592ca Mon Sep 17 00:00:00 2001
From: Danial Klimkin
Date: Wed, 28 Aug 2024 15:58:23 +0200
Subject: [PATCH] Fix clang-format-test.el past 625841c
---
clang/tools/
https://github.com/dkrupp updated
https://github.com/llvm/llvm-project/pull/106389
>From e979542270b21f4733baf25a7037675af598ca07 Mon Sep 17 00:00:00 2001
From: Daniel Krupp
Date: Wed, 28 Aug 2024 15:32:35 +0200
Subject: [PATCH] Adding optin.taint.TaintedDiv checker
Tainted division operation
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Danial Klimkin (dklimkin)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/106398.diff
1 Files Affected:
- (modified) clang/tools/clang-format/clang-format-test.el (+5-5)
``diff
diff --git a/clang/tool
https://github.com/dklimkin closed
https://github.com/llvm/llvm-project/pull/106398
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Danial Klimkin
Date: 2024-08-28T16:00:50+02:00
New Revision: 158ba731c9ecaf39f1790728a589797599c38af8
URL:
https://github.com/llvm/llvm-project/commit/158ba731c9ecaf39f1790728a589797599c38af8
DIFF:
https://github.com/llvm/llvm-project/commit/158ba731c9ecaf39f1790728a589797599c38af8.diff
https://github.com/kiran-isaac updated
https://github.com/llvm/llvm-project/pull/102896
>From eb7551e83618d8452f5dadae1be4aff8f6c9d23c Mon Sep 17 00:00:00 2001
From: Kiran
Date: Thu, 8 Aug 2024 13:07:24 +0100
Subject: [PATCH 1/2] [ARM] musttail fixes
Backend:
- Caller and callee arguments no l
nikic wrote:
@mikaelholmen Thanks for the reproducer, this makes the issue clear. BasicAA is
incorrectly returning NoAlias for the pointers due to
https://github.com/llvm/llvm-project/pull/98608.
The issue is that the `add` gets decomposed, but we preserve the nuw flag,
effectively making it
https://github.com/dkrupp updated
https://github.com/llvm/llvm-project/pull/106389
>From ccc5da054903568fbd317d5c773251ed84f8f087 Mon Sep 17 00:00:00 2001
From: Daniel Krupp
Date: Wed, 28 Aug 2024 15:32:35 +0200
Subject: [PATCH] Adding optin.taint.TaintedDiv checker
Tainted division operation
https://github.com/kiran-isaac updated
https://github.com/llvm/llvm-project/pull/102896
>From eb7551e83618d8452f5dadae1be4aff8f6c9d23c Mon Sep 17 00:00:00 2001
From: Kiran
Date: Thu, 8 Aug 2024 13:07:24 +0100
Subject: [PATCH 1/3] [ARM] musttail fixes
Backend:
- Caller and callee arguments no l
https://github.com/kiran-isaac updated
https://github.com/llvm/llvm-project/pull/102896
>From eb7551e83618d8452f5dadae1be4aff8f6c9d23c Mon Sep 17 00:00:00 2001
From: Kiran
Date: Thu, 8 Aug 2024 13:07:24 +0100
Subject: [PATCH 1/3] [ARM] musttail fixes
Backend:
- Caller and callee arguments no l
@@ -3088,70 +3057,44 @@ bool
ARMTargetLowering::IsEligibleForTailCallOptimization(
getEffectiveCallingConv(CalleeCC, isVarArg),
getEffectiveCallingConv(CallerCC, CallerF.isVarArg()), MF, C, Ins,
CCAssignFnForReturn(CalleeCC, isVarArg),
-
@@ -67,33 +327,85 @@ declare i32 @e3(ptr nocapture byval(%struct.LargeStruct)
align 16 %in) nounwind
; We can't do tail call since address of s is passed to the callee and part of
; s is in caller's local frame.
kiran-isaac wrote:
I agree, I have removed the c
101 - 200 of 503 matches
Mail list logo