Author: Owen Pan
Date: 2024-01-20T00:08:23-08:00
New Revision: a7d7da6e45992b79fe712c1e228cc57c9f27fa7a
URL:
https://github.com/llvm/llvm-project/commit/a7d7da6e45992b79fe712c1e228cc57c9f27fa7a
DIFF:
https://github.com/llvm/llvm-project/commit/a7d7da6e45992b79fe712c1e228cc57c9f27fa7a.diff
LOG:
https://github.com/owenca closed https://github.com/llvm/llvm-project/pull/78682
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca closed https://github.com/llvm/llvm-project/pull/70338
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Vlad Serebrennikov
Date: 2024-01-20T12:23:08+04:00
New Revision: 9eb0f86c279f40a792ec27bf0e9b491b8c90a640
URL:
https://github.com/llvm/llvm-project/commit/9eb0f86c279f40a792ec27bf0e9b491b8c90a640
DIFF:
https://github.com/llvm/llvm-project/commit/9eb0f86c279f40a792ec27bf0e9b491b8c90a640.
https://github.com/Endilll closed
https://github.com/llvm/llvm-project/pull/78103
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tru wrote:
Thanks @owenca !
https://github.com/llvm/llvm-project/pull/78682
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/NorthBlue333 updated
https://github.com/llvm/llvm-project/pull/77456
>From 4dab4e9e082551a435b79f5d4647741f4b98168c Mon Sep 17 00:00:00 2001
From: NorthBlue333
Date: Tue, 9 Jan 2024 14:01:14 +0100
Subject: [PATCH] [clang-format] Do not update cursor pos if no includes
replac
https://github.com/Endilll created
https://github.com/llvm/llvm-project/pull/78836
This patch prevents tests for unresolved issues to report availability (e.g.
`no` or `18`) on `cxx_dr_status.html` page. But it still checks whether
specified status matches status on the official list, preventi
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Vlad Serebrennikov (Endilll)
Changes
This patch prevents tests for unresolved issues to report availability (e.g.
`no` or `18`) on `cxx_dr_status.html` page. But it still checks whether
specified status matches status on the official list
Author: Piotr Zegar
Date: 2024-01-20T10:11:05+01:00
New Revision: 1ad1f981a62213c1fc3145e8330f3c9b0dbe2b85
URL:
https://github.com/llvm/llvm-project/commit/1ad1f981a62213c1fc3145e8330f3c9b0dbe2b85
DIFF:
https://github.com/llvm/llvm-project/commit/1ad1f981a62213c1fc3145e8330f3c9b0dbe2b85.diff
L
https://github.com/PiotrZSL closed
https://github.com/llvm/llvm-project/pull/70595
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
NorthBlue333 wrote:
Oh, sorry @HazardyKnusperkeks I missed this in your previous comment.
I've updated the PR.
As a side note, I tried adding these two tests but they fail, meaning the
cursor is still incorrectly computed with CRLF when replacing lines.
This is due to adding lines between the
https://github.com/NorthBlue333 updated
https://github.com/llvm/llvm-project/pull/77456
>From 86d92ae648c3d0cee305bdf0bcaadbb19a7f5f1c Mon Sep 17 00:00:00 2001
From: NorthBlue333
Date: Tue, 9 Jan 2024 14:01:14 +0100
Subject: [PATCH] [clang-format] Do not update cursor pos if no includes
replac
https://github.com/NorthBlue333 updated
https://github.com/llvm/llvm-project/pull/77456
>From c3fa004487895acb96c7eeafbd27696c35f5148e Mon Sep 17 00:00:00 2001
From: NorthBlue333
Date: Tue, 9 Jan 2024 14:01:14 +0100
Subject: [PATCH] [clang-format] Do not update cursor pos if no includes
replac
https://github.com/NorthBlue333 updated
https://github.com/llvm/llvm-project/pull/77456
>From ac5dccff5c1922912e88f687d2685d547f4bded9 Mon Sep 17 00:00:00 2001
From: NorthBlue333
Date: Tue, 9 Jan 2024 14:01:14 +0100
Subject: [PATCH] [clang-format] Do not update cursor pos if no includes
replac
https://github.com/SihangZhu created
https://github.com/llvm/llvm-project/pull/78837
Storage-class specifiers like static are not the first things in a declaration.
According to the C Standard, this usage is obsolescent. This patch add a
diagnose to warn it.
This is a counterpart of gcc's Wswi
https://github.com/SihangZhu edited
https://github.com/llvm/llvm-project/pull/78837
___
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: None (SihangZhu)
Changes
Storage-class specifiers like static are not the first things in a declaration.
According to the C Standard, this usage is obsolescent. This patch add a
diagnose to warn it.
This is a counterpart of gcc's Wold-sty
https://github.com/SihangZhu edited
https://github.com/llvm/llvm-project/pull/78837
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -90,7 +90,7 @@ namespace dr2026 { // dr2026: 11
}
}
-namespace dr2049 { // dr2049: 18 drafting
+namespace dr2049 { // dr2049: drafting
cor3ntin wrote:
https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2023/p2308r1.html was
approved
https://github.com
@@ -59,7 +59,7 @@ f<{.a= 0}>();
#endif
}
-namespace dr2459 { // dr2459: 18 drafting
+namespace dr2459 { // dr2459: drafting
cor3ntin wrote:
https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2023/p2308r1.html was
approved
https://github.com/llvm/llvm-pro
@@ -83,7 +83,7 @@ using ::dr2521::operator""_div;
#if __cplusplus >= 202302L
-namespace dr2553 { // dr2553: 18 review
+namespace dr2553 { // dr2553: review
cor3ntin wrote:
All of the deducing this issue have agreement in core and i think we should
signal we
@@ -45,7 +45,7 @@ void fallthrough(int n) {
#endif
}
-namespace dr2450 { // dr2450: 18 review
+namespace dr2450 { // dr2450: review
cor3ntin wrote:
https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2023/p2308r1.html was
approved
https://github.com/llvm/ll
cor3ntin wrote:
I am slightly concerned with this direction.
A lot of these issues are actually fully resolved and just haven't made it to
an official list yet.
I would prefer that if we are fairly certain of the status of an issue we keep
a version number (and if we are not, then we don't)
h
https://github.com/PiotrZSL approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/77943
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
PiotrZSL wrote:
@BenBlaise Any plans to finish this ? Clang-tidy 18 branch-out is in ~3 days.
https://github.com/llvm/llvm-project/pull/76065
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
https://github.com/PiotrZSL updated
https://github.com/llvm/llvm-project/pull/73069
>From 286c4445f8cba6ea2f49fb9e8f732f04ebdb6c97 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?F=C3=A9lix-Antoine=20Constantin?=
Date: Thu, 16 Nov 2023 22:03:15 -0500
Subject: [PATCH] =?UTF-8?q?[clang-tidy]=C2=A0Added
https://github.com/PiotrZSL closed
https://github.com/llvm/llvm-project/pull/73069
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Félix-Antoine Constantin
Date: 2024-01-20T10:45:59+01:00
New Revision: 7a8f5d97afbff948fa2437c81d08a5963ee7d8cf
URL:
https://github.com/llvm/llvm-project/commit/7a8f5d97afbff948fa2437c81d08a5963ee7d8cf
DIFF:
https://github.com/llvm/llvm-project/commit/7a8f5d97afbff948fa2437c81d08a5963ee
PiotrZSL wrote:
@pizzud Just reminder that Clang-tidy 18 branch out is in ~3 days
https://github.com/llvm/llvm-project/pull/67467
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Endilll wrote:
@cor3ntin During our offline discussion with @AaronBallman, points "we don't
want to read tea leaves" and "CWG can change their opinion" were repeated so
often, that I thought we have consensus on using only official source on
information, i.e. issue list. I also don't see why i
https://github.com/rlanday created
https://github.com/llvm/llvm-project/pull/78838
None
>From bde3b1bba398f60cf4953ab33d1d832266d696c7 Mon Sep 17 00:00:00 2001
From: Ryan Landay
Date: Sat, 20 Jan 2024 18:10:19 +0800
Subject: [PATCH] Fix typo in AttrDocs.td (__single_inhertiance =>
__single_in
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 i
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Ryan Landay (rlanday)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/78838.diff
1 Files Affected:
- (modified) clang/include/clang/Basic/AttrDocs.td (+1-1)
``diff
diff --git a/clang/include/clang/Basic/Attr
https://github.com/cor3ntin approved this pull request.
Let's merge that and decided in #78836 what to do for the dr status page
https://github.com/llvm/llvm-project/pull/67948
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.
https://github.com/ChipsSpectre created
https://github.com/llvm/llvm-project/pull/78840
…a nullptr casted to an array of non-constant size to a reference (#76634).
This situation is undefined behavior, and should not lead to a compiler crash.
Thus, the problematic cast is only executed on non-
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (ChipsSpectre)
Changes
…a nullptr casted to an array of non-constant size to a reference (#76634).
This situation is undefined behavior, and should not lead to a compiler crash.
Thus, the problematic cast is only executed on non-null
https://github.com/ChipsSpectre converted_to_draft
https://github.com/llvm/llvm-project/pull/78840
___
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 a7d7da6e45992b79fe712c1e228cc57c9f27fa7a
2503669a55f8dae534440a4eacb66a4500f78e3f --
https://github.com/cor3ntin requested changes to this pull request.
Getting rid of yes on issues that have been resolved but not yet published or
are in a similar high-confidence scenario removes useful information and I
really would like us not to do that.
For unresolved issues currently mark
https://github.com/unterumarmung created
https://github.com/llvm/llvm-project/pull/78842
The `.yml` file extension is a valid extension for the YAML files, but it was
not previously supported by the Clang Apply Replacements tool. This commit adds
support for processing `.yml` files. Without th
https://github.com/unterumarmung updated
https://github.com/llvm/llvm-project/pull/78842
>From 14767cd17ca81ceded59b630968bfd5f2dfab26b Mon Sep 17 00:00:00 2001
From: Daniil Dudkin
Date: Sat, 20 Jan 2024 14:07:40 +0300
Subject: [PATCH] [clang-apply-replacements] Add support for the `.yml` file
https://github.com/PiotrZSL approved this pull request.
LGTM, you may want to wait like 1-2 day before merging so if someone else would
have some comments.
https://github.com/llvm/llvm-project/pull/78842
___
cfe-commits mailing list
cfe-commits@lists.
https://github.com/vgvassilev created
https://github.com/llvm/llvm-project/pull/78843
This test demonstrates template instantiation via the interpreter code. In
order to do that we can allocate the object on the stack and extend its
lifetime by boxing it into a clang::Value.
That avoids the s
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Vassil Vassilev (vgvassilev)
Changes
This test demonstrates template instantiation via the interpreter code. In
order to do that we can allocate the object on the stack and extend its
lifetime by boxing it into a clang::Value.
That avoid
vgvassilev wrote:
Yes, you are right, I've put a fix here
https://github.com/llvm/llvm-project/pull/78843. Can you take a look?
https://github.com/llvm/llvm-project/pull/76218
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.
https://github.com/kmclaughlin-arm updated
https://github.com/llvm/llvm-project/pull/78321
>From 11dce217ed307601d0ea1eb5b016b47f80e67786 Mon Sep 17 00:00:00 2001
From: Kerry McLaughlin
Date: Thu, 11 Jan 2024 17:46:00 +
Subject: [PATCH 1/7] [SME2][Clang] Add tests with ZT0 state
---
llvm/
@@ -4833,9 +4833,26 @@ void CXXNameMangler::mangleExpression(const Expr *E,
unsigned Arity,
E = cast(E)->getSubExpr();
goto recurse;
- case Expr::SubstNonTypeTemplateParmExprClass:
+ case Expr::SubstNonTypeTemplateParmExprClass: {
+// Mangle a substituted parame
@@ -6472,7 +6494,20 @@ void CXXNameMangler::mangleValueInTemplateArg(QualType
T, const APValue &V,
Out << "plcvPcad";
Kind = Offset;
} else {
- if (!V.getLValuePath().empty() || V.isLValueOnePastTheEnd()) {
+ // Clang 11 and before mangled an array su
https://github.com/bolshakov-a updated
https://github.com/llvm/llvm-project/pull/78041
>From 9bef9d5dd970f1af653d5b10d0a062d96a69ad01 Mon Sep 17 00:00:00 2001
From: Andrey Ali Khan Bolshakov
Date: Sun, 6 Aug 2023 19:38:23 +0300
Subject: [PATCH] [c++20] P1907R1: Support for generalized non-type
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 8e8bbbd48ee23764cc6030fd259dde6a1107c00c
9bef9d5dd970f1af653d5b10d0a062d96a69ad01 --
Author: Congcong Cai
Date: 2024-01-20T20:05:22+08:00
New Revision: fd3346dba825f6b9c2873bdeafe34da8f8b4f3e1
URL:
https://github.com/llvm/llvm-project/commit/fd3346dba825f6b9c2873bdeafe34da8f8b4f3e1
DIFF:
https://github.com/llvm/llvm-project/commit/fd3346dba825f6b9c2873bdeafe34da8f8b4f3e1.diff
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/77943
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kmclaughlin-arm closed
https://github.com/llvm/llvm-project/pull/78321
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/78763
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin commented:
I think this requires more documentation + a changelog entry.
(the documentation can link to a libc++ page if such thing exist, but there
needs to be an explanation of the values _somewhere_. Maybe even in the help
message too.
We probably do not want user
@@ -275,6 +275,8 @@ def warn_drv_unknown_argument_clang_cl_with_suggestion :
Warning<
InGroup;
def err_drv_unknown_target_triple : Error<"unknown target triple '%0'">;
+def err_drv_stdlib_hardening_unavailable : Error<"standard library hardening
is available only when libc
@@ -7730,6 +7730,14 @@ def source_date_epoch : Separate<["-"],
"source-date-epoch">,
} // let Visibility = [CC1Option]
+def stdlib_hardening_EQ : Joined<["-"], "fstdlib-hardening=">,
+ Values<"none,fast,extensive,debug">,
+ NormalizedValues<["STDLIB_HARDENING_MODE_NONE",
@@ -275,6 +275,8 @@ def warn_drv_unknown_argument_clang_cl_with_suggestion :
Warning<
InGroup;
def err_drv_unknown_target_triple : Error<"unknown target triple '%0'">;
+def err_drv_stdlib_hardening_unavailable : Error<"libc++ hardening is
available only when libc++ is used
https://github.com/hnakamura5 closed
https://github.com/llvm/llvm-project/pull/78571
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Hirofumi Nakamura
Date: 2024-01-20T21:15:58+09:00
New Revision: fcb6737f82246c6046526f699c9a82a96f71ab55
URL:
https://github.com/llvm/llvm-project/commit/fcb6737f82246c6046526f699c9a82a96f71ab55
DIFF:
https://github.com/llvm/llvm-project/commit/fcb6737f82246c6046526f699c9a82a96f71ab55.d
https://github.com/tgross35 updated
https://github.com/llvm/llvm-project/pull/76558
>From 90a465d0a7e9744a4a8043152016e500927a0d95 Mon Sep 17 00:00:00 2001
From: Trevor Gross
Date: Fri, 11 Aug 2023 22:16:01 -0400
Subject: [PATCH 1/4] [IR] Add an xpassing test for `f128` intrinsic lowering
`f12
https://github.com/tgross35 updated
https://github.com/llvm/llvm-project/pull/76558
>From e869ad1bc601d95b6364dc5619e79a06e8b0fc82 Mon Sep 17 00:00:00 2001
From: Trevor Gross
Date: Fri, 11 Aug 2023 22:16:01 -0400
Subject: [PATCH 1/4] [IR] Add an xpassing test for `f128` intrinsic lowering
`f12
https://github.com/tgross35 updated
https://github.com/llvm/llvm-project/pull/76558
>From e869ad1bc601d95b6364dc5619e79a06e8b0fc82 Mon Sep 17 00:00:00 2001
From: Trevor Gross
Date: Fri, 11 Aug 2023 22:16:01 -0400
Subject: [PATCH 1/4] [IR] Add an xpassing test for `f128` intrinsic lowering
`f12
https://github.com/tgross35 updated
https://github.com/llvm/llvm-project/pull/76558
>From e869ad1bc601d95b6364dc5619e79a06e8b0fc82 Mon Sep 17 00:00:00 2001
From: Trevor Gross
Date: Fri, 11 Aug 2023 22:16:01 -0400
Subject: [PATCH 1/4] [IR] Add an xpassing test for `f128` intrinsic lowering
`f12
https://github.com/bolshakov-a updated
https://github.com/llvm/llvm-project/pull/78041
>From 35e5e87d5c6f250f9f7a6cf5e5aeaebe5929f155 Mon Sep 17 00:00:00 2001
From: Andrey Ali Khan Bolshakov
Date: Sun, 6 Aug 2023 19:38:23 +0300
Subject: [PATCH] [c++20] P1907R1: Support for generalized non-type
https://github.com/bolshakov-a updated
https://github.com/llvm/llvm-project/pull/78041
>From 7abb30826263ad69d529dd6af585bf3bce40f979 Mon Sep 17 00:00:00 2001
From: Andrey Ali Khan Bolshakov
Date: Sun, 6 Aug 2023 19:38:23 +0300
Subject: [PATCH] [c++20] P1907R1: Support for generalized non-type
https://github.com/jplehr updated
https://github.com/llvm/llvm-project/pull/76571
>From a6c437a52674613b90c451c2ed4105265f420a32 Mon Sep 17 00:00:00 2001
From: JP Lehr
Date: Fri, 29 Dec 2023 04:32:24 -0500
Subject: [PATCH] [OpenMP][USM] Introduces -fopenmp-force-usm flag
This flag forces the c
https://github.com/hnakamura5 created
https://github.com/llvm/llvm-project/pull/78846
Make TableGen's statements to be parsed considering their structure.
- Removed label
- Avoid class from being parsed as c++'s class
- Support if statement of the form `if then { ... }`
- Support defset statem
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Hirofumi Nakamura (hnakamura5)
Changes
Make TableGen's statements to be parsed considering their structure.
- Removed label
- Avoid class from being parsed as c++'s class
- Support if statement of the form `if then { ... }`
-
https://github.com/hnakamura5 edited
https://github.com/llvm/llvm-project/pull/78846
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tru approved this pull request.
https://github.com/llvm/llvm-project/pull/78823
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hnakamura5 wrote:
Numeric like identifiers part https://github.com/llvm/llvm-project/pull/78571
https://github.com/llvm/llvm-project/pull/76059
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-
hnakamura5 wrote:
unwrapped line parser for statements part
https://github.com/llvm/llvm-project/pull/78846 .
https://github.com/llvm/llvm-project/pull/76059
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
https://github.com/rymiel created
https://github.com/llvm/llvm-project/pull/78847
If clang-format is not sure whether a `requires` keyword starts a requires
clause or a requires expression, it looks ahead to see if any token
disqualifies it from being a requires clause. Among these tokens was
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Emilia Kond (rymiel)
Changes
If clang-format is not sure whether a `requires` keyword starts a requires
clause or a requires expression, it looks ahead to see if any token
disqualifies it from being a requires clause. Among these t
@@ -1071,6 +1071,16 @@ TEST_F(TokenAnnotatorTest,
UnderstandsRequiresClausesAndConcepts) {
"concept C = (!Foo) && Bar;");
rymiel wrote:
So, GitHub won't actually let me leave a comment on the line I want to actually
talk about, but I notice
https://github.com/AaronBallman approved this pull request.
LGTM! Let me know if you need someone to land this on your behalf.
https://github.com/llvm/llvm-project/pull/78838
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.or
rlanday wrote:
Yes please!
https://github.com/llvm/llvm-project/pull/78838
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/78793
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
AaronBallman wrote:
> > I had an offline discussion with @Endilll during my morning office hours
> > today, and our current plan is:
> >
> > 1. Remove `Implicit` from the enumeration, rename `Call` and `List` to
> > `ParenList` and `BraceList`, respectively
> > 2. Add a new bit to the bit-fiel
Author: Ryan Landay
Date: 2024-01-20T09:20:22-05:00
New Revision: d01145f7607432586faa771d6519196cd27458df
URL:
https://github.com/llvm/llvm-project/commit/d01145f7607432586faa771d6519196cd27458df
DIFF:
https://github.com/llvm/llvm-project/commit/d01145f7607432586faa771d6519196cd27458df.diff
L
AaronBallman wrote:
Thank you for the fix!
https://github.com/llvm/llvm-project/pull/78838
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/78838
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Congcong Cai
Date: 2024-01-20T22:43:15+08:00
New Revision: 0f80f5e362fb43a9335bd154c5f7976a96e32cfc
URL:
https://github.com/llvm/llvm-project/commit/0f80f5e362fb43a9335bd154c5f7976a96e32cfc
DIFF:
https://github.com/llvm/llvm-project/commit/0f80f5e362fb43a9335bd154c5f7976a96e32cfc.diff
https://github.com/mhaehnel approved this pull request.
LGTM
Disclaimer: I am a co-worker of the change author and discussed the change also
offline.
https://github.com/llvm/llvm-project/pull/76950
___
cfe-commits mailing list
cfe-commits@lists.llvm.
hahnjo wrote:
> It seems after this change we started to fail on `LLVM ::
> ExecutionEngine/JITLink/RISCV/ELF_ehframe.s` when using debug libgcxx.
First, sorry for the slow response and breaking the builds with expensive
checks in the first place. I believe the report is actually fully correct
https://github.com/rymiel created
https://github.com/llvm/llvm-project/pull/78852
Simply always return false if tok::ampamp is passed into
determineUnaryOperatorByUsage.
I've done a fix earlier regarding ampamp becoming an unary operator, this would
technically supersede that.
I'm not sure w
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Emilia Kond (rymiel)
Changes
Simply always return false if tok::ampamp is passed into
determineUnaryOperatorByUsage.
I've done a fix earlier regarding ampamp becoming an unary operator, this would
technically supersede that.
I'm
kelbon wrote:
> LGTM, thank you!
What's next? I cannot push changes
https://github.com/llvm/llvm-project/pull/78200
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cpplearner wrote:
Unary `&&` is a GCC extension to get the address of a label. See
https://gcc.gnu.org/onlinedocs/gcc/Labels-as-Values.html
Does this PR change the formatting of reasonable code that uses unary `&&`,
e.g. `void *ptr; /* … */ ptr = &&foo;` in a function body?
https://github.com
rymiel wrote:
clang-format does not support that extension in the first place, there's an
open issue for it. But I will try a few edge cases to see if it changes
anything or if it was already broken
https://github.com/llvm/llvm-project/pull/78852
___
@@ -2459,6 +2459,52 @@ struct FormatStyle {
/// \version 12
EmptyLineBeforeAccessModifierStyle EmptyLineBeforeAccessModifier;
+ /// \brief Number of lines after includes.
+ /// If set, determines the number of lines to insert after includes.
+ /// Limited by MaxEmptyLin
H-G-Hristov wrote:
This FreeBSD failure seems unrelated:
```
# .---command stderr
# | In file included from
/usr/home/buildkite/.buildkite-agent/builds/freebsd-test-1/llvm-project/libcxx-ci/libcxx/test/std/algorithms/numeric.ops/transform.reduce/pstl.exception_handling.pass.cpp:20:
#
seranu wrote:
> I think splitting the two features would be better.
I'm going to close this MR and create two new ones.
> Also please add a note to the changelog.
Where can I find the changelog?
https://github.com/llvm/llvm-project/pull/77918
___
H-G-Hristov wrote:
This FreeBSD failure seems unrelated, the same appears in `variant`:
```
# .---command stderr
# | In file included from
/usr/home/buildkite/.buildkite-agent/builds/freebsd-test-1/llvm-project/libcxx-ci/libcxx/test/libcxx/assertions/modes/override_with_fast_mode.pa
https://github.com/fhahn updated https://github.com/llvm/llvm-project/pull/73158
>From 13a26e8e7440c3b501730b22588af393a3e543cd Mon Sep 17 00:00:00 2001
From: Florian Hahn
Date: Thu, 6 Jul 2023 08:07:45 +0100
Subject: [PATCH] [VPlan] Implement cloning of VPlans.
This patch implements cloning fo
@@ -0,0 +1,139 @@
+//===--- UseStdMinMaxCheck.cpp - clang-tidy
---===//
+//
+// 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: Ap
11happy wrote:
> Sorry for making this so iterative ^^ but I think that after these everything
> is good from my side.
No Problem! actually I am quite happy to contribute to the org and am able to
learn something.
https://github.com/llvm/llvm-project/pull/77816
___
Author: kelbon
Date: 2024-01-20T12:37:35-05:00
New Revision: 818de32f31e8075657dd27938e4aeb1a46f3f631
URL:
https://github.com/llvm/llvm-project/commit/818de32f31e8075657dd27938e4aeb1a46f3f631
DIFF:
https://github.com/llvm/llvm-project/commit/818de32f31e8075657dd27938e4aeb1a46f3f631.diff
LOG: W
1 - 100 of 253 matches
Mail list logo
