https://github.com/martinboehme created
https://github.com/llvm/llvm-project/pull/70437
I'm working on a patch that will use this function from a different test.
>From 34daee21bca7dbfe906e270a88e829e6beb79c97 Mon Sep 17 00:00:00 2001
From: Martin Braenne
Date: Fri, 27 Oct 2023 09:27:12 +
Author: Owen Pan
Date: 2023-10-27T02:22:36-07:00
New Revision: 6c3bc910588f962e49470098ccc3b13c29cae493
URL:
https://github.com/llvm/llvm-project/commit/6c3bc910588f962e49470098ccc3b13c29cae493
DIFF:
https://github.com/llvm/llvm-project/commit/6c3bc910588f962e49470098ccc3b13c29cae493.diff
https://github.com/khei4 updated https://github.com/llvm/llvm-project/pull/70427
>From 1ae7edddc79a0e96fd4b142d09b0752aa3d9ff85 Mon Sep 17 00:00:00 2001
From: khei4
Date: Fri, 27 Oct 2023 17:46:34 +0900
Subject: [PATCH] use: create instead of protected Constructor
handle: expected
---
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Brad Smith (brad0)
Changes
Same as 12b87f6ef720080fab1e2d48ca2d8c5ba478ee5d
---
Full diff: https://github.com/llvm/llvm-project/pull/70434.diff
5 Files Affected:
- (modified) clang/lib/Driver/ToolChains/DragonFly.cpp (+3)
- (modified)
https://github.com/brad0 created https://github.com/llvm/llvm-project/pull/70434
Same as 12b87f6ef720080fab1e2d48ca2d8c5ba478ee5d
>From 0c21090272eb3e1d6477585ef223413dc627a451 Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Fri, 27 Oct 2023 05:13:59 -0400
Subject: [PATCH] [Driver] Silence
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/70432
It can suppression further crash.
Fixes: #67687
>From c12aeb6f2a83b9cb3c3815e72d57638cf7de43a0 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Fri, 27 Oct 2023 16:51:59 +0800
Subject: [PATCH] WIP fix
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/68413
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
philnik777 wrote:
Why does the proxy not satisfy `weakly_incrementable`? Is it maybe just missing
some member functions?
https://github.com/llvm/llvm-project/pull/68494
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
philnik777 wrote:
Why does the proxy not satisfy `weakly_incrementable`? Is it maybe just missing
some member functions?
https://github.com/llvm/llvm-project/pull/68494
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/khei4 converted_to_draft
https://github.com/llvm/llvm-project/pull/70427
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/brad0 created https://github.com/llvm/llvm-project/pull/70429
None
>From d8a4aecf5a22382fe57f654616fa59358477d9a4 Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Fri, 27 Oct 2023 04:53:19 -0400
Subject: [PATCH] [Driver] Handle Flang in same manner between Gnu and
owenca wrote:
IMO, the new behavior is consistent with the
[documentation](https://clang.llvm.org/docs/ClangFormatStyleOptions.html#columnlimit)
for `ColumnLimt: 0`.
https://github.com/llvm/llvm-project/pull/69871
___
cfe-commits mailing list
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Kohei Asano (khei4)
Changes
This patch fixes the code example on CommonOptionParser on
https://intel.github.io/llvm-docs/clang/LibTooling.html
CommonOptionParser's constructor is protected and we can use
`CommonOptionParser::create`
https://github.com/khei4 created https://github.com/llvm/llvm-project/pull/70427
This patch fixes the code example on CommonOptionParser on
https://intel.github.io/llvm-docs/clang/LibTooling.html
CommonOptionParser's constructor is protected and we can use
`CommonOptionParser::create` instead
Author: Chuanqi Xu
Date: 2023-10-27T16:52:21+08:00
New Revision: e9a7876c2c81423f2289aa85eeac32d7a55cd3c8
URL:
https://github.com/llvm/llvm-project/commit/e9a7876c2c81423f2289aa85eeac32d7a55cd3c8
DIFF:
https://github.com/llvm/llvm-project/commit/e9a7876c2c81423f2289aa85eeac32d7a55cd3c8.diff
Author: Timm Bäder
Date: 2023-10-27T10:43:35+02:00
New Revision: e64e4784ee1c3ed9bb06458e9c427542927224ab
URL:
https://github.com/llvm/llvm-project/commit/e64e4784ee1c3ed9bb06458e9c427542927224ab
DIFF:
https://github.com/llvm/llvm-project/commit/e64e4784ee1c3ed9bb06458e9c427542927224ab.diff
Author: Timm Bäder
Date: 2023-10-27T10:43:35+02:00
New Revision: 051fade10f03515b0980c58541fe4c28220e0e67
URL:
https://github.com/llvm/llvm-project/commit/051fade10f03515b0980c58541fe4c28220e0e67
DIFF:
https://github.com/llvm/llvm-project/commit/051fade10f03515b0980c58541fe4c28220e0e67.diff
https://github.com/brad0 closed https://github.com/llvm/llvm-project/pull/69817
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Brad Smith
Date: 2023-10-27T04:43:19-04:00
New Revision: c149ff3d372a70e7d9de838a1ff0357394b8d017
URL:
https://github.com/llvm/llvm-project/commit/c149ff3d372a70e7d9de838a1ff0357394b8d017
DIFF:
https://github.com/llvm/llvm-project/commit/c149ff3d372a70e7d9de838a1ff0357394b8d017.diff
https://github.com/balazske commented:
The fix looks good but the formatting of the test code could be better, like in
the other tests.
https://github.com/llvm/llvm-project/pull/68774
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -1373,6 +1373,40 @@ TEST_P(ASTImporterOptionSpecificTestBase,
ImportCorrectTemplatedDecl) {
ASSERT_EQ(ToTemplated1, ToTemplated);
}
+TEST_P(ASTImporterOptionSpecificTestBase,
+ ImportTemplateSpecializationStaticMember) {
+ auto FromCode = R"(
+ template
https://github.com/balazske edited
https://github.com/llvm/llvm-project/pull/68774
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zixuan-wu accepted this revision.
zixuan-wu added a comment.
This revision is now accepted and ready to land.
LGTM if nobody objects.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70401/new/
https://reviews.llvm.org/D70401
hctim wrote:
I talked with some folks internally and we came to the consensus that this'll
almost certainly break some debugging tools and such, it probably won't effect
the runtime of binaries, but I wouldn't say that this is a super
confidence-inspiring thing to do.
> AMD language runtimes
hctim wrote:
(friendly ping @eugenis / @vitalybuka)
https://github.com/llvm/llvm-project/pull/70186
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -49,7 +49,7 @@ struct ExprDependenceScope {
using ExprDependence = ExprDependenceScope::ExprDependence;
struct TypeDependenceScope {
- enum TypeDependence : uint8_t {
+ enum TypeDependence : unsigned {
cor3ntin wrote:
@erichkeane @AaronBallman wdyt?
Timm =?utf-8?q?Bäder?= ,
Timm =?utf-8?q?Bäder?= ,
Timm =?utf-8?q?Bäder?= ,
Timm =?utf-8?q?Bäder?= ,
Timm =?utf-8?q?Bäder?= ,
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,816 @@
+// RUN: %clang_cc1 -verify -std=c++2a -fsyntax-only
@@ -49,7 +49,7 @@ struct ExprDependenceScope {
using ExprDependence = ExprDependenceScope::ExprDependence;
struct TypeDependenceScope {
- enum TypeDependence : uint8_t {
+ enum TypeDependence : unsigned {
cor3ntin wrote:
I had a chat with @Endilll offline.
https://github.com/banach-space approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/69817
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Endilll wrote:
That's a good point. I'm not opposed to wrap this attribute if we must. I'd
like to hear from @AaronBallman on this matter.
https://github.com/llvm/llvm-project/pull/70349
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/brad0 created https://github.com/llvm/llvm-project/pull/70416
None
>From 6247344956c34137ced2ae89be1154bd31173d46 Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Fri, 27 Oct 2023 02:22:21 -0400
Subject: [PATCH] [Driver][NFC] Make use of final
---
tbaederr wrote:
Sure but clang 16 will emit a warning for them: https://godbolt.org/z/jnsc4336G
- are we passing `-Wno-unknown-attributes` to the build?
https://github.com/llvm/llvm-project/pull/70349
___
cfe-commits mailing list
Endilll wrote:
@tbaederr As highlighted in https://github.com/llvm/llvm-project/pull/69104,
this attribute doesn't change semantics of the program, so it's safe to ignore.
https://github.com/llvm/llvm-project/pull/70349
___
cfe-commits mailing list
tbaederr wrote:
Since this is done unconditionally, doesn't this mean we now need an unreleased
clang to compile? Or are `[[clang::]]` attributes just ignored if they are
unknown?
https://github.com/llvm/llvm-project/pull/70349
___
cfe-commits
@@ -20,7 +20,7 @@ namespace clang {
/// Describes the different kinds of linkage
/// (C++ [basic.link], C99 6.2.2) that an entity may have.
-enum Linkage : unsigned char {
+enum Linkage : unsigned {
Endilll wrote:
This case is similar to `TypeDependence` we
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/68297
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -751,13 +751,8 @@ class Preprocessor {
std::unique_ptr CurTokenLexer;
/// The kind of lexer we're currently working with.
- enum CurLexerKind {
-CLK_Lexer,
-CLK_TokenLexer,
-CLK_CachingLexer,
-CLK_DependencyDirectivesLexer,
-
https://github.com/cor3ntin commented:
This is excellent!
I made some naming suggestion. Now that we don't have an enum i think it is
clearer to drop the "kind" terminology
https://github.com/llvm/llvm-project/pull/70381
___
cfe-commits mailing list
@@ -643,23 +643,7 @@ void Preprocessor::SkipTokensWhileUsingPCH() {
while (true) {
bool InPredefines =
(CurLexer && CurLexer->getFileID() == getPredefinesFileID());
-switch (CurLexerKind) {
-case CLK_Lexer:
- CurLexer->Lex(Tok);
- break;
-
@@ -751,13 +751,8 @@ class Preprocessor {
std::unique_ptr CurTokenLexer;
/// The kind of lexer we're currently working with.
- enum CurLexerKind {
-CLK_Lexer,
-CLK_TokenLexer,
-CLK_CachingLexer,
-CLK_DependencyDirectivesLexer,
-
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/70381
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
301 - 341 of 341 matches
Mail list logo