Author: MagentaTreehouse
Date: 2024-09-14T09:00:48+02:00
New Revision: 1d2f7277e695d046efaf2818dd1a4b251ce745fd
URL:
https://github.com/llvm/llvm-project/commit/1d2f7277e695d046efaf2818dd1a4b251ce745fd
DIFF:
https://github.com/llvm/llvm-project/commit/1d2f7277e695d046efaf2818dd1a4b251ce745fd.di
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/97908
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/108675
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MK-Alias wrote:
Thanks for the input, a followup question is: Which one takes precedence? Does
the `--function-arg-placeholders` argument take precedence over `.clangd`
config!? Or the other way around!?
https://github.com/llvm/llvm-project/pull/108005
HighCommander4 wrote:
> Thanks for the input, a followup question is: Which one takes precedence?
> Does the `--function-arg-placeholders` argument take precedence over
> `.clangd` config!? Or the other way around!?
`.clangd` should take precedence because it's the a more specific setting
(co
@@ -18197,7 +18197,7 @@ Value *CodeGenFunction::EmitPPCBuiltinExpr(unsigned
BuiltinID,
CallOps.push_back(Ops[i]);
llvm::Function *F = CGM.getIntrinsic(ID);
Value *Call = Builder.CreateCall(F, CallOps);
-return Builder.CreateAlignedStore(Call, Ops[0], MaybeAli
nikic wrote:
It looks like you're trying to undo
https://github.com/llvm/llvm-project/pull/106612? cc @mgorny
https://github.com/llvm/llvm-project/pull/108549
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mail
shr-project wrote:
> It looks like you're trying to undo #106612? cc @mgorny
Ah, not intentionally, I should have checked the git history more carefully
before creating this PR (which is still only a draft). I don't use libscanbuild
at all, but in OpenEmbedded builds the python modules are ins
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/108689
Fixes: #106444
>From af8720ade42fef1571b59e2ca21943abc6b998d1 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Sat, 14 Sep 2024 15:38:11 +0800
Subject: [PATCH] [ast-matcher] Fixed a crash when traverse la
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Congcong Cai (HerrCai0907)
Changes
Fixes: #106444
---
Full diff: https://github.com/llvm/llvm-project/pull/108689.diff
3 Files Affected:
- (modified) clang/docs/ReleaseNotes.rst (+2)
- (modified) clang/lib/ASTMatchers/ASTMatchFinder.c
Author: JOE1994
Date: 2024-09-14T04:38:50-04:00
New Revision: 918972bded27de6a2bfacc15b4ad3edebd81f405
URL:
https://github.com/llvm/llvm-project/commit/918972bded27de6a2bfacc15b4ad3edebd81f405
DIFF:
https://github.com/llvm/llvm-project/commit/918972bded27de6a2bfacc15b4ad3edebd81f405.diff
LOG:
mgorny wrote:
Sounds like a bug in OpenEmbedded then. CPython installs Python modules in
`/usr/lib`, unconditionally.
https://github.com/llvm/llvm-project/pull/108549
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-b
alex1701c wrote:
That is very unfortunate. It means one will either get different results for
the clang-format versions, one disables the formatting, or works around it by
sth. Like Q_EMIT(something()->mySignal()).
https://github.com/llvm/llvm-project/pull/99791
___
philnik777 wrote:
gentle ping~
https://github.com/llvm/llvm-project/pull/106005
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -72,11 +81,24 @@ void AvoidCArraysCheck::registerMatchers(MatchFinder
*Finder) {
void AvoidCArraysCheck::check(const MatchFinder::MatchResult &Result) {
const auto *ArrayType = Result.Nodes.getNodeAs("typeloc");
-
+ const bool IsInParam =
+ Result.Nodes.getNodeAs("
philnik777 wrote:
gentle ping~
https://github.com/llvm/llvm-project/pull/99473
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -73,8 +73,6 @@ enum CPUFeatures {
FEAT_SSBS,
FEAT_SSBS2,
FEAT_BTI,
- FEAT_LS64,
- FEAT_LS64_V,
Wilco1 wrote:
It became a defacto ABI when the first implementation became available. We
could change the names of the global/init function to explicitly
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/108555
>From 35742b5f2f085b8f6626456cd0d21ebecbe8f831 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 12 Sep 2024 23:11:51 +0800
Subject: [PATCH 1/3] [clang-tidy]suggest use `std::span` as replacement of c
pskrgag wrote:
> LGTM, feel free to merge this patch.
Thank you for review! I have no rights to push to llvm-project. So you can
merge if you think it's good time to do so =)
https://github.com/llvm/llvm-project/pull/106081
___
cfe-commits mailing li
Author: JOE1994
Date: 2024-09-14T05:29:40-04:00
New Revision: 223e2efa5e886502a9467b7ef700ebce9b7886e8
URL:
https://github.com/llvm/llvm-project/commit/223e2efa5e886502a9467b7ef700ebce9b7886e8
DIFF:
https://github.com/llvm/llvm-project/commit/223e2efa5e886502a9467b7ef700ebce9b7886e8.diff
LOG:
https://github.com/zyn0217 created
https://github.com/llvm/llvm-project/pull/108693
When rebuilding immediate invocations inside
`RemoveNestedImmediateInvocation()`, we employed a `TreeTransform` to exercise
the traversal. The transformation has a side effect that, for template
specialization
@@ -249,7 +249,10 @@ Attribute Changes in Clang
(#GH106864)
- Introduced a new attribute ``[[clang::coro_await_elidable]]`` on coroutine
return types
- to express elideability at call sites where the coroutine is co_awaited as a
prvalue.
+ to express elideability at call
@@ -8281,8 +8288,62 @@ Example:
co_await t;
}
-The behavior is undefined if the caller coroutine is destroyed earlier than the
-callee coroutine.
+Such elision replaces the heap allocated activation frame of the callee
coroutine
+with a local variable within the enclosi
@@ -8261,12 +8261,19 @@ def CoroAwaitElidableDoc : Documentation {
The ``[[clang::coro_await_elidable]]`` is a class attribute which can be
applied
to a coroutine return type.
-When a coroutine function that returns such a type calls another coroutine
function,
-the compiler
@@ -8281,8 +8288,62 @@ Example:
co_await t;
}
-The behavior is undefined if the caller coroutine is destroyed earlier than the
-callee coroutine.
+Such elision replaces the heap allocated activation frame of the callee
coroutine
+with a local variable within the enclosi
@@ -8281,8 +8288,62 @@ Example:
co_await t;
}
-The behavior is undefined if the caller coroutine is destroyed earlier than the
-callee coroutine.
+Such elision replaces the heap allocated activation frame of the callee
coroutine
+with a local variable within the enclosi
@@ -8261,12 +8261,19 @@ def CoroAwaitElidableDoc : Documentation {
The ``[[clang::coro_await_elidable]]`` is a class attribute which can be
applied
to a coroutine return type.
vogelsgesang wrote:
```suggestion
to a coroutine return type. It provides a hint to
@@ -84,4 +84,35 @@ Task nonelidable() {
co_return 1;
}
+// CHECK-LABEL: define{{.*}} @_Z8addTasksO4TaskIiES1_{{.*}} {
+Task addTasks([[clang::coro_must_await]] Task &&t1, Task &&t2) {
+ int i1 = co_await t1;
+ int i2 = co_await t2;
+ co_return i1 + i2;
+}
+
+// CHECK-LABE
@@ -84,4 +84,35 @@ Task nonelidable() {
co_return 1;
}
+// CHECK-LABEL: define{{.*}} @_Z8addTasksO4TaskIiES1_{{.*}} {
+Task addTasks([[clang::coro_must_await]] Task &&t1, Task &&t2) {
+ int i1 = co_await t1;
+ int i2 = co_await t2;
+ co_return i1 + i2;
+}
+
+// CHECK-LABE
@@ -880,14 +898,12 @@ ExprResult
Sema::BuildUnresolvedCoawaitExpr(SourceLocation Loc, Expr *Operand,
}
auto *RD = Promise->getType()->getAsCXXRecordDecl();
- bool AwaitElidable =
- isCoroAwaitElidableCall(Operand) &&
- isAttributedCoroAwaitElidable(
-
@@ -8261,12 +8261,19 @@ def CoroAwaitElidableDoc : Documentation {
The ``[[clang::coro_await_elidable]]`` is a class attribute which can be
applied
to a coroutine return type.
-When a coroutine function that returns such a type calls another coroutine
function,
-the compiler
https://github.com/vogelsgesang commented:
Thanks for implementing this! Looking forward to using it 🙂
https://github.com/llvm/llvm-project/pull/108474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
https://github.com/vogelsgesang edited
https://github.com/llvm/llvm-project/pull/108474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MK-Alias wrote:
Because the `.clangd` config is made out of blocks, and no block was specified
for our new `ArgumentLists` key, I've put it under `Completion`. So now it is
like
```yaml
Completion:
AllScopes: None
ArgumentLists: None
```
It seems like the most logical choice, because it's
Author: Congcong Cai
Date: 2024-09-14T21:27:35+08:00
New Revision: 11f9008daec2b5593dfcc781ee879f93de121da2
URL:
https://github.com/llvm/llvm-project/commit/11f9008daec2b5593dfcc781ee879f93de121da2
DIFF:
https://github.com/llvm/llvm-project/commit/11f9008daec2b5593dfcc781ee879f93de121da2.diff
https://github.com/a-tarasyuk created
https://github.com/llvm/llvm-project/pull/108701
Fixes #108687
>From 3e7e69e36e93f12c33ccdda9fea78e22cc9bdd84 Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Sat, 14 Sep 2024 17:02:01 +0300
Subject: [PATCH] [Clang] handle invalid close location in static
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Oleksandr T. (a-tarasyuk)
Changes
Fixes #108687
---
Full diff: https://github.com/llvm/llvm-project/pull/108701.diff
3 Files Affected:
- (modified) clang/docs/ReleaseNotes.rst (+1-1)
- (modified) clang/lib/Parse/ParseDeclCXX.cpp (+2)
Author: Kazu Hirata
Date: 2024-09-14T07:39:00-07:00
New Revision: bae275f65ecd27a16743f7b4695940dcfa70b218
URL:
https://github.com/llvm/llvm-project/commit/bae275f65ecd27a16743f7b4695940dcfa70b218
DIFF:
https://github.com/llvm/llvm-project/commit/bae275f65ecd27a16743f7b4695940dcfa70b218.diff
L
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/108675
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -73,8 +73,6 @@ enum CPUFeatures {
FEAT_SSBS,
FEAT_SSBS2,
FEAT_BTI,
- FEAT_LS64,
- FEAT_LS64_V,
jroelofs wrote:
We could do that automatically every release by putting the version number in
its name.
https://github.com/llvm/llvm-project/pull/108024
@@ -1110,6 +1110,8 @@ Decl *Parser::ParseStaticAssertDeclaration(SourceLocation
&DeclEnd) {
}
T.consumeClose();
+ if (T.getCloseLocation().isInvalid())
+return nullptr;
cor3ntin wrote:
`consumeClose` returms true on failure so this can just be
`if(
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/108701
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/108706
Fixes: #108049
cert-flp30-c only provides non-compliant example with normal loop.
Essentially it wants to avoid that floating point variables are used as loop
counters which are checked in condition expr and
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Congcong Cai (HerrCai0907)
Changes
Fixes: #108049
cert-flp30-c only provides non-compliant example with normal loop.
Essentially it wants to avoid that floating point variables are used as loop
counters which are checked in con
@@ -361,6 +361,23 @@ bool any(double3);
_HLSL_BUILTIN_ALIAS(__builtin_hlsl_any)
bool any(double4);
+//===--===//
+// asfloat builtins
+//===-
@@ -945,50 +959,11 @@ class DiagnosticsEngine : public
RefCountedBase {
void Report(const StoredDiagnostic &storedDiag);
- /// Determine whethere there is already a diagnostic in flight.
- bool isDiagnosticInFlight() const {
-return CurDiagID != std::numeric_limits::
https://github.com/joaosaffran updated
https://github.com/llvm/llvm-project/pull/108686
>From 32e71ea8fe790ea81163fac76bd2029991d780bd Mon Sep 17 00:00:00 2001
From: Joao Saffran
Date: Sat, 14 Sep 2024 06:23:08 +
Subject: [PATCH] implementing asfloat using bit_cast
---
clang/lib/Headers/h
@@ -361,6 +361,23 @@ bool any(double3);
_HLSL_BUILTIN_ALIAS(__builtin_hlsl_any)
bool any(double4);
+//===--===//
+// asfloat builtins
+//===-
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/108701
>From cbe06cf33277f904fe002db90d428a55b777ff5d Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Sat, 14 Sep 2024 17:02:01 +0300
Subject: [PATCH] [Clang] handle invalid close location in static assert
declara
@@ -1110,6 +1110,8 @@ Decl *Parser::ParseStaticAssertDeclaration(SourceLocation
&DeclEnd) {
}
T.consumeClose();
+ if (T.getCloseLocation().isInvalid())
+return nullptr;
a-tarasyuk wrote:
@cor3ntin Thanks for the review. I've added changes to omit re
https://github.com/e-kwsm updated
https://github.com/llvm/llvm-project/pull/94014
>From 3fcf81d949e461fb91b4e1713365da29ae1078cd Mon Sep 17 00:00:00 2001
From: Eisuke Kawashima
Date: Sat, 11 May 2024 23:57:11 +0900
Subject: [PATCH] fix(clang/**.py): fix comparison to None
from PEP8 (https://pe
HighCommander4 wrote:
> Is this okay!?
Yep. By the way, this was specified in the [original
proposal](https://github.com/llvm/llvm-project/issues/63565#issuecomment-1975065771):
> Here's a concrete proposal for a config file syntax that would address this
> request, https://github.com/clangd/
https://github.com/MK-Alias updated
https://github.com/llvm/llvm-project/pull/108005
>From 4e9cbd615c56604fbc3c05f677b9da11f30977a2 Mon Sep 17 00:00:00 2001
From: MK-Alias
Date: Tue, 10 Sep 2024 13:24:46 +0200
Subject: [PATCH 1/2] issue-63565: community requested small QoL fix for more
configu
MK-Alias wrote:
Updated the source. It should now confirm to everything. To recap: `.clangd`
now has a added key: `ArgumentLists` in the `Completion` block. It supports
these values: `None` | `OpenDelimiter` | `Delimiters` | `FullPlaceholders`. The
`.clangd` config will overwite the prevously
https://github.com/MK-Alias updated
https://github.com/llvm/llvm-project/pull/108005
>From 4e9cbd615c56604fbc3c05f677b9da11f30977a2 Mon Sep 17 00:00:00 2001
From: MK-Alias
Date: Tue, 10 Sep 2024 13:24:46 +0200
Subject: [PATCH 1/3] issue-63565: community requested small QoL fix for more
configu
https://github.com/nicovank approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/106856
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MK-Alias updated
https://github.com/llvm/llvm-project/pull/108005
>From 4e9cbd615c56604fbc3c05f677b9da11f30977a2 Mon Sep 17 00:00:00 2001
From: MK-Alias
Date: Tue, 10 Sep 2024 13:24:46 +0200
Subject: [PATCH 1/4] issue-63565: community requested small QoL fix for more
configu
https://github.com/gedare created
https://github.com/llvm/llvm-project/pull/108717
Fixes #73584.
>From a95b990e48df19b8b674fe9df6bea803415129bf Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Sat, 14 Sep 2024 13:13:26 -0600
Subject: [PATCH] [clang-format] Fix regression with BlockIndent of B
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Gedare Bloom (gedare)
Changes
Fixes #73584.
---
Full diff: https://github.com/llvm/llvm-project/pull/108717.diff
2 Files Affected:
- (modified) clang/lib/Format/ContinuationIndenter.cpp (+7)
- (modified) clang/unittests/Format/F
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/108634
>From cc3f586d2fee5b89eece3bf341d1ffe1d169e3f7 Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Fri, 13 Sep 2024 13:41:49 -0600
Subject: [PATCH 1/2] [clang-format] Fix regression in AlwaysBreak for-await
Fixes
@@ -2,3 +2,15 @@ config.substitutions = list(config.substitutions)
config.substitutions.insert(
0, (r"%clang\b", """*** Do not use the driver in Sema tests. ***""")
)
+
+if lit_config.update_tests:
+import sys
+import os
+
+curdir = os.path.dirname(os.path.realp
@@ -1,3 +1,5 @@
+// RUN: %clang_cc1 -verify %s
+// RUN: diff %s %s.expected
nhaehnle wrote:
I still think those new invocations of `diff` should be in `lit-plugin.test`.
Those `diff` lines attempt to check that this file (or rather its temporarily
copied versio
https://github.com/nhaehnle commented:
Thanks for splitting this up. It does look mostly good to me, but I do have two
comments.
https://github.com/llvm/llvm-project/pull/108425
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llv
https://github.com/nhaehnle edited
https://github.com/llvm/llvm-project/pull/108425
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/igelbox updated
https://github.com/llvm/llvm-project/pull/108187
>From aee4cf70dedaa3c8b7b6508238e01f92d60c631c Mon Sep 17 00:00:00 2001
From: Sergei
Date: Tue, 10 Sep 2024 16:19:05 +
Subject: [PATCH 1/4] fix quick OOM in FormatDiagnostic
---
.../ClangTidyDiagnosticCons
@@ -945,50 +959,11 @@ class DiagnosticsEngine : public
RefCountedBase {
void Report(const StoredDiagnostic &storedDiag);
- /// Determine whethere there is already a diagnostic in flight.
- bool isDiagnosticInFlight() const {
-return CurDiagID != std::numeric_limits::
Author: Nikolas Klauser
Date: 2024-09-14T22:25:08+02:00
New Revision: e39205654dc11c50bd117e8ccac243a641ebd71f
URL:
https://github.com/llvm/llvm-project/commit/e39205654dc11c50bd117e8ccac243a641ebd71f
DIFF:
https://github.com/llvm/llvm-project/commit/e39205654dc11c50bd117e8ccac243a641ebd71f.dif
Author: Nikolas Klauser
Date: 2024-09-14T22:25:08+02:00
New Revision: e39205654dc11c50bd117e8ccac243a641ebd71f
URL:
https://github.com/llvm/llvm-project/commit/e39205654dc11c50bd117e8ccac243a641ebd71f
DIFF:
https://github.com/llvm/llvm-project/commit/e39205654dc11c50bd117e8ccac243a641ebd71f.dif
Author: Martin Storsjö
Date: 2024-09-15T00:02:39+03:00
New Revision: 7d11f5249c8068851270e94ac02c180e7b83590d
URL:
https://github.com/llvm/llvm-project/commit/7d11f5249c8068851270e94ac02c180e7b83590d
DIFF:
https://github.com/llvm/llvm-project/commit/7d11f5249c8068851270e94ac02c180e7b83590d.diff
https://github.com/cor3ntin approved this pull request.
Thanks, LGTM
https://github.com/llvm/llvm-project/pull/108701
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
a-tarasyuk wrote:
@cor3ntin when you get a chance, could you merge this? I don’t have the
permissions. thanks!
https://github.com/llvm/llvm-project/pull/108701
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mail
shafik wrote:
So this looks like a regression: https://gcc.godbolt.org/z/h6GexT18E
Can we figure out which PR this regression came from and make sure the code
change makes sense in light of that PR?
Also please add a more detailed summary to this PR, describing why the crash
happens and the
https://github.com/shafik requested changes to this pull request.
So this looks like a regression: https://gcc.godbolt.org/z/h6GexT18E
Can we figure out which PR this regression came from and make sure the code
change makes sense in light of that PR?
Also please add a more detailed summary to
a-tarasyuk wrote:
> So this looks like a regression: https://gcc.godbolt.org/z/h6GexT18E
@shafik https://gcc.godbolt.org/z/Ebvnjrc8e
https://github.com/llvm/llvm-project/pull/108701
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists
shafik wrote:
> > So this looks like a regression: https://gcc.godbolt.org/z/h6GexT18E
>
> @shafik https://gcc.godbolt.org/z/Ebvnjrc8e
Interesting it crashes in trunk w/o `c++20`:
https://gcc.godbolt.org/z/h9xa8eaqo but the backtrace looks the same.
We should test this in C++20 and C++23 as w
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/108701
>From cbe06cf33277f904fe002db90d428a55b777ff5d Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Sat, 14 Sep 2024 17:02:01 +0300
Subject: [PATCH 1/2] [Clang] handle invalid close location in static assert
dec
https://github.com/edenreich created
https://github.com/llvm/llvm-project/pull/108721
## Summary
Currently it's not backward compatible, because the flag -list-ignored doesn't
exists on older version of clang-format.
To make this tool still backward compatible, we can first check if the
.cla
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
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Eden Reich (edenreich)
Changes
## Summary
Currently it's not backward compatible, because the flag -list-ignored doesn't
exists on older version of clang-format.
To make this tool still backward compatible, we can first check if t
https://github.com/edenreich edited
https://github.com/llvm/llvm-project/pull/108721
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/edenreich edited
https://github.com/llvm/llvm-project/pull/108721
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/edenreich edited
https://github.com/llvm/llvm-project/pull/108721
___
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/108693
___
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: Younan Zhang (zyn0217)
Changes
When rebuilding immediate invocations inside
`RemoveNestedImmediateInvocation()`, we employed a `TreeTransform` to exercise
the traversal. The transformation has a side effect that, for template
specializat
@@ -0,0 +1,5 @@
+// RUN: %clang_cc1 -std=c++2a -triple=x86_64-linux -fsyntax-only %s -verify
zyn0217 wrote:
You can add another C++20 mode run to
clang/test/SemaCXX/static-assert-cxx26.cpp, instead of adding a new file.
https://github.com/llvm/llvm-project/pull
@@ -0,0 +1,5 @@
+// RUN: %clang_cc1 -std=c++2a -triple=x86_64-linux -fsyntax-only %s -verify
a-tarasyuk wrote:
The file refers to cxx26 —is it ok to add tests related to C++20 here?"
https://github.com/llvm/llvm-project/pull/108701
__
https://github.com/a-tarasyuk edited
https://github.com/llvm/llvm-project/pull/108701
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,5 @@
+// RUN: %clang_cc1 -std=c++2a -triple=x86_64-linux -fsyntax-only %s -verify
zyn0217 wrote:
I believe so, yes. See the example:
clang/test/SemaCXX/constant-expression-cxx2b.cpp
https://github.com/llvm/llvm-project/pull/108701
___
@@ -0,0 +1,5 @@
+// RUN: %clang_cc1 -std=c++2a -triple=x86_64-linux -fsyntax-only %s -verify
zyn0217 wrote:
Also, you might want to wrap the case with a namespace referring to the issue
number.
Furthermore, I think the test should be triaged as a Parser one. Ca
zyn0217 wrote:
> Can we figure out which PR this regression came from and make sure the code
> change makes sense in light of that PR?
https://github.com/llvm/llvm-project/pull/102044. We have backported the
P2741R3 implementation to C++11.
https://github.com/llvm/llvm-project/pull/108701
__
https://github.com/zyn0217 commented:
So I took a stab at it.
In `ParseStaticAssertDeclaration()`, we parsed the message in unclosed
static_assert as a constant expression rather than an unevaluated string
literal per the logic on lines 1079-1089.
So this approach gives up building an express
https://github.com/zyn0217 edited
https://github.com/llvm/llvm-project/pull/108701
___
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/108701
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vegerot wrote:
@AaronBallman sure. Should I still use http://llvm-compile-time-tracker.com/ ,
or do I run the benchmark another way?
https://github.com/llvm/llvm-project/pull/97926
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://list
https://github.com/c8ef created https://github.com/llvm/llvm-project/pull/108725
ref:
https://docs.python.org/3/library/unittest.html#unittest.TestCase.assertEqual
>From 1f56fd3444b6f7667a7036a29cba825f7eac77b0 Mon Sep 17 00:00:00 2001
From: c8ef
Date: Sun, 15 Sep 2024 10:00:12 +0800
Subject:
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (c8ef)
Changes
ref:
https://docs.python.org/3/library/unittest.html#unittest.TestCase.assertEqual
---
Full diff: https://github.com/llvm/llvm-project/pull/108725.diff
1 Files Affected:
- (modified) clang/tools/scan-build-py/tests/
https://github.com/vegerot updated
https://github.com/llvm/llvm-project/pull/97926
>From 7d221cbcf91248f44b016aea107109756fa9f785 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Max=20=F0=9F=91=A8=F0=9F=8F=BD=E2=80=8D=F0=9F=92=BB=20Copl?=
=?UTF-8?q?an?=
Date: Sat, 6 Jul 2024 17:22:55 -0700
Subject: [
https://github.com/kyulee-com updated
https://github.com/llvm/llvm-project/pull/90934
>From 95709740e820c0efddb1fdb53436a03194a1b88e Mon Sep 17 00:00:00 2001
From: Kyungwoo Lee
Date: Fri, 26 Apr 2024 20:02:52 -0700
Subject: [PATCH] [ThinLTO][NFC] Prep for two-codegen rounds
---
clang/lib/Code
https://github.com/kyulee-com edited
https://github.com/llvm/llvm-project/pull/90934
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kyulee-com edited
https://github.com/llvm/llvm-project/pull/90934
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 109 matches
Mail list logo