https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/101788
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2024-08-03T08:25:44+02:00
New Revision: 3eaca31e4607161c0b878c242c040de2cdadcced
URL:
https://github.com/llvm/llvm-project/commit/3eaca31e4607161c0b878c242c040de2cdadcced
DIFF:
https://github.com/llvm/llvm-project/commit/3eaca31e4607161c0b878c242c040de2cdadcced.diff
L
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-hip-vega20` running
on `hip-vega20-0` while building `clang` at step 3 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/123/builds/3135
Here is the relevant piece of the build log for
CoTinker wrote:
I don't know, but maybe Eli can give you an answer.
https://github.com/llvm/llvm-project/pull/86388
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-mc
Author: Freddy Ye (FreddyLeaf)
Changes
Ref.: https://cdrdv2.intel.com/v1/dl/getContent/828965
---
Patch is 307.16 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/101598.diff
25 Files Affected:
- (modifi
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Freddy Ye (FreddyLeaf)
Changes
Ref.: https://cdrdv2.intel.com/v1/dl/getContent/828965
---
Patch is 307.16 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/101598.diff
25 Files Affected:
- (mod
llvmbot wrote:
@llvm/pr-subscribers-llvm-ir
Author: Freddy Ye (FreddyLeaf)
Changes
Ref.: https://cdrdv2.intel.com/v1/dl/getContent/828965
---
Patch is 307.16 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/101598.diff
25 Files Affected:
- (m
https://github.com/FreddyLeaf ready_for_review
https://github.com/llvm/llvm-project/pull/101598
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2024-08-03T08:04:24+02:00
New Revision: 5d64b37f3403dc1683bd4f8166f4286e6562c6cf
URL:
https://github.com/llvm/llvm-project/commit/5d64b37f3403dc1683bd4f8166f4286e6562c6cf
DIFF:
https://github.com/llvm/llvm-project/commit/5d64b37f3403dc1683bd4f8166f4286e6562c6cf.diff
L
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/101794
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr edited
https://github.com/llvm/llvm-project/pull/101788
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/101788
>From 5cb12152ccb6e2dbdf09c87ae7562ccd7ca6ff57 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 2 Aug 2024 19:05:04 +0200
Subject: [PATCH] [clang][Interp] Simplify and fix variable scope
https://github.com/tbaederr edited
https://github.com/llvm/llvm-project/pull/101794
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jcsxky closed https://github.com/llvm/llvm-project/pull/79084
___
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: Timm Baeder (tbaederr)
Changes
This doesn't fix the attached test case, but at least we're not crashing
anymore.
---
Full diff: https://github.com/llvm/llvm-project/pull/101794.diff
2 Files Affected:
- (modified) clang/lib/AST/Interp/
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/101794
This doesn't fix the attached test case, but at least we're not crashing
anymore.
>From 4fb09d3f10635de87742265ea2e1da12b1ad3679 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Sat, 3 Aug
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/101792
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2024-08-03T07:16:21+02:00
New Revision: ea18a40ed920b11ef35c3cfdd7e5f0b4bc796da4
URL:
https://github.com/llvm/llvm-project/commit/ea18a40ed920b11ef35c3cfdd7e5f0b4bc796da4
DIFF:
https://github.com/llvm/llvm-project/commit/ea18a40ed920b11ef35c3cfdd7e5f0b4bc796da4.diff
L
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/101787
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2024-08-03T07:11:27+02:00
New Revision: d631ebb7562d48d9e3d0f9ac0543364587d470b6
URL:
https://github.com/llvm/llvm-project/commit/d631ebb7562d48d9e3d0f9ac0543364587d470b6
DIFF:
https://github.com/llvm/llvm-project/commit/d631ebb7562d48d9e3d0f9ac0543364587d470b6.diff
L
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heide
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heide
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/101792.diff
4 Files Affected:
- (modified) clang/lib/AST/Interp/Interp.cpp (+6)
- (modified) clang/lib/AST/Interp/Pointer.cpp (+6)
- (modified) c
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/101792
None
>From 08aaea88863ad000d68316391a8702cd777496af Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Sat, 3 Aug 2024 06:37:24 +0200
Subject: [PATCH] [clang][Interp] Check for 'delete this' in
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/101788.diff
9 Files Affected:
- (modified) clang/lib/AST/Interp/Compiler.cpp (+15-48)
- (modifi
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
Add more relational operators.
---
Full diff: https://github.com/llvm/llvm-project/pull/101787.diff
1 Files Affected:
- (modified) clang/lib/AST/Interp/Source.h (+9-4)
``diff
diff --git a/clang/
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/101788
None
>From 02ba0cd327c91d65ac6d4ee7699a590cea6a92ec Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 2 Aug 2024 17:45:19 +0200
Subject:
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/101787
Add more relational operators.
>From 02ba0cd327c91d65ac6d4ee7699a590cea6a92ec Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 2 Aug 2024 17:45:19 +0200
Subject: [PATCH] [clang][Interp]
https://github.com/bob80905 closed
https://github.com/llvm/llvm-project/pull/101256
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HenryZ16 closed
https://github.com/llvm/llvm-project/pull/101307
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`openmp-offload-libc-amdgpu-runtime` running on `omp-vega20-1` while building
`clang` at step 10 "Add check check-offload".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/73/builds/3167
Here is the re
Author: Timm Bäder
Date: 2024-08-03T05:45:00+02:00
New Revision: 53e87908c67f158bfe196a3c7cec690dc5eed1fc
URL:
https://github.com/llvm/llvm-project/commit/53e87908c67f158bfe196a3c7cec690dc5eed1fc
DIFF:
https://github.com/llvm/llvm-project/commit/53e87908c67f158bfe196a3c7cec690dc5eed1fc.diff
LO
Author: Timm Bäder
Date: 2024-08-03T05:44:59+02:00
New Revision: 4d2c9d8cd849e8097f41b3c3b52e4475188b5489
URL:
https://github.com/llvm/llvm-project/commit/4d2c9d8cd849e8097f41b3c3b52e4475188b5489
DIFF:
https://github.com/llvm/llvm-project/commit/4d2c9d8cd849e8097f41b3c3b52e4475188b5489.diff
LO
Author: Timm Bäder
Date: 2024-08-03T05:44:59+02:00
New Revision: 13faed8737b3021e59c3df6db3066876ce831dc9
URL:
https://github.com/llvm/llvm-project/commit/13faed8737b3021e59c3df6db3066876ce831dc9
DIFF:
https://github.com/llvm/llvm-project/commit/13faed8737b3021e59c3df6db3066876ce831dc9.diff
LO
https://github.com/mizvekov closed
https://github.com/llvm/llvm-project/pull/101395
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Matheus Izvekov
Date: 2024-08-03T00:36:48-03:00
New Revision: 1299af66322aa21246b0de6204dff553e46c408c
URL:
https://github.com/llvm/llvm-project/commit/1299af66322aa21246b0de6204dff553e46c408c
DIFF:
https://github.com/llvm/llvm-project/commit/1299af66322aa21246b0de6204dff553e46c408c.dif
https://github.com/mizvekov updated
https://github.com/llvm/llvm-project/pull/101395
>From 9e2dc4d465f2dc3d31bace2fca95b0656538b655 Mon Sep 17 00:00:00 2001
From: Matheus Izvekov
Date: Wed, 31 Jul 2024 16:54:02 -0300
Subject: [PATCH] [clang] create class-type injected NTTP with correct value
k
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (TsXor)
Changes
Related: https://github.com/llvm/llvm-project/issues/76664
I used metadata reflection so that we can import C library functions just by
declaring annotated python functions. This makes C function types visible to
typ
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/zyn0217 edited
https://github.com/llvm/llvm-project/pull/101020
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zyn0217 approved this pull request.
Makes sense to me. Thanks
https://github.com/llvm/llvm-project/pull/101395
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jamieschmeiser wrote:
@rnk It is just a convenience to make things easier to read and should be
removed if it is causing compile time issues. I just happened to see this
comment now but will be unavailable for about 2 weeks. I will take care of it
when I return (I'm not ignoring this...) If
@@ -596,6 +600,90 @@ bool BPFMIPreEmitPeephole::adjustBranch() {
return Changed;
}
+static const unsigned CallerSavedRegs[] = {BPF::R0, BPF::R1, BPF::R2,
+ BPF::R3, BPF::R4, BPF::R5};
+
+struct BPFFastCall {
+ MachineInstr *MI;
+ u
tbaederr wrote:
In https://github.com/llvm/llvm-project/pull/74803, the question of "why" came
up and was never answered.
https://github.com/llvm/llvm-project/pull/101689
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/c
Author: Timm Bäder
Date: 2024-08-03T04:59:16+02:00
New Revision: 400c7c7cf2b15138aa674355983be74270de93bc
URL:
https://github.com/llvm/llvm-project/commit/400c7c7cf2b15138aa674355983be74270de93bc
DIFF:
https://github.com/llvm/llvm-project/commit/400c7c7cf2b15138aa674355983be74270de93bc.diff
LO
llvmbot wrote:
@llvm/pr-subscribers-mc
Author: Phoebe Wang (phoebewang)
Changes
Ref.: https://cdrdv2.intel.com/v1/dl/getContent/828965
---
Patch is 445.25 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/101783.diff
28 Files Affected:
- (modi
@@ -361,6 +368,13 @@ CodeGenFunction::AddInitializerToStaticVarDecl(const
VarDecl &D,
}
return GV;
}
+ if (!getLangOpts().CPlusPlus) {
+// In C, when an initializer is given, the Linux kernel relies on clang to
+// zero-initialize all members not explicitly
@@ -664,11 +664,25 @@ LinkageComputer::getLVForNamespaceScopeDecl(const
NamedDecl *D,
if (PrevVar->getStorageClass() == SC_Static)
return LinkageInfo::internal();
}
+
+if (Context.getLangOpts().HLSL &&
+Var->hasAttr())
+ return LinkageInfo::i
https://github.com/topperc closed
https://github.com/llvm/llvm-project/pull/101733
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Craig Topper
Date: 2024-08-02T18:44:43-07:00
New Revision: 6e70ca1e802f26b4c9cbe90c11b2c5317789ed23
URL:
https://github.com/llvm/llvm-project/commit/6e70ca1e802f26b4c9cbe90c11b2c5317789ed23
DIFF:
https://github.com/llvm/llvm-project/commit/6e70ca1e802f26b4c9cbe90c11b2c5317789ed23.diff
https://github.com/yabinc updated
https://github.com/llvm/llvm-project/pull/97121
>From e1d75774c554a5125428b3f3a6304a921c81c5f0 Mon Sep 17 00:00:00 2001
From: Yabin Cui
Date: Thu, 27 Jun 2024 17:11:52 -0700
Subject: [PATCH 1/2] [clang][CodeGen] Zero init unspecified fields in
initializers in
https://github.com/4vtomat approved this pull request.
LGTM~
https://github.com/llvm/llvm-project/pull/101733
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/phoebewang closed
https://github.com/llvm/llvm-project/pull/101616
___
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 12189f800585ab459afa20b4f159db93ae474b57
e1d75774c554a5125428b3f3a6304a921c81c5f0 --e
https://github.com/yabinc updated
https://github.com/llvm/llvm-project/pull/97121
>From e1d75774c554a5125428b3f3a6304a921c81c5f0 Mon Sep 17 00:00:00 2001
From: Yabin Cui
Date: Thu, 27 Jun 2024 17:11:52 -0700
Subject: [PATCH] [clang][CodeGen] Zero init unspecified fields in initializers
in C (#
ziqingluo-90 wrote:
Though we probably don't need this warning anymore but this is really a great
catch of wording issues!
https://github.com/llvm/llvm-project/pull/101583
___
cfe-commits mailing list
cfe-commits@lis
@@ -0,0 +1,58 @@
+// NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py
UTC_ARGS: --version 5
+// REQUIRES: powerpc-registered-target
+// RUN: %clang_cc1 -triple powerpc64le-unknown-linux -O2 -target-cpu pwr7 \
+// RUN: -emit-llvm %s -o - | FileCheck %s
@@ -0,0 +1,10 @@
+; RUN: not opt -S -dxil-op-lower -mtriple=dxil-pc-shadermodel6.3-library %s
2>&1 | FileCheck %s
+
+; DXIL operation length does not support scalar types
+; CHECK: error: invalid intrinsic signature
+
+define noundef float @test_length_float(float noundef %p0) {
https://github.com/bob80905 closed
https://github.com/llvm/llvm-project/pull/101433
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jyu2-git closed
https://github.com/llvm/llvm-project/pull/101101
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zygoloid commented:
Changes to the algorithm for selecting the most constrained function look
correct. It's not clear to me whether it's correct to stop calling
`CheckMemberSpecialization` for member templates, though -- do we still do the
necessary checks somewhere else?
h
@@ -7964,8 +7964,9 @@ NamedDecl *Sema::ActOnVariableDeclarator(
D.setRedeclaration(CheckVariableDeclaration(NewVD, Previous));
} else {
// If this is an explicit specialization of a static data member, check it.
-if (IsMemberSpecialization && !IsVariableTemplateSp
@@ -10465,7 +10466,7 @@ Sema::ActOnFunctionDeclarator(Scope *S, Declarator &D,
DeclContext *DC,
Previous))
NewFD->setInvalidDecl();
}
-} else if (isMemberSpecialization && isa(NewFD)) {
+} else if (isMembe
https://github.com/zygoloid edited
https://github.com/llvm/llvm-project/pull/101721
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jinge90 approved this pull request.
https://github.com/llvm/llvm-project/pull/99335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jtb20 wrote:
Thank you for the review! Again, I do not have commit access, so please apply
for me if possible.
Julian
https://github.com/llvm/llvm-project/pull/92731
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-b
@@ -0,0 +1,10 @@
+; RUN: not opt -S -dxil-op-lower -mtriple=dxil-pc-shadermodel6.3-library %s
2>&1 | FileCheck %s
+
+; DXIL operation length does not support scalar types
+; CHECK: error: invalid intrinsic signature
+
+define noundef float @test_length_float(float noundef %p0) {
https://github.com/bogner approved this pull request.
https://github.com/llvm/llvm-project/pull/101256
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -664,11 +664,25 @@ LinkageComputer::getLVForNamespaceScopeDecl(const
NamedDecl *D,
if (PrevVar->getStorageClass() == SC_Static)
return LinkageInfo::internal();
}
+
+if (Context.getLangOpts().HLSL &&
+Var->hasAttr())
+ return LinkageInfo::i
@@ -354,6 +354,23 @@ llvm::Value *CGHLSLRuntime::emitInputSemantic(IRBuilder<>
&B,
return nullptr;
}
+void CGHLSLRuntime::emitFunctionProlog(const FunctionDecl *FD,
llvm-beanz wrote:
I think this is poorly named. It isn't emitting a function prolog it is e
rnk wrote:
Are we sure we want this behavior? Demangling is really expensive. A user
noticed a significant (10%) compile time regression from this change. We could
skip the demangling and do it offline. It's also worth pointing out that we
have to redo this work every time we optimize the same
@@ -6763,15 +6763,18 @@ attribute requires a string literal argument to
identify the handle being releas
def UnsafeBufferUsageDocs : Documentation {
let Category = DocCatFunction;
let Content = [{
-The attribute ``[[clang::unsafe_buffer_usage]]`` should be placed on functi
@@ -927,21 +927,28 @@ class CArrayToPtrAssignmentGadget : public FixableGadget {
/// over one of its pointer parameters.
class UnsafeBufferUsageAttrGadget : public WarningGadget {
constexpr static const char *const OpTag = "call_expr";
- const CallExpr *Op;
+ const Expr *Op
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Mike Rice (mikerice1969)
Changes
In 2acf77f987331c05520c5bfd849326909ffce983 code was added to use the 'full'
name including syntax and scope.
Instead of building up a large string for each name, add syntax and scope
checks to the value
@@ -927,21 +927,28 @@ class CArrayToPtrAssignmentGadget : public FixableGadget {
/// over one of its pointer parameters.
class UnsafeBufferUsageAttrGadget : public WarningGadget {
constexpr static const char *const OpTag = "call_expr";
- const CallExpr *Op;
+ const Expr *Op
https://github.com/mikerice1969 created
https://github.com/llvm/llvm-project/pull/101768
In 2acf77f987331c05520c5bfd849326909ffce983 code was added to use the 'full'
name including syntax and scope.
Instead of building up a large string for each name, add syntax and scope
checks to the value
@@ -927,21 +927,28 @@ class CArrayToPtrAssignmentGadget : public FixableGadget {
/// over one of its pointer parameters.
class UnsafeBufferUsageAttrGadget : public WarningGadget {
constexpr static const char *const OpTag = "call_expr";
- const CallExpr *Op;
+ const Expr *Op
@@ -2261,6 +2262,12 @@ class UnsafeBufferUsageReporter : public
UnsafeBufferUsageHandler {
// note_unsafe_buffer_operation doesn't have this mode yet.
assert(!IsRelatedToDecl && "Not implemented yet!");
MsgParam = 3;
+ } else if (isa(Operation)) {
@@ -0,0 +1,113 @@
+// RUN: %clang_cc1 -std=c++20 -Wunsafe-buffer-usage \
+// RUN:-fsafe-buffer-usage-suggestions -verify %s
+
+using size_t = __typeof(sizeof(int));
+
+namespace std {
+ class type_info;
+ class bad_cast;
+ class bad_typeid;
+
+ template class span
@@ -0,0 +1,113 @@
+// RUN: %clang_cc1 -std=c++20 -Wunsafe-buffer-usage \
+// RUN:-fsafe-buffer-usage-suggestions -verify %s
+
+using size_t = __typeof(sizeof(int));
+
+namespace std {
+ class type_info;
+ class bad_cast;
+ class bad_typeid;
+
+ template class span
kimgr wrote:
> The exported target is new and hasn't been in a release yet. It should be in
> LLVM-19 once it's released and Debian gets updated. Of course, this is also
> true of any other new thing that gets added now anyway.
👍
> I am a little confused about what your goal is with the path
@@ -959,12 +966,12 @@ class UnsafeBufferUsageAttrGadget : public WarningGadget {
/// perform buffer operations that depend on the correctness of the parameters.
class UnsafeBufferUsageCtorAttrGadget : public WarningGadget {
constexpr static const char *const OpTag = "cxx_cons
https://github.com/ilovepi approved this pull request.
LGTM, but test locally w/ LLVM_ENABLE_EXPENSIVE_CHECKS=On
https://llvm.org/docs/CMake.html. If that passes, then go ahead and land this,
since I think the nondeterminism in output order will be accounted for.
https://github.com/llvm/llvm-p
https://github.com/yxsamliu closed
https://github.com/llvm/llvm-project/pull/101341
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Yaxun (Sam) Liu
Date: 2024-08-02T18:37:22-04:00
New Revision: db1d3b23a37c7a57fa8b9e5bc94e1b22e278d361
URL:
https://github.com/llvm/llvm-project/commit/db1d3b23a37c7a57fa8b9e5bc94e1b22e278d361
DIFF:
https://github.com/llvm/llvm-project/commit/db1d3b23a37c7a57fa8b9e5bc94e1b22e278d361.dif
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Jessica Clarke (jrtc27)
Changes
ASTContext::getIntWidth returns 1 if isBooleanType(), and falls back on
getTypeSize in the default case, which itself just returns the Width
from getTypeInfo's returned struct, so can be used in all c
https://github.com/jrtc27 created
https://github.com/llvm/llvm-project/pull/101765
ASTContext::getIntWidth returns 1 if isBooleanType(), and falls back on
getTypeSize in the default case, which itself just returns the Width
from getTypeInfo's returned struct, so can be used in all cases here,
no
etcwilde wrote:
> We can't really help you with whatever is wrong with Debian's packages unless
> it is a bug caused by our build configurations. The change you're proposing
> in this PR could just as easily be impacted in some distro's packages.
The exported target is new and hasn't been in a
https://github.com/vsapsai closed
https://github.com/llvm/llvm-project/pull/101762
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Volodymyr Sapsai
Date: 2024-08-02T19:15:47-03:00
New Revision: d9f786fd13fe03256ef0f2983ecd379d0e7e8c93
URL:
https://github.com/llvm/llvm-project/commit/d9f786fd13fe03256ef0f2983ecd379d0e7e8c93
DIFF:
https://github.com/llvm/llvm-project/commit/d9f786fd13fe03256ef0f2983ecd379d0e7e8c93.di
kimgr wrote:
@llvm-beanz Thanks for clarifying! Yeah, I no longer think this PR has anything
to offer. If you think it's future-proof to assume the interface include dir of
clang-resource-headers is one level down from the resource dir, that works to
polish and seed it into a hardwired `-resou
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-modules
Author: Volodymyr Sapsai (vsapsai)
Changes
Fix the false warning
> incompatible pointer types passing 'va_list' (aka '__builtin_va_list') to
parameter of type 'struct __va_list_tag *' [-Wincompatible-pointer-type
https://github.com/vsapsai created
https://github.com/llvm/llvm-project/pull/101762
Fix the false warning
> incompatible pointer types passing 'va_list' (aka '__builtin_va_list') to
> parameter of type 'struct __va_list_tag *' [-Wincompatible-pointer-types]
The warning is wrong because both in
https://github.com/bogner approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/97362
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3636,6 +3636,10 @@ class IsTypeDeclaredInsideVisitor
return {};
}
+ std::optional VisitUnaryTransformType(const UnaryTransformType *T) {
+return CheckType(T->getBaseType());
mizvekov wrote:
Strictly speaking, you only need to visit the base typ
https://github.com/mizvekov approved this pull request.
LGTM
You can limit the amount of nodes traversed, but I don't expect this will have
a significant impact.
https://github.com/llvm/llvm-project/pull/101517
___
cfe-commits mailing list
cfe-commit
https://github.com/mizvekov edited
https://github.com/llvm/llvm-project/pull/101517
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/banach-space approved this pull request.
https://github.com/llvm/llvm-project/pull/101682
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jyu2-git edited
https://github.com/llvm/llvm-project/pull/101101
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jyu2-git wrote:
> LG with a nit
Thanks Alexey for the code review!!
https://github.com/llvm/llvm-project/pull/101101
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 428 matches
Mail list logo
