sylvestre wrote:
I will merge it as it as it fixes
https://github.com/llvm/llvm-project/issues/127378 and apt.llvm.org has been
broken for a few days because of this
This PR can be improved later.
https://github.com/llvm/llvm-project/pull/127315
__
Author: Michał Górny
Date: 2025-02-16T08:48:52+01:00
New Revision: dbc98cfa46d52ede06e8be7fc5e855d807ba0fac
URL:
https://github.com/llvm/llvm-project/commit/dbc98cfa46d52ede06e8be7fc5e855d807ba0fac
DIFF:
https://github.com/llvm/llvm-project/commit/dbc98cfa46d52ede06e8be7fc5e855d807ba0fac.diff
https://github.com/sylvestre edited
https://github.com/llvm/llvm-project/pull/127315
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sylvestre closed
https://github.com/llvm/llvm-project/pull/127315
___
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-tidy
@llvm/pr-subscribers-clang-tools-extra
Author: None (Andrewyuan34)
Changes
This PR fixes issue #124906, where `modernize-use-ranges` failed to
detect cases where a `const` member function returning an lvalue was
incorrectly assumed to have
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/Andrewyuan34 created
https://github.com/llvm/llvm-project/pull/127377
This PR fixes issue #124906, where `modernize-use-ranges` failed to detect
cases where a `const` member function returning an lvalue was incorrectly
assumed to have side effects, preventing the transforma
owenca wrote:
I prefer that we limit this to breaking after the left parenthesis of a control
statement, with true/false or Always, Never, Multiline, BlockIndent, etc.
https://github.com/llvm/llvm-project/pull/108332
___
cfe-commits mailing list
cfe-c
https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/127370
None
>From 356e99802d0d77be7fad4f9297a605334e08ffbd Mon Sep 17 00:00:00 2001
From: Kazu Hirata
Date: Sat, 15 Feb 2025 01:40:13 -0800
Subject: [PATCH] [clang-tidy] Avoid repeated hash lookups (NFC)
---
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Kazu Hirata (kazutakahirata)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/127370.diff
1 Files Affected:
- (modified) clang-tools-extra/clang-tidy/bugprone/VirtualNearMissCheck.cpp
(+3-3)
``di
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Kazu Hirata (kazutakahirata)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/127369.diff
1 Files Affected:
- (modified) clang/lib/AST/ExternalASTMerger.cpp (+7-9)
``diff
diff --git a/clang/lib/AST/ExternalAS
https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/127369
None
>From 9400819602c7c8b79cb21528cd668070132bc608 Mon Sep 17 00:00:00 2001
From: Kazu Hirata
Date: Sat, 15 Feb 2025 01:41:07 -0800
Subject: [PATCH] [AST] Avoid repeated map lookups (NFC)
---
clang/li
https://github.com/changkhothuychung updated
https://github.com/llvm/llvm-project/pull/120920
error: too big or took too long to generate
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
mgorny wrote:
> But `lib` is used for 32-bit libraries, and `lib64` for 64-bit libraries.
…and just to be clear, Flang runtimes are installed into plain system
`CMAKE_INSTALL_LIBDIR` rather than `clang/lib/...` path (along with the "host"
Flang libraries), and they don't have any arch-suffix,
mgorny wrote:
But `lib` is used for 32-bit libraries, and `lib64` for 64-bit libraries.
https://github.com/llvm/llvm-project/pull/127345
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arsenm wrote:
> seeing breaks in downstream build of rocPRIM
Probably need to revert the downstream revert of the original problematic patch
https://github.com/llvm/llvm-project/pull/114062
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
Vicente Mataix =?utf-8?q?Ferrándiz?= ,
Vicente Mataix =?utf-8?q?Ferrándiz?= ,
Vicente Mataix =?utf-8?q?Ferrándiz?=
Message-ID:
In-Reply-To:
HerrCai0907 wrote:
> Can you help me?, I have no experience in this repository
release-note is placed `clang-tools-extra/docs/ReleaseNotes.rst`. You can
@@ -2245,6 +2245,36 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
break;
}
+ case Builtin::BI__builtin_hlsl_and: {
+if (SemaRef.checkArgCount(TheCall, 2))
+ return true;
+if (CheckVectorElementCallArgs(&SemaRef, TheCal
https://github.com/phoebewang approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/127331
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/trelau updated
https://github.com/llvm/llvm-project/pull/123539
>From bb0542e8f2ad50892ee9d2c1f76ec1def85c3e56 Mon Sep 17 00:00:00 2001
From: Trevor Laughlin
Date: Sun, 19 Jan 2025 19:21:10 -0500
Subject: [PATCH 1/2] [cindex] Add API to query the class methods of a type
---
llvmbot wrote:
@llvm/pr-subscribers-clangd
@llvm/pr-subscribers-clang-tools-extra
Author: Nathan Ridge (HighCommander4)
Changes
Fixes https://github.com/clangd/clangd/issues/2324
---
Full diff: https://github.com/llvm/llvm-project/pull/127359.diff
1 Files Affected:
- (modified) clang-t
https://github.com/HighCommander4 created
https://github.com/llvm/llvm-project/pull/127359
Fixes https://github.com/clangd/clangd/issues/2324
>From 0ff06807c2c8255f472fc45e9a8922201384bf08 Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Sat, 15 Feb 2025 01:57:10 -0500
Subject: [PATCH] [clang
@@ -54,6 +54,33 @@ unsigned clang_visitCXXBaseClasses(CXType PT, CXFieldVisitor
visitor,
return true;
}
+unsigned clang_visitCXXMethods(CXType PT, CXFieldVisitor visitor,
+ CXClientData client_data) {
+ CXCursor PC = clang_getTypeDeclaration(P
https://github.com/Alcaro updated
https://github.com/llvm/llvm-project/pull/127331
>From 90e3f502e582f44cd19e1b8b71d218330b13c9e4 Mon Sep 17 00:00:00 2001
From: Alcaro
Date: Sat, 15 Feb 2025 16:05:40 +0100
Subject: [PATCH 1/4] [clang][x86] Support -masm=intel in cpuid.h
---
clang/lib/Headers/
https://github.com/Endilll edited
https://github.com/llvm/llvm-project/pull/123539
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Endilll commented:
Seems fine to me, but I'd like @AaronBallman to take a look, too, because
libclang doesn't have a dedicated maintainer and I'm not familiar enough with
it to sign off new functions we add to it.
https://github.com/llvm/llvm-project/pull/123539
DeinAlptraum wrote:
@Endilll can you also review the non-Python part, or do you know someone who
can?
https://github.com/llvm/llvm-project/pull/123539
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
Endilll wrote:
> edit: looks like release notes were "reset" after cutting rc (?)
Yes, this happens every time a release branch is created.
I checked your release notes, they seem fine to me.
https://github.com/llvm/llvm-project/pull/123539
___
cfe-co
@@ -2245,6 +2245,36 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
break;
}
+ case Builtin::BI__builtin_hlsl_and: {
+if (SemaRef.checkArgCount(TheCall, 2))
+ return true;
+if (CheckVectorElementCallArgs(&SemaRef, TheCal
@@ -2245,6 +2245,36 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
break;
}
+ case Builtin::BI__builtin_hlsl_and: {
+if (SemaRef.checkArgCount(TheCall, 2))
+ return true;
+if (CheckVectorElementCallArgs(&SemaRef, TheCal
phoebewang wrote:
Please update test in clang\test\Headers\cpuid.c
https://github.com/llvm/llvm-project/pull/127331
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DimitryAndric approved this pull request.
https://github.com/llvm/llvm-project/pull/126302
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ronlieb wrote:
seeing breaks in downstream build of rocPRIM
https://github.com/llvm/llvm-project/pull/114062
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/farzonl edited
https://github.com/llvm/llvm-project/pull/127098
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
trelau wrote:
Ping
https://github.com/llvm/llvm-project/pull/123539
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
trelau wrote:
Not sure what happen, but there was a merge conflict with ReleaseNotes.rst
where it looked like previous enhancements to liblang and Python bindings had
been removed. To resolve the conflict I only left changes relevant to this PR.
https://github.com/llvm/llvm-project/pull/123539
@@ -2245,6 +2245,36 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
break;
}
+ case Builtin::BI__builtin_hlsl_and: {
+if (SemaRef.checkArgCount(TheCall, 2))
+ return true;
+if (CheckVectorElementCallArgs(&SemaRef, TheCal
https://github.com/trelau updated
https://github.com/llvm/llvm-project/pull/123539
>From bb0542e8f2ad50892ee9d2c1f76ec1def85c3e56 Mon Sep 17 00:00:00 2001
From: Trevor Laughlin
Date: Sun, 19 Jan 2025 19:21:10 -0500
Subject: [PATCH] [cindex] Add API to query the class methods of a type
---
cla
https://github.com/Icohedron edited
https://github.com/llvm/llvm-project/pull/127098
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2245,6 +2245,36 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
break;
}
+ case Builtin::BI__builtin_hlsl_and: {
+if (SemaRef.checkArgCount(TheCall, 2))
+ return true;
+if (CheckVectorElementCallArgs(&SemaRef, TheCal
@@ -2245,6 +2245,36 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
break;
}
+ case Builtin::BI__builtin_hlsl_and: {
+if (SemaRef.checkArgCount(TheCall, 2))
+ return true;
+if (CheckVectorElementCallArgs(&SemaRef, TheCal
https://github.com/Icohedron edited
https://github.com/llvm/llvm-project/pull/127098
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2245,6 +2245,36 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
break;
}
+ case Builtin::BI__builtin_hlsl_and: {
+if (SemaRef.checkArgCount(TheCall, 2))
+ return true;
+if (CheckVectorElementCallArgs(&SemaRef, TheCal
https://github.com/farzonl requested changes to this pull request.
Codgen and testing is fine. SemaHLSL needs work to conform to how we have been
doing things.
https://github.com/llvm/llvm-project/pull/127098
___
cfe-commits mailing list
cfe-commits@
@@ -2245,6 +2245,36 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
break;
}
+ case Builtin::BI__builtin_hlsl_and: {
+if (SemaRef.checkArgCount(TheCall, 2))
+ return true;
+if (CheckVectorElementCallArgs(&SemaRef, TheCal
@@ -2245,6 +2245,36 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
break;
}
+ case Builtin::BI__builtin_hlsl_and: {
+if (SemaRef.checkArgCount(TheCall, 2))
+ return true;
+if (CheckVectorElementCallArgs(&SemaRef, TheCal
@@ -4765,6 +4765,12 @@ def HLSLAll : LangBuiltin<"HLSL_LANG"> {
let Prototype = "bool(...)";
}
+def HLSLAnd : LangBuiltin<"HLSL_LANG"> {
+ let Spellings = ["__builtin_hlsl_and"];
farzonl wrote:
I suppose the reason there is no c++ builtin existing here is
@@ -19463,6 +19463,11 @@ Value *CodeGenFunction::EmitHLSLBuiltinExpr(unsigned
BuiltinID,
CGM.getHLSLRuntime().getAllIntrinsic(), ArrayRef{Op0},
nullptr,
"hlsl.all");
}
+ case Builtin::BI__builtin_hlsl_and: {
+Value *Op0 = EmitScalarExpr(E->getArg(0));
+
@@ -1,9 +1,13 @@
// RUN: %clangxx %s -### -o %t.o --target=amd64-unknown-freebsd
-stdlib=platform 2>&1 \
// RUN: | FileCheck --check-prefix=CHECK-DEFAULT %s
// RUN: %clangxx %s -### -o %t.o --target=amd64-unknown-freebsd10.0
-stdlib=platform 2>&1 \
-// RUN: | FileCheck --
https://github.com/MaskRay approved this pull request.
code/test looks good for me as a driver maintainer. Needs a stamp from a
FreeBSD reviewer.
https://github.com/llvm/llvm-project/pull/126302
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/farzonl approved this pull request.
https://github.com/llvm/llvm-project/pull/127346
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/llvm-beanz approved this pull request.
https://github.com/llvm/llvm-project/pull/127098
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/llvm-beanz updated
https://github.com/llvm/llvm-project/pull/127346
>From e62dc4bfc4f1cff2a624caf70fcc7bb0dc4a6236 Mon Sep 17 00:00:00 2001
From: Chris Bieneman
Date: Sat, 15 Feb 2025 14:34:05 -0600
Subject: [PATCH 1/2] [HLSL] Disallow virtual inheritance and functions
This
https://github.com/tstellar updated
https://github.com/llvm/llvm-project/pull/127268
>From a705acfd439e6310648b0ed1ad612eaffc408ad5 Mon Sep 17 00:00:00 2001
From: Tom Stellard
Date: Fri, 14 Feb 2025 17:46:25 +
Subject: [PATCH 1/6] [CMake][Release] Statically link clang with stage1
runtimes
https://github.com/changkhothuychung updated
https://github.com/llvm/llvm-project/pull/120920
error: too big or took too long to generate
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
https://github.com/Andrewyuan34 updated
https://github.com/llvm/llvm-project/pull/127162
>From c6a732cb59340adfc045196c9c27ad9b2227c377 Mon Sep 17 00:00:00 2001
From: Andrewyuan34
Date: Thu, 13 Feb 2025 22:35:36 -0500
Subject: [PATCH] [clang-tidy] Fix invalid fixit from modernize-use-ranges for
bsdjhb wrote:
> It's not missing as in like someone forgot. It is intentional. Same goes for
> FreeBSD and OpenBSD's Drivers. The base OS uses libc++ and anything else is
> not supported.
You could use it with a libstdc++ from ports. GCC supports both variants for
the same reason. FreeBSD's
https://github.com/Andrewyuan34 updated
https://github.com/llvm/llvm-project/pull/127162
>From 101ec5ef672d385d91d35c23e3472b7b1d91bc15 Mon Sep 17 00:00:00 2001
From: Andrewyuan34
Date: Thu, 13 Feb 2025 22:35:36 -0500
Subject: [PATCH 1/2] [clang-tidy] Fix invalid fixit from modernize-use-ranges
@@ -109,6 +109,9 @@ Changes in existing checks
- Improved :doc:`misc-redundant-expression
` check by providing additional
examples and fixing some macro related false positives.
+
+- Improved :doc:`modernize-use-ranges
+ ` check by updating suppress
warnings logic for `
arichardson wrote:
ping, is this ok to merge?
https://github.com/llvm/llvm-project/pull/126302
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
CLANG_INSTALL_LIBDIR_BASENAME can be "lib" or "lib64". I think it should only
be used for libraries outside of llvm-project, like how cuda/amdgpu libs are
installed by system. For llvm-project installed libs, it's much better to stick
with a uniform "lib", which is easy to test
llvmbot wrote:
@llvm/pr-subscribers-hlsl
Author: Chris B (llvm-beanz)
Changes
This PR disallows virtual inheritance and virtual functions in HLSL.
---
Full diff: https://github.com/llvm/llvm-project/pull/127346.diff
4 Files Affected:
- (modified) clang/include/clang/Basic/DiagnosticPar
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 50581ef1ee45815b9230043319de5ae93680d4ad
e62dc4bfc4f1cff2a624caf70fcc7bb0dc4a6236 --e
Author: Chris Bieneman
Date: 2025-02-15T14:19:31-06:00
New Revision: 50581ef1ee45815b9230043319de5ae93680d4ad
URL:
https://github.com/llvm/llvm-project/commit/50581ef1ee45815b9230043319de5ae93680d4ad
DIFF:
https://github.com/llvm/llvm-project/commit/50581ef1ee45815b9230043319de5ae93680d4ad.diff
https://github.com/llvm-beanz created
https://github.com/llvm/llvm-project/pull/127346
This PR disallows virtual inheritance and virtual functions in HLSL.
>From e62dc4bfc4f1cff2a624caf70fcc7bb0dc4a6236 Mon Sep 17 00:00:00 2001
From: Chris Bieneman
Date: Sat, 15 Feb 2025 14:34:05 -0600
Subject
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-ppc64le-rhel`
running on `ppc64le-clang-rhel-test` while building `clang` at step 5
"build-unified-tree".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/145/builds/5160
Here is the relevant pi
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `sanitizer-ppc64le-linux`
running on `ppc64le-sanitizer` while building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/72/builds/8263
Here is the relevant piece of the bu
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`ppc64le-lld-multistage-test` running on `ppc64le-lld-multistage-test` while
building `clang` at step 12 "build-stage2-unified-tree".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/168/builds/8747
He
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
@llvm/pr-subscribers-clang
Author: Michał Górny (mgorny)
Changes
Fix `tools::addFortranRuntimeLibraryPath` to use
`CLANG_INSTALL_LIBDIR_BASENAME` rather than hardwired `lib` directory. This
fixes flang being unable to find its runtime when
https://github.com/mgorny created
https://github.com/llvm/llvm-project/pull/127345
Fix `tools::addFortranRuntimeLibraryPath` to use
`CLANG_INSTALL_LIBDIR_BASENAME` rather than hardwired `lib` directory. This
fixes flang being unable to find its runtime when installed into nonstandard
directory
https://github.com/changkhothuychung updated
https://github.com/llvm/llvm-project/pull/120920
error: too big or took too long to generate
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
changkhothuychung wrote:
> @changkhothuychung The be more specific. You should add the paper entry here:
>
> https://github.com/llvm/llvm-project/blob/4887e41055686eede9c155e6b3296b92fe86c2d5/libcxx/docs/ReleaseNotes/21.rst?plain=1#L41
>
>
> And update the status accordingly:
> https://github.
changkhothuychung wrote:
> #105419
I think let's address LWG4082 in another PR for better tracking?
https://github.com/llvm/llvm-project/pull/120920
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listin
https://github.com/changkhothuychung updated
https://github.com/llvm/llvm-project/pull/120920
error: too big or took too long to generate
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
https://github.com/changkhothuychung updated
https://github.com/llvm/llvm-project/pull/120920
error: too big or took too long to generate
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
https://github.com/llvm-beanz closed
https://github.com/llvm/llvm-project/pull/123141
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/changkhothuychung updated
https://github.com/llvm/llvm-project/pull/120920
error: too big or took too long to generate
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-m68k-linux-cross`
running on `suse-gary-m68k-cross` while building `clang` at step 4 "build stage
1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/27/builds/6180
Here is the relevant piece o
Author: Ryosuke Niwa
Date: 2025-02-15T11:04:06-08:00
New Revision: c17df0af23c941cd4fc97851ea51c91eee7c49e4
URL:
https://github.com/llvm/llvm-project/commit/c17df0af23c941cd4fc97851ea51c91eee7c49e4
DIFF:
https://github.com/llvm/llvm-project/commit/c17df0af23c941cd4fc97851ea51c91eee7c49e4.diff
rniwa wrote:
Thanks for the timely review!
https://github.com/llvm/llvm-project/pull/127309
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rniwa closed https://github.com/llvm/llvm-project/pull/127309
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
alexrp wrote:
Ping; would like to get this into LLVM 20.
https://github.com/llvm/llvm-project/pull/119997
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
yronglin wrote:
> Well, I don't really know much about these default init exprs but your
> explanation looks reasonable. I'd say, we should try it and revert it again
> if there are still some problems.
Many thanks for the review!
https://github.com/llvm/llvm-project/pull/127338
_
yronglin wrote:
I don't know whey we stripped off the top level `FullExpr` and `ConstantExpr`
in `CXXDefaultArgExpr::getExpr`, but not in `CXXDefaultInitExpr::getExpr`. Do
you know about this? @cor3ntin
https://github.com/llvm/llvm-project/pull/127338
_
https://github.com/steakhal approved this pull request.
Well, I don't really know much about these default init exprs but your
explanation looks reasonable. I'd say, we should try it and revert it again if
there are still some problems.
https://github.com/llvm/llvm-project/pull/127338
=?utf-8?q?Balázs_Kéri?=
Message-ID:
In-Reply-To:
@@ -129,6 +129,8 @@ The ``SuppressAddressSpaces`` option suppresses
warnings for null dereferences of all pointers with address spaces. You can
disable this behavior with the option
``-analyzer-config core.NullDereference:Supp
=?utf-8?q?Bal=C3=A1zs_K=C3=A9ri?=
Message-ID:
In-Reply-To:
@@ -285,6 +283,12 @@ def FixedAddressChecker : Checker<"FixedAddr">,
HelpText<"Check for assignment of a fixed address to a pointer">,
Documentation;
+def FixedAddressDereferenceChecker
+: Checker<"FixedAddr
=?utf-8?q?Balázs_Kéri?=
Message-ID:
In-Reply-To:
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/127191
___
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?=
Message-ID:
In-Reply-To:
@@ -1,6 +1,6 @@
// NOTE: Use '-fobjc-gc' to test the analysis being run twice, and multiple
reports are not issued.
-// RUN: %clang_analyze_cc1 -triple i386-apple-darwin10
-analyzer-checker=core,alpha.core,osx.cocoa.AtSync -Wn
=?utf-8?q?Bal=C3=A1zs_K=C3=A9ri?=
Message-ID:
In-Reply-To:
@@ -2919,6 +2921,42 @@ Check for assignment of a fixed address to a pointer.
p = (int *) 0x1; // warn
}
+.. _alpha-core-FixedAddressDereference:
+
+alpha.core.FixedAddressDereference (C, C++, ObjC)
+
=?utf-8?q?Bal=C3=A1zs_K=C3=A9ri?=
Message-ID:
In-Reply-To:
@@ -31,7 +31,12 @@ class DereferenceChecker
: public Checker< check::Location,
check::Bind,
EventDispatcher > {
- enum DerefKind { NullPointer, UndefinedPointerValue,
=?utf-8?q?Balázs_Kéri?=
Message-ID:
In-Reply-To:
@@ -155,30 +162,47 @@ static bool isDeclRefExprToReference(const Expr *E) {
void DereferenceChecker::reportBug(DerefKind K, ProgramStateRef State,
const Stmt *S, CheckerContext &C) const {
-
=?utf-8?q?Bal=C3=A1zs_K=C3=A9ri?=
Message-ID:
In-Reply-To:
https://github.com/steakhal requested changes to this pull request.
Looks pretty good. Thank you!
https://github.com/llvm/llvm-project/pull/127191
___
cfe-commits mailing list
cfe-commits@li
=?utf-8?q?Balázs_Kéri?=
Message-ID:
In-Reply-To:
@@ -211,17 +211,15 @@ def DereferenceModeling : Checker<"DereferenceModeling">,
Documentation,
Hidden;
-def NullDereferenceChecker : Checker<"NullDereference">,
- HelpText<"Check for dereferences of null pointers">,
- C
=?utf-8?q?Bal=C3=A1zs_K=C3=A9ri?=
Message-ID:
In-Reply-To:
@@ -2919,6 +2921,42 @@ Check for assignment of a fixed address to a pointer.
p = (int *) 0x1; // warn
}
+.. _alpha-core-FixedAddressDereference:
+
+alpha.core.FixedAddressDereference (C, C++, ObjC)
+
=?utf-8?q?Balázs_Kéri?=
Message-ID:
In-Reply-To:
@@ -1,7 +1,163 @@
// RUN: %clang_analyze_cc1 -analyzer-checker=core,alpha.core -verify %s
-// expected-no-diagnostics
-void foo(void) {
+extern void __assert_fail (__const char *__assertion, __const char *__file,
+unsigned
=?utf-8?q?Bal=C3=A1zs_K=C3=A9ri?=
Message-ID:
In-Reply-To:
@@ -2919,6 +2921,42 @@ Check for assignment of a fixed address to a pointer.
p = (int *) 0x1; // warn
}
+.. _alpha-core-FixedAddressDereference:
+
+alpha.core.FixedAddressDereference (C, C++, ObjC)
+
@@ -2464,6 +2464,10 @@ CFGBlock
*CFGBuilder::VisitCXXDefaultInitExpr(CXXDefaultInitExpr *Init,
autoCreateBlock();
appendStmt(Block, Init);
}
+
+// Unlike CXXDefaultArgExpr::getExpr stripped off the top level FullExpr
and
+// ConstantExpr, CXXDefaultIni
https://github.com/yronglin updated
https://github.com/llvm/llvm-project/pull/127338
>From 092af69607909859e9d352e51122d9569c050464 Mon Sep 17 00:00:00 2001
From: yronglin
Date: Sat, 15 Feb 2025 17:03:32 +0800
Subject: [PATCH 1/2] [Analyzer][CFG] Correctly handle rebuilt default arg and
defaul
@@ -2464,6 +2464,10 @@ CFGBlock
*CFGBuilder::VisitCXXDefaultInitExpr(CXXDefaultInitExpr *Init,
autoCreateBlock();
appendStmt(Block, Init);
}
+
+// Unlike CXXDefaultArgExpr::getExpr stripped off the top level FullExpr
and
+// ConstantExpr, CXXDefaultIni
1 - 100 of 177 matches
Mail list logo
