https://github.com/kito-cheng approved this pull request.
LGTM :)
https://github.com/llvm/llvm-project/pull/85899
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
I wanted to make you aware of this new core issue
https://cplusplus.github.io/CWG/issues/2876.html (which i think we should have
tests for). Thanks
https://github.com/llvm/llvm-project/pull/86526
___
cfe-commits mailing list
cfe-commi
https://github.com/wzssyqa approved this pull request.
https://github.com/llvm/llvm-project/pull/87866
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wzssyqa closed
https://github.com/llvm/llvm-project/pull/87319
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
wzssyqa wrote:
Replaced by: https://github.com/llvm/llvm-project/issues/87150
https://github.com/llvm/llvm-project/pull/87319
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wangpc-pp created
https://github.com/llvm/llvm-project/pull/87966
Remove them from experimental.
See also:
https://github.com/riscv/riscv-isa-manual/blob/main/src/zimop.adoc
>From 5b312a80cdc3396e3e35d906176f56349392d437 Mon Sep 17 00:00:00 2001
From: Wang Pengcheng
Date:
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Pengcheng Wang (wangpc-pp)
Changes
Remove them from experimental.
See also:
https://github.com/riscv/riscv-isa-manual/blob/main/src/zimop.adoc
---
Patch is 26.77 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/
https://github.com/kito-cheng approved this pull request.
LGTM :)
https://github.com/llvm/llvm-project/pull/87966
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -265,6 +269,18 @@ namespace dr527 { // dr527: na
int ax = a.x, bx = b.x, cx = c.x, dx = d.x, ex = E::e->x, fx = F::f->x;
}
+namespace dr528 { // dr528: 2.7
cor3ntin wrote:
This is NAD, should we not say that?
https://github.com/llvm/llvm-project/pull/87
@@ -1568,6 +1568,18 @@ namespace dr391 { // dr391: 2.8 c++11
}
// dr392 is in dr392.cpp
+
+namespace dr393 { // dr393: 2.7
+
+template
+struct S {};
+
+void f1(S);
+void f2(S);
+void g(int(*S::*)[]);
cor3ntin wrote:
Can you add the January 2013 examples ? (b
@@ -661,6 +680,10 @@ namespace dr553 {
}
// dr554: na
+
+// dr555: na
cor3ntin wrote:
It might be worth copying these tests.
I agree that the changes are non-behavioral but I'd like @AaronBallman to give
a second opinion
https://github.com/llvm/llvm-project
Author: Pengcheng Wang
Date: 2024-04-08T16:40:02+08:00
New Revision: 364028a1a51689d2b33d3ec50c426fbeac269679
URL:
https://github.com/llvm/llvm-project/commit/364028a1a51689d2b33d3ec50c426fbeac269679
DIFF:
https://github.com/llvm/llvm-project/commit/364028a1a51689d2b33d3ec50c426fbeac269679.diff
https://github.com/wangpc-pp closed
https://github.com/llvm/llvm-project/pull/87966
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/NagyDonat approved this pull request.
https://github.com/llvm/llvm-project/pull/87795
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/balazske updated
https://github.com/llvm/llvm-project/pull/87322
From 79bbe640c0d60744f484db9965865455b0b15246 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bal=C3=A1zs=20K=C3=A9ri?=
Date: Tue, 2 Apr 2024 09:59:48 +0200
Subject: [PATCH 1/3] [clang][analyzer] Add "pedantic" mode to
https://github.com/jcsxky updated
https://github.com/llvm/llvm-project/pull/87173
>From 51f3dc24417eb0b74f029b85b47519b6d152 Mon Sep 17 00:00:00 2001
From: huqizhi
Date: Sun, 31 Mar 2024 09:38:05 +0800
Subject: [PATCH] [Clang][Sema] set declaration invalid earlier to prevent
crash in calcu
https://github.com/jcsxky edited https://github.com/llvm/llvm-project/pull/87173
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Zentrik
Date: 2024-04-08T11:18:59+02:00
New Revision: cf7d36fe342c5c5ac39150ca0b4b70a3d17ae66b
URL:
https://github.com/llvm/llvm-project/commit/cf7d36fe342c5c5ac39150ca0b4b70a3d17ae66b
DIFF:
https://github.com/llvm/llvm-project/commit/cf7d36fe342c5c5ac39150ca0b4b70a3d17ae66b.diff
LOG:
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/87795
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Endilll updated
https://github.com/llvm/llvm-project/pull/87909
>From 222f444f91d477bbc1ab2657a7f820740031f765 Mon Sep 17 00:00:00 2001
From: Vlad Serebrennikov
Date: Sun, 7 Apr 2024 10:52:10 +0300
Subject: [PATCH 1/2] [clang] Add tests for some CWG 5xx issues
---
clang/tes
cor3ntin wrote:
Thanks!
https://github.com/llvm/llvm-project/pull/87795
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
github-actions[bot] wrote:
@Zentrik Congratulations on having your first Pull Request (PR) merged into the
LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested
by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with
a build,
@@ -265,6 +269,18 @@ namespace dr527 { // dr527: na
int ax = a.x, bx = b.x, cx = c.x, dx = d.x, ex = E::e->x, fx = F::f->x;
}
+namespace dr528 { // dr528: 2.7
Endilll wrote:
For NAD issues, we write tests that check for status quo, and report the first
ver
@@ -661,6 +680,10 @@ namespace dr553 {
}
// dr554: na
+
+// dr555: na
Endilll wrote:
I'm not keen to copy tests for an issue that is officially recognized as
duplicate.
https://github.com/llvm/llvm-project/pull/87909
@@ -1568,6 +1568,18 @@ namespace dr391 { // dr391: 2.8 c++11
}
// dr392 is in dr392.cpp
+
+namespace dr393 { // dr393: 2.7
+
+template
+struct S {};
+
+void f1(S);
+void f2(S);
+void g(int(*S::*)[]);
Endilll wrote:
Done.
https://github.com/llvm/llvm-project
vvd170501 wrote:
Ping
https://github.com/llvm/llvm-project/pull/87208
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -45,6 +51,19 @@ class ExprMutationAnalyzer {
using MutationFinder = const Stmt *(ExprMutationAnalyzer::*)(const Expr *);
using ResultMap = llvm::DenseMap;
+ ExprMutationAnalyzer(const Stmt &Stm, ASTContext &Context, Cache
*ParentCache)
+ : Stm(Stm), Context(Conte
https://github.com/cjacek approved this pull request.
https://github.com/llvm/llvm-project/pull/87725
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
=?utf-8?q?Balázs_Kéri?= ,
=?utf-8?q?Balázs_Kéri?=
Message-ID:
In-Reply-To:
https://github.com/NagyDonat approved this pull request.
Thanks for the update! Feel free to merge the commit and initiate the de-alpha
process.
https://github.com/llvm/llvm-project/pull/87322
https://github.com/cjacek approved this pull request.
> (If we actually need some of these intrinsics in ARM64EC mode, we can
> revisit later.)
FWIW, most of them are provided by MSVC/Windows SDK via `softintrin.h`
(included from `intrin.h`) as functions, not actual intrisics.
https://github
Author: Fraser Cormack
Date: 2024-04-08T11:05:38+01:00
New Revision: 8461d901a770516cf2069fe3bce979a6f8fc8d76
URL:
https://github.com/llvm/llvm-project/commit/8461d901a770516cf2069fe3bce979a6f8fc8d76
DIFF:
https://github.com/llvm/llvm-project/commit/8461d901a770516cf2069fe3bce979a6f8fc8d76.diff
frasercrmck wrote:
> This change broke standalone build against LLVM dylib — it now insists on
> linking to non-existing static libraries:
>
> ```
> FAILED: prepare_builtins
> : && /usr/lib/ccache/bin/x86_64-pc-linux-gnu-g++ -march=znver2
> --param=l1-cache-size=32 --param=l1-cache-line-size=
https://github.com/aytey updated https://github.com/llvm/llvm-project/pull/87521
>From 1f70839ea1607f151c9f7eb390fcb974b32a54ca Mon Sep 17 00:00:00 2001
From: "Andrew V. Teylu"
Date: Wed, 3 Apr 2024 17:18:08 +0100
Subject: [PATCH 1/3] [analyzer] `canReasonAbout` does not support
`nonloc::LazyCo
Author: Balázs Kéri
Date: 2024-04-08T12:19:03+02:00
New Revision: c2067c1f471ac54312cb5e1e0efd4ea5fd21cc79
URL:
https://github.com/llvm/llvm-project/commit/c2067c1f471ac54312cb5e1e0efd4ea5fd21cc79
DIFF:
https://github.com/llvm/llvm-project/commit/c2067c1f471ac54312cb5e1e0efd4ea5fd21cc79.diff
L
https://github.com/balazske closed
https://github.com/llvm/llvm-project/pull/87322
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -27,31 +36,51 @@ set( LIBCLC_TARGETS_TO_BUILD "all"
option( ENABLE_RUNTIME_SUBNORMAL "Enable runtime linking of subnormal
support." OFF )
-find_package(LLVM REQUIRED HINTS "${LLVM_CMAKE_DIR}")
-include(AddLLVM)
+if(CMAKE_SOURCE_DIR STREQUAL CMAKE_CURRENT_SOURCE_DIR)
-
@@ -358,3 +399,7 @@ foreach( t ${LIBCLC_TARGETS_TO_BUILD} )
endif()
endforeach( d )
endforeach( t )
+
+if( NOT LIBCLC_STANDALONE_BUILD )
+ add_subdirectory( test )
frasercrmck wrote:
Ach yeah sorry, I have another path locally which adds some LIT tests
https://github.com/frasercrmck updated
https://github.com/llvm/llvm-project/pull/87622
>From b87c2862491aed03f883d6b124a845b0d7c47699 Mon Sep 17 00:00:00 2001
From: Fraser Cormack
Date: Wed, 3 Apr 2024 16:09:30 +0100
Subject: [PATCH] [libclc] Refactor build system to allow in-tree builds
The p
https://github.com/frasercrmck edited
https://github.com/llvm/llvm-project/pull/87622
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -290,6 +290,16 @@ TEST_F(FormatTestTableGen, MultiClass) {
"}\n");
}
+TEST_F(FormatTestTableGen, MultiClassesWithPasteOperator) {
+ // This is a sensitive example for the handling of the paste operators in
+ // brace type calculation.
+ verifyFormat("multi
Author: Hirofumi Nakamura
Date: 2024-04-08T21:12:12+09:00
New Revision: 5a855d51272608e2122c45d86676aa2247a11d19
URL:
https://github.com/llvm/llvm-project/commit/5a855d51272608e2122c45d86676aa2247a11d19
DIFF:
https://github.com/llvm/llvm-project/commit/5a855d51272608e2122c45d86676aa2247a11d19.d
https://github.com/hnakamura5 closed
https://github.com/llvm/llvm-project/pull/87924
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ronlieb approved this pull request.
https://github.com/llvm/llvm-project/pull/87659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Joseph Huber
Date: 2024-04-08T07:14:52-05:00
New Revision: 86b0918e8e5f8e1aacebf4ba8901fc66aed3412f
URL:
https://github.com/llvm/llvm-project/commit/86b0918e8e5f8e1aacebf4ba8901fc66aed3412f
DIFF:
https://github.com/llvm/llvm-project/commit/86b0918e8e5f8e1aacebf4ba8901fc66aed3412f.diff
https://github.com/jhuber6 closed
https://github.com/llvm/llvm-project/pull/87659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hnakamura5 wrote:
Thank you!
https://github.com/llvm/llvm-project/pull/87924
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bahareh-farhadi ready_for_review
https://github.com/llvm/llvm-project/pull/87797
___
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 (bahareh-farhadi)
Changes
PR
[https://github.com/llvm/llvm-project/pull/84461](https://github.com/llvm/llvm-project/pull/84461)
disabled `clang/unittests/Interpreter/InterpreterExtensionsTest.cpp` for AIX
by turning on `CLANG_INTERP
https://github.com/hnakamura5 created
https://github.com/llvm/llvm-project/pull/87983
Refactoring suggested here https://github.com/llvm/llvm-project/pull/87924 .
>From d55a0986ed175dbc18c073507691321a30952979 Mon Sep 17 00:00:00 2001
From: hnakamura5
Date: Mon, 8 Apr 2024 21:20:04 +0900
Subje
https://github.com/abhina-sree approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/87797
___
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: Hirofumi Nakamura (hnakamura5)
Changes
Refactoring suggested here https://github.com/llvm/llvm-project/pull/87924 .
---
Full diff: https://github.com/llvm/llvm-project/pull/87983.diff
1 Files Affected:
- (modified) clang/unittest
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/87361
>From b8a626116b0719c1acf75e9e7300df8e2bf82f99 Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Tue, 2 Apr 2024 18:00:05 +0200
Subject: [PATCH 1/2] [Clang] Reduce the size of Decl and classes derived from
@@ -268,17 +268,34 @@ class alignas(8) Decl {
/// }
/// void A::f(); // SemanticDC == namespace 'A'
///// LexicalDC == global namespace
- llvm::PointerUnion DeclCtx;
+ llvm::PointerIntPair<
philnik777 wrote:
IDK. I'm personally not
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Nikolas Klauser (philnik777)
Changes
Class | Old size (in bytes) | New size (in bytes)
--|-|
Decl | 40
llvmbot wrote:
@llvm/pr-subscribers-clang-modules
Author: Nikolas Klauser (philnik777)
Changes
Class | Old size (in bytes) | New size (in bytes)
--|-|
Decl | 40
Author: bahareh-farhadi
Date: 2024-04-08T08:35:28-04:00
New Revision: 8c6e0459c49da298f3b911fc3699c2254a20d882
URL:
https://github.com/llvm/llvm-project/commit/8c6e0459c49da298f3b911fc3699c2254a20d882
DIFF:
https://github.com/llvm/llvm-project/commit/8c6e0459c49da298f3b911fc3699c2254a20d882.dif
https://github.com/abhina-sree closed
https://github.com/llvm/llvm-project/pull/87797
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
github-actions[bot] wrote:
@bahareh-farhadi Congratulations on having your first Pull Request (PR) merged
into the LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested
by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with
@@ -688,22 +689,45 @@ class Environment {
/// and functions referenced in `FuncDecl`. `FuncDecl` must have a body.
void initFieldsGlobalsAndFuncs(const FunctionDecl *FuncDecl);
+ static PrValueToResultObject
+ buildResultObjectMap(DataflowAnalysisContext *DACtx,
+
@@ -688,22 +689,45 @@ class Environment {
/// and functions referenced in `FuncDecl`. `FuncDecl` must have a body.
void initFieldsGlobalsAndFuncs(const FunctionDecl *FuncDecl);
+ static PrValueToResultObject
+ buildResultObjectMap(DataflowAnalysisContext *DACtx,
+
@@ -385,6 +388,185 @@ getFieldsGlobalsAndFuncs(const Stmt &S, FieldSet &Fields,
}
}
+namespace {
+
+// Visitor that builds a map from record prvalues to result objects.
+// This traverses the body of the function to be analyzed; for each result
+// object that it encounters,
https://github.com/kovdan01 edited
https://github.com/llvm/llvm-project/pull/85235
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jcsxky wrote:
Still crash on
```cpp
template constexpr auto x = F();
template constexpr int a() { return 1; }
template
struct A {
using Func = decltype(
[](T) {
return x<[] constexpr { return a(); }>;
// return x<[] constexpr { return b(); }>;
}.template operator()('
@@ -460,25 +460,35 @@ class TransferVisitor : public
ConstStmtVisitor {
// So make sure we have a value if we didn't propagate one above.
if (S->isPRValue() && S->getType()->isRecordType()) {
if (Env.getValue(*S) == nullptr) {
-Value *Val = Env.createValu
@@ -688,71 +682,51 @@ class TransferVisitor : public
ConstStmtVisitor {
return;
}
-llvm::DenseMap FieldLocs;
-RecordInitListHelper InitListHelper(S);
+RecordStorageLocation &Loc = Env.getResultObjectLocation(*S);
+Env.setValue(*S, refreshRecordValue(
Author: Jefferson Le Quellec
Date: 2024-04-08T14:11:43+01:00
New Revision: 2921a0928c71f4ee652a2478283e47ab5ffebf58
URL:
https://github.com/llvm/llvm-project/commit/2921a0928c71f4ee652a2478283e47ab5ffebf58
DIFF:
https://github.com/llvm/llvm-project/commit/2921a0928c71f4ee652a2478283e47ab5ffebf5
sdkrystian wrote:
@erichkeane PR updated.
When we encounter a class member access expression with a nested-name-specifier
that is invalid, should we simply ignore it and instead use the type of the
object expression as the lookup context? This is the current behavior, but it
leads to lots of
https://github.com/ldrumm closed https://github.com/llvm/llvm-project/pull/86807
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/phoebewang created
https://github.com/llvm/llvm-project/pull/87987
This relands #87149.
The previous commit exposed failures on some targets. The reason is only a few
targets support COFF ObjectFormatType on Windows:
https://github.com/llvm/llvm-project/blob/main/llvm/lib/T
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Phoebe Wang (phoebewang)
Changes
This relands #87149.
The previous commit exposed failures on some targets. The reason is only a few
targets support COFF ObjectFormatType on Windows:
https://github.com/llvm/llvm-project/blob/main/llvm/li
https://github.com/phoebewang edited
https://github.com/llvm/llvm-project/pull/87987
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -345,6 +345,9 @@ Bug Fixes to Compiler Builtins
Bug Fixes to Attribute Support
^^
+- Clang now correctly matches plugin attributes with microsoft ``[attribute]``
syntax.
erichkeane wrote:
> Microsoft attributes use single brack
@@ -0,0 +1,37 @@
+//===- SemaHLSL.h - Semantic Analysis for HLSL constructs
-===//
+//
+// 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: Apa
https://github.com/llvm-beanz approved this pull request.
I really like this direction. Thank you for taking the initiative on cleaning
Sema up!
I had one small comment below, but you can take it or leave it.
https://github.com/llvm/llvm-project/pull/87912
_
https://github.com/llvm-beanz edited
https://github.com/llvm/llvm-project/pull/87912
___
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 5d7fd6a04a6748936dece9d90481b2ba4ec97e53
6ba913fb0d3efdb17ae481ccad63ccde2170d4e2 --
https://github.com/frasercrmck created
https://github.com/llvm/llvm-project/pull/87989
None
>From b87c2862491aed03f883d6b124a845b0d7c47699 Mon Sep 17 00:00:00 2001
From: Fraser Cormack
Date: Wed, 3 Apr 2024 16:09:30 +0100
Subject: [PATCH 1/2] [libclc] Refactor build system to allow in-tree bui
github-actions[bot] wrote:
:warning: Python code formatter, darker found issues in your code. :warning:
You can test this locally with the following command:
``bash
darker --check --diff -r
8461d901a770516cf2069fe3bce979a6f8fc8d76...29c9a184c11ed0ba46a8c91b6fbeea828c9fce0d
libcl
11happy wrote:
Humble reminder! @PiotrZSL
https://github.com/llvm/llvm-project/pull/84481
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,37 @@
+//===- SemaHLSL.h - Semantic Analysis for HLSL constructs
-===//
+//
+// 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: Apa
Author: Simon Pilgrim
Date: 2024-04-08T14:31:46+01:00
New Revision: 0e8736694f752898ed7957a11a11c42f8f6a98d1
URL:
https://github.com/llvm/llvm-project/commit/0e8736694f752898ed7957a11a11c42f8f6a98d1
DIFF:
https://github.com/llvm/llvm-project/commit/0e8736694f752898ed7957a11a11c42f8f6a98d1.diff
Author: Simon Pilgrim
Date: 2024-04-08T14:31:46+01:00
New Revision: f139387fb6e76a5249e8d7c2d124565e6b566ef4
URL:
https://github.com/llvm/llvm-project/commit/f139387fb6e76a5249e8d7c2d124565e6b566ef4
DIFF:
https://github.com/llvm/llvm-project/commit/f139387fb6e76a5249e8d7c2d124565e6b566ef4.diff
erichkeane wrote:
> @erichkeane PR updated.
>
> When we encounter a class member access expression with a
> nested-name-specifier that is invalid, should we simply ignore it and instead
> use the type of the object expression as the lookup context? This is the
> current behavior, but it leads
https://github.com/martinboehme updated
https://github.com/llvm/llvm-project/pull/87320
>From 2d2aa88aab3e47e41588397471a90c03bb55d900 Mon Sep 17 00:00:00 2001
From: Martin Braenne
Date: Tue, 2 Apr 2024 08:00:00 +
Subject: [PATCH 1/2] [clang][dataflow] Propagate locations from result object
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/87361
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/87361
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -268,17 +268,34 @@ class alignas(8) Decl {
/// }
/// void A::f(); // SemanticDC == namespace 'A'
///// LexicalDC == global namespace
- llvm::PointerUnion DeclCtx;
+ llvm::PointerIntPair<
erichkeane wrote:
I don't think it ends up
@@ -460,25 +460,35 @@ class TransferVisitor : public
ConstStmtVisitor {
// So make sure we have a value if we didn't propagate one above.
if (S->isPRValue() && S->getType()->isRecordType()) {
if (Env.getValue(*S) == nullptr) {
-Value *Val = Env.createValu
@@ -385,6 +388,185 @@ getFieldsGlobalsAndFuncs(const Stmt &S, FieldSet &Fields,
}
}
+namespace {
+
+// Visitor that builds a map from record prvalues to result objects.
+// This traverses the body of the function to be analyzed; for each result
+// object that it encounters,
sdkrystian wrote:
> Sorry, I'm not really getting the question here. Obviously that 2nd error is
> invalid and we shouldn't be emitting it, and doing the `::z` lookup inside of
> `A` seems wrong to me. But I'm not sure I get the issue.
We currently emit the second error. I'm asking if we shoul
https://github.com/PiotrZSL approved this pull request.
LGTM,
Would be good to run it on llvm or any other code base, and check what it found.
https://github.com/llvm/llvm-project/pull/84481
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https
martinboehme wrote:
I think all comments should be addressed.
@Xazax-hun Should be ready for further review.
@ymand LMK if you have any additional comments.
https://github.com/llvm/llvm-project/pull/87320
___
cfe-commits mailing list
cfe-commits@list
@@ -110,6 +110,12 @@ New checks
Detects error-prone Curiously Recurring Template Pattern usage, when the CRTP
can be constructed outside itself and the derived class.
+- New :doc:`bugprone-null-check-after-dereference
+ ` check.
+
+ This check identifies redundant pointe
@@ -217,34 +231,72 @@ void matchDereferenceExpr(const Stmt *stmt,
Env.assume(Env.arena().makeNot(getVal(kIsNull, *RootValue).formula()));
}
-void matchCastExpr(const CastExpr *cond, const MatchFinder::MatchResult
&Result,
- NullPointerAnalysisModel::Transf
mgorny wrote:
Thanks. This seems to fix that problem. However, now I'm seeing missing dep in
build ordering:
```
[1793/1922] cd /tmp/portage/dev-libs/libclc-19.0.0./work/libclc_build &&
/usr/lib/llvm/17/bin/llvm-spirv --spirv-max-version=1.1 -o spirv-mesa3d-.spv
builtins.opt.spirv-mesa3d-
https://github.com/erichkeane updated
https://github.com/llvm/llvm-project/pull/87821
>From ca5f957bb002d79997e630cdf1aaad5703ea2ba4 Mon Sep 17 00:00:00 2001
From: erichkeane
Date: Fri, 5 Apr 2024 10:53:47 -0700
Subject: [PATCH 1/2] [OpenACC] Implement Sema work for OpenACC Clauses
Now that we
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 f6357bb4283e72d1248b9c7eb67d98bea71d5f50
27da3822961873340af021a5cbf136a155b5f110 --
mgorny wrote:
```
set( spv_suffix ${arch_suffix}.spv )
add_custom_command( OUTPUT "${spv_suffix}"
COMMAND ${LLVM_SPIRV} ${spvflags} -o "${spv_suffix}"
${builtins_opt_lib_tgt}
DEPENDS ${builtins_link_lib_tgt} )
```
I think that ought to be `DEPENDS ${builtins_opt_lib_
@@ -37,6 +37,32 @@ bool diagnoseConstructAppertainment(SemaOpenACC &S,
OpenACCDirectiveKind K,
}
return false;
}
+
+bool doesClauseApplyToDirective(OpenACCDirectiveKind DirectiveKind,
+OpenACCClauseKind ClauseKind) {
+ switch (ClauseKind) {
@@ -82,22 +119,22 @@ bool
SemaOpenACC::ActOnStartStmtDirective(OpenACCDirectiveKind K,
return diagnoseConstructAppertainment(*this, K, StartLoc, /*IsStmt=*/true);
}
-StmtResult SemaOpenACC::ActOnEndStmtDirective(OpenACCDirectiveKind K,
-
1 - 100 of 389 matches
Mail list logo
