https://github.com/zixuan-wu approved this pull request.
LGTM. But I think commit msg should be refined(this PR is not for upstreaming?)
https://github.com/llvm/llvm-project/pull/115961
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
zixuan-wu wrote:
LGTM. But I think commit msg should be refined(this PR is not for upstreaming?)
https://github.com/llvm/llvm-project/pull/115961
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/c
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `premerge-monolithic-linux`
running on `premerge-linux-1` while building `clang` at step 7
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/153/builds/22321
Here is
https://github.com/rniwa updated
https://github.com/llvm/llvm-project/pull/126353
>From b3d79dd62e88afa0e83101165ea878f12176e0f2 Mon Sep 17 00:00:00 2001
From: Ryosuke Niwa
Date: Fri, 7 Feb 2025 23:21:06 -0800
Subject: [PATCH 1/2] [WebKit Checkers] Treat const Objective-C ivar as a safe
origin
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Ryosuke Niwa (rniwa)
Changes
Like const C++ member variables, treat const Ref, RefPtr, CheckedRef,
CheckedPtr Objective-C ivars as a safe pointer origin in WebKit checkers.
---
Full diff: https://github.com/llvm/llvm-project/pull/126353.d
https://github.com/rniwa created
https://github.com/llvm/llvm-project/pull/126353
Like const C++ member variables, treat const Ref, RefPtr, CheckedRef,
CheckedPtr Objective-C ivars as a safe pointer origin in WebKit checkers.
>From b3d79dd62e88afa0e83101165ea878f12176e0f2 Mon Sep 17 00:00:00 2
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`clang-cmake-x86_64-avx512-win` running on `avx512-intel64-win` while building
`clang` at step 4 "cmake stage 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/81/builds/4525
Here is the relevant pi
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`sanitizer-aarch64-linux-bootstrap-hwasan` running on `sanitizer-buildbot11`
while building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/55/builds/6756
Here is the rel
sakria9 wrote:
@zyn0217 Hi, I have 1) modified the release note 2) moved the test 3) supported
dump structural value in `JSONNodeDumper`.
Please take a look.
https://github.com/llvm/llvm-project/pull/126341
___
cfe-commits mailing list
cfe-commits@lis
@@ -0,0 +1,12 @@
+// RUN: %clang_cc1 -std=c++20 -ast-dump -ast-dump-filter=pr126341 %s |
FileCheck %s
sakria9 wrote:
Moved.
https://github.com/llvm/llvm-project/pull/126341
___
cfe-commits mailing list
cfe-commits@lis
https://github.com/sakria9 updated
https://github.com/llvm/llvm-project/pull/126341
>From a382aa9de206a194397f9424150f170d35101b42 Mon Sep 17 00:00:00 2001
From: sakria9
Date: Sat, 8 Feb 2025 04:00:08 +
Subject: [PATCH 1/8] [ASTDump] Add support for structural value template
arguments in T
https://github.com/asb edited https://github.com/llvm/llvm-project/pull/126352
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-libcxx
Author: Alex Bradbury (asb)
Changes
By far the most important part of this patch is updating GettingInvolved.rst to
include the invite link, but I've grepped for any other discord.com links.
I'm no Discord expert, but from my experience (confirm
llvmbot wrote:
@llvm/pr-subscribers-libc
Author: Alex Bradbury (asb)
Changes
By far the most important part of this patch is updating GettingInvolved.rst to
include the invite link, but I've grepped for any other discord.com links.
I'm no Discord expert, but from my experience (confirmed
https://github.com/asb created https://github.com/llvm/llvm-project/pull/126352
By far the most important part of this patch is updating GettingInvolved.rst to
include the invite link, but I've grepped for any other discord.com links.
I'm no Discord expert, but from my experience (confirmed via
https://github.com/zyn0217 closed
https://github.com/llvm/llvm-project/pull/124793
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Yanzuo Liu
Date: 2025-02-08T14:31:47+08:00
New Revision: 6f241e36831927e3aea113cfc017c34fdeda340a
URL:
https://github.com/llvm/llvm-project/commit/6f241e36831927e3aea113cfc017c34fdeda340a
DIFF:
https://github.com/llvm/llvm-project/commit/6f241e36831927e3aea113cfc017c34fdeda340a.diff
LO
https://github.com/zwuis updated
https://github.com/llvm/llvm-project/pull/124793
>From 16596add29b63ee0282e026dec7b1d5946863113 Mon Sep 17 00:00:00 2001
From: Yanzuo Liu
Date: Wed, 29 Jan 2025 00:38:15 +0800
Subject: [PATCH 1/4] Fix wrong initialization kind
---
clang/lib/Sema/SemaInit.cpp
https://github.com/sakria9 updated
https://github.com/llvm/llvm-project/pull/126341
>From a382aa9de206a194397f9424150f170d35101b42 Mon Sep 17 00:00:00 2001
From: sakria9
Date: Sat, 8 Feb 2025 04:00:08 +
Subject: [PATCH 1/6] [ASTDump] Add support for structural value template
arguments in T
@@ -54,6 +54,8 @@ ABI Changes in This Version
AST Dumping Potentially Breaking Changes
+- Added support for dumping structural value template arguments in AST dumps.
(#GH126341)
zyn0217 wrote:
We don't usually referen
@@ -0,0 +1,12 @@
+// RUN: %clang_cc1 -std=c++20 -ast-dump -ast-dump-filter=pr126341 %s |
FileCheck %s
zyn0217 wrote:
Can you consolidate the test into a pre-existing file e.g.
`AST/ast-dump-templates.cpp`?
https://github.com/llvm/llvm-project/pull/126341
_
https://github.com/owenca closed
https://github.com/llvm/llvm-project/pull/112482
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Gedare Bloom
Date: 2025-02-07T22:10:35-08:00
New Revision: e0a21e23a7aa6acf3e07b866c3c599db5eb4b67f
URL:
https://github.com/llvm/llvm-project/commit/e0a21e23a7aa6acf3e07b866c3c599db5eb4b67f
DIFF:
https://github.com/llvm/llvm-project/commit/e0a21e23a7aa6acf3e07b866c3c599db5eb4b67f.diff
zyn0217 wrote:
@zwuis Can you resolve the conflicts before we help you merge it? thanks
https://github.com/llvm/llvm-project/pull/124793
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sakria9 wrote:
@zyn0217 Hi, I have added a test and modfied the release note. Please take a
look.
Test log:
```
$ ./bin/llvm-lit
../clang/test/AST/ast-dump-template-argument-structural-value.cpp
llvm-lit: /workspaces/clice/deps/llvm/llvm/utils/lit/lit/llvm/config.py:506:
note: using clang:
https://github.com/sakria9 updated
https://github.com/llvm/llvm-project/pull/126341
>From a382aa9de206a194397f9424150f170d35101b42 Mon Sep 17 00:00:00 2001
From: sakria9
Date: Sat, 8 Feb 2025 04:00:08 +
Subject: [PATCH 1/5] [ASTDump] Add support for structural value template
arguments in T
@@ -1909,7 +1909,22 @@ class Sema final : public SemaBase {
/// '\#pragma clang attribute push' directives to the given declaration.
void AddPragmaAttributes(Scope *S, Decl *D);
- void PrintPragmaAttributeInstantiationPoint();
+ using DiagFuncRef =
+ llvm::function_
https://github.com/sakria9 updated
https://github.com/llvm/llvm-project/pull/126341
>From a382aa9de206a194397f9424150f170d35101b42 Mon Sep 17 00:00:00 2001
From: sakria9
Date: Sat, 8 Feb 2025 04:00:08 +
Subject: [PATCH 1/3] [ASTDump] Add support for structural value template
arguments in T
https://github.com/mpark updated
https://github.com/llvm/llvm-project/pull/126349
>From eac633d78359245f1ce478b5f3cff26c6f5c858f Mon Sep 17 00:00:00 2001
From: Michael Park
Date: Fri, 7 Feb 2025 21:23:26 -0800
Subject: [PATCH] [C++20][Modules][Serialization] Add an additional test case
for #12
https://github.com/owenca updated
https://github.com/llvm/llvm-project/pull/112482
>From 8282f754ae40b64a88e2d22a6cb21e7028da8371 Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Tue, 15 Oct 2024 23:55:49 -0600
Subject: [PATCH 1/8] [clang-format] add BinPackLongBracedList style option
The use
@@ -1909,7 +1909,19 @@ class Sema final : public SemaBase {
/// '\#pragma clang attribute push' directives to the given declaration.
void AddPragmaAttributes(Scope *S, Decl *D);
- void PrintPragmaAttributeInstantiationPoint();
+ using DiagFuncRef =
zyn02
@@ -1654,11 +1654,20 @@ void Sema::EmitDiagnostic(unsigned DiagID, const
DiagnosticBuilder &DB) {
}
case DiagnosticIDs::SFINAE_Suppress:
+ if (DiagnosticsEngine::Level Level = getDiagnostics().getDiagnosticLevel(
+ DiagInfo.getID(), DiagInfo.getLocat
https://github.com/ehan701 approved this pull request.
https://github.com/llvm/llvm-project/pull/124353
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/112482
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca edited
https://github.com/llvm/llvm-project/pull/112482
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/lalaniket8 updated
https://github.com/llvm/llvm-project/pull/125646
>From 7be637fa9fcde8977f650e208c7ddc1495080941 Mon Sep 17 00:00:00 2001
From: anikelal
Date: Tue, 4 Feb 2025 12:13:20 +0530
Subject: [PATCH 1/2] [Driver][HIP] Do not pass -dependency-file flag for HIP
Device
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/124353
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,21 @@
+// 1) test on platforms that (do or do not) require runtime relocation
+//
MaskRay wrote:
sed 's,^//$,,'
https://github.com/llvm/llvm-project/pull/124353
___
cfe-commits mailing list
cfe-commits@lists
llvmbot wrote:
@llvm/pr-subscribers-clang-modules
Author: Michael Park (mpark)
Changes
https://github.com/llvm/llvm-project/commit/4b35dd57b88a59b169c3471cbc398113d3bf98e8
was shipped to address https://github.com/llvm/llvm-project/issues/120277 .
It was thought to be a regression in 19.
https://github.com/mpark created
https://github.com/llvm/llvm-project/pull/126349
https://github.com/llvm/llvm-project/commit/4b35dd57b88a59b169c3471cbc398113d3bf98e8
was shipped to address https://github.com/llvm/llvm-project/issues/120277 .
It was thought to be a regression in 19.x according
https://github.com/damyanp edited
https://github.com/llvm/llvm-project/pull/122981
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/damyanp commented:
LGTM - I'm surprised we don't have all of the tokens we're going to need for
root signatures included in this PR.
https://github.com/llvm/llvm-project/pull/122981
___
cfe-commits mailing list
cfe-commits@lists.llv
@@ -0,0 +1,171 @@
+#include "clang/Parse/ParseHLSLRootSignature.h"
+
+namespace clang {
+namespace hlsl {
+
+// Lexer Definitions
+
+static bool IsNumberChar(char C) {
+ // TODO(#120472): extend for float support exponents
damyanp wrote:
The issue linked here (#
https://github.com/zyn0217 commented:
Thanks for the patch. This needs a test and a release note.
https://github.com/llvm/llvm-project/pull/126341
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/c
gedare wrote:
Review comments addressed. Nice splitting of the test case.
https://github.com/llvm/llvm-project/pull/112482
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/112482
>From 8282f754ae40b64a88e2d22a6cb21e7028da8371 Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Tue, 15 Oct 2024 23:55:49 -0600
Subject: [PATCH 1/7] [clang-format] add BinPackLongBracedList style option
The use
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/112482
>From 8282f754ae40b64a88e2d22a6cb21e7028da8371 Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Tue, 15 Oct 2024 23:55:49 -0600
Subject: [PATCH 1/6] [clang-format] add BinPackLongBracedList style option
The use
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/112482
>From 8282f754ae40b64a88e2d22a6cb21e7028da8371 Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Tue, 15 Oct 2024 23:55:49 -0600
Subject: [PATCH 1/4] [clang-format] add BinPackLongBracedList style option
The use
https://github.com/sakria9 updated
https://github.com/llvm/llvm-project/pull/126341
>From a382aa9de206a194397f9424150f170d35101b42 Mon Sep 17 00:00:00 2001
From: sakria9
Date: Sat, 8 Feb 2025 04:00:08 +
Subject: [PATCH 1/2] [ASTDump] Add support for structural value template
arguments in T
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (sakria9)
Changes
Dump structural value in AST.
Example code
```cpp
template<_Complex int x>
struct E;
template<>
struct E<{1, 2}>;
```
Command `clang++ -Xclang -ast-dump usr.cpp -fsyntax-only -std=c++23`
Current AST dump result
`
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
https://github.com/sakria9 created
https://github.com/llvm/llvm-project/pull/126341
Dump structural value in AST.
Example code
```cpp
template<_Complex int x>
struct E;
template<>
struct E<{1, 2}>;
```
Command `clang++ -Xclang -ast-dump usr.cpp -fsyntax-only -std=c++23`
Current AST dump resu
https://github.com/Armanidashh commented:
Deontae
https://github.com/llvm/llvm-project/pull/122370
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
owenca wrote:
See #126340.
https://github.com/llvm/llvm-project/pull/126019
___
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-format
Author: Owen Pan (owenca)
Changes
Fixes #125012.
---
Full diff: https://github.com/llvm/llvm-project/pull/126340.diff
2 Files Affected:
- (modified) clang/lib/Format/TokenAnnotator.cpp (+5-2)
- (modified) clang/unittests/Format/TokenAnn
https://github.com/owenca created
https://github.com/llvm/llvm-project/pull/126340
Fixes #125012.
>From a0e44456a4fa1f5ef1168ac0e399b7113252bf96 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Fri, 7 Feb 2025 19:53:17 -0800
Subject: [PATCH] [clang-format] Handle C-style cast of member function p
https://github.com/Flandini updated
https://github.com/llvm/llvm-project/pull/125638
>From 0674909f03703a70c3e259acd0590f50cea4615f Mon Sep 17 00:00:00 2001
From: Michael Flanders
Date: Mon, 27 Jan 2025 11:35:03 -0600
Subject: [PATCH 01/30] wip
---
.../Checkers/StackAddrEscapeChecker.cpp
chrisPyr wrote:
Ok, to make it clear, I’ll split it up by projects.
And I will leave function local variables as-is.
Is that fair enough?
On Sat, Feb 8, 2025 at 04:45 Vitaly Buka ***@***.***> wrote:
> All these changes are independent and quite straight forward.
>
> I'd suggest to split in some
https://github.com/Flandini edited
https://github.com/llvm/llvm-project/pull/125638
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Flandini edited
https://github.com/llvm/llvm-project/pull/125638
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/lalaniket8 updated
https://github.com/llvm/llvm-project/pull/125646
>From 3608d77e3633d853682ffbae16c5b227259ed69c Mon Sep 17 00:00:00 2001
From: anikelal
Date: Tue, 4 Feb 2025 12:13:20 +0530
Subject: [PATCH 1/2] [Driver][HIP] Do not pass -dependency-file flag for HIP
Device
https://github.com/Flandini edited
https://github.com/llvm/llvm-project/pull/125638
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -247,45 +240,134 @@ void StackAddrEscapeChecker::checkPreCall(const
CallEvent &Call,
}
}
-void StackAddrEscapeChecker::checkPreStmt(const ReturnStmt *RS,
- CheckerContext &C) const {
- if (!ChecksEnabled[CK_StackAddrEscapeChecker
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 4294fe173e9696a4c090857fa0766cd44c1964dc
83b532a3382a07e472558b8813d43a6f51801423 --e
llvmbot wrote:
@llvm/pr-subscribers-llvm-transforms
Author: None (luxufan)
Changes
This commit enhances the ThinLTO pipeline to support the elimination of unused
Run-Time Type Information (RTTI) data when the `-fno-split-lto-unit` flag is
used. Previously, dead RTTI data was not effectiv
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
@llvm/pr-subscribers-llvm-ir
Author: None (luxufan)
Changes
This commit enhances the ThinLTO pipeline to support the elimination of unused
Run-Time Type Information (RTTI) data when the `-fno-split-lto-unit` flag is
used. Previously, dead
https://github.com/luxufan created
https://github.com/llvm/llvm-project/pull/126336
This commit enhances the ThinLTO pipeline to support the elimination of unused
Run-Time Type Information (RTTI) data when the `-fno-split-lto-unit` flag is
used. Previously, dead RTTI data was not effectively r
https://github.com/YutongZhuu updated
https://github.com/llvm/llvm-project/pull/125370
>From 380ae2020f71cc5006db2e29b0a69f61297f585c Mon Sep 17 00:00:00 2001
From: Yutong Zhu
Date: Sat, 1 Feb 2025 20:09:13 -0500
Subject: [PATCH 1/2] Force AttributedStmtClass to not be scope parents
---
clang
owenca wrote:
The root cause is that the l_paren after the return type is not annotated.
That's why the space between them is missing.
https://github.com/llvm/llvm-project/pull/126019
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lis
@@ -874,6 +874,11 @@ TEST_F(TokenAnnotatorTest, UnderstandsCasts) {
EXPECT_TOKEN(Tokens[14], tok::r_paren, TT_CastRParen);
EXPECT_TOKEN(Tokens[15], tok::amp, TT_UnaryOperator);
+ Tokens = annotate("func((foo(bar::*)(void))&a);");
owenca wrote:
So `foo` i
haberman wrote:
Consider the case that `boring()` uses `preserve_most`:
```c
__attribute__((preserve_most)) void boring(void *);
__attribute__((preserve_none)) void (continuation)(void *, void *, void *, void
*, void*);
__attribute__((preserve_none)) void entry(void *a, void *b, void *c, void
@@ -14185,6 +14185,32 @@ TEST_F(FormatTest, LayoutCxx11BraceInitializers) {
"ff, g, hh, ii, jj, kk,\n"
"};",
NoBinPacking);
+ NoBinPacking.BinPackLongBracedLists = false;
+ verifyFormat("const Aa a = {\n"
+
@@ -14185,6 +14185,32 @@ TEST_F(FormatTest, LayoutCxx11BraceInitializers) {
"ff, g, hh, ii, jj, kk,\n"
"};",
NoBinPacking);
+ NoBinPacking.BinPackLongBracedLists = false;
+ verifyFormat("const Aa a = {\n"
+
@@ -1212,6 +1212,22 @@ struct FormatStyle {
/// \version 3.7
bool BinPackArguments;
+ /// If ``BinPackLongBracedLists`` is ``true`` it overrides
+ /// ``BinPackArguments`` if there are 20 or more items in a braced
+ /// initializer list.
+ /// \code
+ ///BinPackLo
@@ -14185,6 +14185,32 @@ TEST_F(FormatTest, LayoutCxx11BraceInitializers) {
"ff, g, hh, ii, jj, kk,\n"
"};",
NoBinPacking);
+ NoBinPacking.BinPackLongBracedLists = false;
owenca wrote:
```suggestion
NoBinPacki
@@ -2520,6 +2536,9 @@ struct FormatStyle {
/// (e.g. a type or variable name), clang-format formats as if the ``{}``
were
/// the parentheses of a function call with that name. If there is no name,
/// a zero-length name is assumed.
+ ///
+ /// ``BinPackArguments`` may
ChuanqiXu9 wrote:
> Ready for review; all relevant tests passing.
I think @rnk was saying you should add a test for this.
https://github.com/llvm/llvm-project/pull/121046
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/c
https://github.com/YutongZhuu updated
https://github.com/llvm/llvm-project/pull/125370
>From 380ae2020f71cc5006db2e29b0a69f61297f585c Mon Sep 17 00:00:00 2001
From: Yutong Zhu
Date: Sat, 1 Feb 2025 20:09:13 -0500
Subject: [PATCH] Force AttributedStmtClass to not be scope parents
---
clang/doc
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `premerge-monolithic-linux`
running on `premerge-linux-1` while building `clang` at step 7
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/153/builds/22281
Here is
https://github.com/Ami-zhang closed
https://github.com/llvm/llvm-project/pull/123682
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Ami-zhang
Date: 2025-02-08T09:48:41+08:00
New Revision: 12a154a94a9c2f6f0690adc7302da9c9e47ec806
URL:
https://github.com/llvm/llvm-project/commit/12a154a94a9c2f6f0690adc7302da9c9e47ec806
DIFF:
https://github.com/llvm/llvm-project/commit/12a154a94a9c2f6f0690adc7302da9c9e47ec806.diff
LOG
fmayer wrote:
> Usually our tweak flags looks like -fsanitize--
>
> "bounds" is a group for (array-bounds + local-bounds) "undefined" is also
> group, but we already have -fsanitize-undefined-, but they kind of affect a
> few sanitizers
>
> so: `-fsanitize-undefined-strict-flex-arrays` or
>
https://github.com/fmayer edited
https://github.com/llvm/llvm-project/pull/126163
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/fmayer updated
https://github.com/llvm/llvm-project/pull/126163
>From e9f2dbecb2b4836100a565b4c741fc3425d08966 Mon Sep 17 00:00:00 2001
From: Florian Mayer
Date: Thu, 6 Feb 2025 16:50:02 -0800
Subject: [PATCH 1/3] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF
https://github.com/fmayer updated
https://github.com/llvm/llvm-project/pull/126163
>From e9f2dbecb2b4836100a565b4c741fc3425d08966 Mon Sep 17 00:00:00 2001
From: Florian Mayer
Date: Thu, 6 Feb 2025 16:50:02 -0800
Subject: [PATCH 1/2] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF
@@ -1192,13 +1193,40 @@ llvm::Value *CodeGenFunction::EmitLoadOfCountedByField(
return nullptr;
}
+LangOptions::StrictFlexArraysLevelKind
+CodeGenFunction::effectiveArrayBoundsFlexArraysLevel() {
+ using StrictFlexArraysLevelKind = LangOptions::StrictFlexArraysLevelKind;
+
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `llvm-clang-aarch64-darwin`
running on `doug-worker-4` while building `clang` at step 6
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/190/builds/14311
Here is th
https://github.com/Meinersbur edited
https://github.com/llvm/llvm-project/pull/122341
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -444,6 +444,19 @@ class LangOptionsBase {
IncompleteOnly = 3,
};
+ enum class ArrayBoundsStrictFlexArraysLevelKind {
fmayer wrote:
i have a weak preference for flags to have a value that can represent the state
that is the same as if they were not
https://github.com/ilovepi updated
https://github.com/llvm/llvm-project/pull/126306
>From 996088f74e517bcde209dfbbb1c1e5c245f96535 Mon Sep 17 00:00:00 2001
From: Paul Kirth
Date: Fri, 7 Feb 2025 12:47:05 -0800
Subject: [PATCH] [Fuchsia] Disable building runtimes with LTO
Temporarily disable LT
Author: Paul Kirth
Date: 2025-02-07T16:57:11-08:00
New Revision: 3e2afe5f019b7a1c60e23cb2743018bd2d0417b1
URL:
https://github.com/llvm/llvm-project/commit/3e2afe5f019b7a1c60e23cb2743018bd2d0417b1
DIFF:
https://github.com/llvm/llvm-project/commit/3e2afe5f019b7a1c60e23cb2743018bd2d0417b1.diff
LO
https://github.com/ilovepi closed
https://github.com/llvm/llvm-project/pull/126293
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
petrhosek wrote:
LGTM
https://github.com/llvm/llvm-project/pull/126293
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -444,6 +444,19 @@ class LangOptionsBase {
IncompleteOnly = 3,
};
+ enum class ArrayBoundsStrictFlexArraysLevelKind {
fmayer wrote:
because we currently only store 3 bits for this, using -1 would be out of range
of that and just truncate to 111 = 7
@@ -1192,13 +1193,40 @@ llvm::Value *CodeGenFunction::EmitLoadOfCountedByField(
return nullptr;
}
+LangOptions::StrictFlexArraysLevelKind
+CodeGenFunction::effectiveArrayBoundsFlexArraysLevel() {
+ using StrictFlexArraysLevelKind = LangOptions::StrictFlexArraysLevelKind;
+
https://github.com/fmayer deleted
https://github.com/llvm/llvm-project/pull/126163
___
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/126206
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zyn0217 wrote:
> I see the test references https://github.com/llvm/llvm-project/issues/124715
> but you don't mention it in the summary and you don't have a release note.
> Does the test exercise the crash?
Oh, thanks for spotting that - I forgot it when I updated the commit body.
Will add a
@@ -444,6 +444,19 @@ class LangOptionsBase {
IncompleteOnly = 3,
};
+ enum class ArrayBoundsStrictFlexArraysLevelKind {
vitalybuka wrote:
We can do the same in cc1, the case will not be used, because driver will
always pass something?
BTW, why -1 doe
https://github.com/shafik commented:
I see the test references https://github.com/llvm/llvm-project/issues/124715
but you don't mention it in the summary and you don't have a release note. Does
the test exercise the crash?
https://github.com/llvm/llvm-project/pull/126206
__
1 - 100 of 527 matches
Mail list logo
