Author: Matheus Izvekov
Date: 2024-06-10T10:29:41-03:00
New Revision: 3f6fe4efbef610c306eb6c71d6f0428fb6d7ad68
URL:
https://github.com/llvm/llvm-project/commit/3f6fe4efbef610c306eb6c71d6f0428fb6d7ad68
DIFF:
https://github.com/llvm/llvm-project/commit/3f6fe4efbef610c306eb6c71d6f0428fb6d7ad68.dif
https://github.com/mizvekov closed
https://github.com/llvm/llvm-project/pull/94905
___
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 e36c8dca38ae30e4cf22a39ff32a050e110a2247
02859cb59f152567e0d107c1e2f4e3e6965d1908 --
https://github.com/DeinAlptraum edited
https://github.com/llvm/llvm-project/pull/78114
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Tomas Matheson
Date: 2024-06-10T14:47:23+01:00
New Revision: 1b13bc05fe4a3b7b4916387543f0a64d41909e83
URL:
https://github.com/llvm/llvm-project/commit/1b13bc05fe4a3b7b4916387543f0a64d41909e83
DIFF:
https://github.com/llvm/llvm-project/commit/1b13bc05fe4a3b7b4916387543f0a64d41909e83.diff
https://github.com/ldionne updated
https://github.com/llvm/llvm-project/pull/94791
>From 61a434bae9f3787122e123540b7c379f410e037b Mon Sep 17 00:00:00 2001
From: Louis Dionne
Date: Fri, 31 May 2024 10:55:53 -0700
Subject: [PATCH 1/2] [libc++] Fix deployment target Lit features
We were not makin
PiotrZSL wrote:
Yes, you can just submit new PR, reference this one.
https://github.com/llvm/llvm-project/pull/90043
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/SidManning approved this pull request.
https://github.com/llvm/llvm-project/pull/94877
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mizvekov created
https://github.com/llvm/llvm-project/pull/94981
This extends default argument deduction to cover class templates as well, and
also applies outside of partial ordering, adding to the provisional wording
introduced in https://github.com/llvm/llvm-project/pull/
llvmbot wrote:
@llvm/pr-subscribers-clang-modules
Author: Matheus Izvekov (mizvekov)
Changes
This extends default argument deduction to cover class templates as well, and
also applies outside of partial ordering, adding to the provisional wording
introduced in https://github.com/llvm/llv
Author: Yaxun (Sam) Liu
Date: 2024-06-10T10:08:26-04:00
New Revision: 53d2f4d967838468423715178a8344739d7a63c9
URL:
https://github.com/llvm/llvm-project/commit/53d2f4d967838468423715178a8344739d7a63c9
DIFF:
https://github.com/llvm/llvm-project/commit/53d2f4d967838468423715178a8344739d7a63c9.dif
https://github.com/yxsamliu closed
https://github.com/llvm/llvm-project/pull/77359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman commented:
LGTM as far as the code goes, but please add some test coverage for the changes.
https://github.com/llvm/llvm-project/pull/94877
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/c
@@ -1462,6 +1472,16 @@ inline APFloat maximum(const APFloat &A, const APFloat
&B) {
return A.isNegative() ? B : A;
return A < B ? B : A;
}
+LLVM_READONLY
kuhar wrote:
also here
https://github.com/llvm/llvm-project/pull/93841
___
@@ -1449,6 +1449,16 @@ inline APFloat minimum(const APFloat &A, const APFloat
&B) {
return A.isNegative() ? A : B;
return B < A ? B : A;
}
+LLVM_READONLY
kuhar wrote:
Please add and an empty line before this function and document its semantics.
https:/
https://github.com/martinboehme created
https://github.com/llvm/llvm-project/pull/94983
The patch includes a repro for a case where we were returning a null `FieldDecl`
when calling `getReferencedDecls()` on the `InitListExpr` for a union.
Also, I noticed while working on this that `RecordInitL
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (martinboehme)
Changes
The patch includes a repro for a case where we were returning a null `FieldDecl`
when calling `getReferencedDecls()` on the `InitListExpr` for a union.
Also, I noticed while working on this that `RecordInitListH
https://github.com/shiltian approved this pull request.
https://github.com/llvm/llvm-project/pull/94931
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Alastair Houghton
Date: 2024-06-10T10:21:13-04:00
New Revision: fe1301b5d846e96655ae6981789f677de6c7572d
URL:
https://github.com/llvm/llvm-project/commit/fe1301b5d846e96655ae6981789f677de6c7572d
DIFF:
https://github.com/llvm/llvm-project/commit/fe1301b5d846e96655ae6981789f677de6c7572d.d
https://github.com/ldionne closed
https://github.com/llvm/llvm-project/pull/85097
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaronliu0130 wrote:
What's "ping PR"? Do you mean pinging the LLVM organization?
https://github.com/llvm/llvm-project/pull/78491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Xazax-hun approved this pull request.
https://github.com/llvm/llvm-project/pull/94983
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -5988,12 +5988,29 @@ CGCallee
CGCallee::prepareConcreteCallee(CodeGenFunction &CGF) const {
/* VarArg handling */
-Address CodeGenFunction::EmitVAArg(VAArgExpr *VE, Address &VAListAddr) {
- VAListAddr = VE->isMicrosoftABI()
- ? EmitMSVAListRef(VE->getSubE
@@ -1328,15 +1328,15 @@ void AggExprEmitter::VisitChooseExpr(const ChooseExpr
*CE) {
void AggExprEmitter::VisitVAArgExpr(VAArgExpr *VE) {
Address ArgValue = Address::invalid();
- Address ArgPtr = CGF.EmitVAArg(VE, ArgValue);
+ RValue ArgPtr = CGF.EmitVAArg(VE, ArgValue);
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 7573d5e4b10cc7befc54d29edd7ec94d9bf11b93
5a03b2a01a108708e953854f0b451cc49d6074d2 --
https://github.com/samolisov created
https://github.com/llvm/llvm-project/pull/94987
This addresses a clang-tidy suggestion.
>From 34352ddc7cb351c74ff3758c3990a480adc4c2c2 Mon Sep 17 00:00:00 2001
From: Pavel Samolysov
Date: Mon, 10 Jun 2024 17:35:14 +0300
Subject: [PATCH] [clang] Replace X &&
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Pavel Samolysov (samolisov)
Changes
This addresses a clang-tidy suggestion.
---
Patch is 24.54 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/94987.diff
29 Files Affected:
- (modified) clang
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 cc8fa1e9206aa69197c891ca2f17b64340c5a6aa
34352ddc7cb351c74ff3758c3990a480adc4c2c2 --
https://github.com/zyn0217 edited
https://github.com/llvm/llvm-project/pull/94889
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/samolisov updated
https://github.com/llvm/llvm-project/pull/94987
>From 691223b4e873257a74b295bfb77839406adc742a Mon Sep 17 00:00:00 2001
From: Pavel Samolysov
Date: Mon, 10 Jun 2024 17:35:14 +0300
Subject: [PATCH] [clang] Replace X && isa(X) with isa_and_nonnull(X).
NFC
Th
https://github.com/ymand approved this pull request.
Thanks! I'm once again struck by how much this project would benefit from
Nullability annotations/checking.
https://github.com/llvm/llvm-project/pull/94983
___
cfe-commits mailing list
cfe-commits@l
Author: Karl-Johan Karlsson
Date: 2024-06-10T16:47:37+02:00
New Revision: aa486ec39c3b4d641ecdc5f4a07e801963920022
URL:
https://github.com/llvm/llvm-project/commit/aa486ec39c3b4d641ecdc5f4a07e801963920022
DIFF:
https://github.com/llvm/llvm-project/commit/aa486ec39c3b4d641ecdc5f4a07e801963920022
https://github.com/karka228 closed
https://github.com/llvm/llvm-project/pull/94931
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zmodem wrote:
We're seeing the test fail on our mac builds:
https://issues.chromium.org/issues/346289767
```
JIT session error: Symbols not found: [ ___clang_Interpreter_SetValueNoAlloc ]
error: Failed to materialize symbols: { (main, {
orc_init_func.incr_module_10, _c_str, $.incr_module_
@@ -0,0 +1,18 @@
+
+!--
+! RUN lines
+!--
+! Embed something that can be easily checked
+! RUN: %flang_fc1 -emit-llvm -triple x86_64-unknown-linux-gnu -o -
-mlink-builtin-bitcode %S/Inputs/bclib.bc %s 2>&1 | FileCheck %s
jsjodin wrote:
The triple
@@ -1146,6 +1150,54 @@ void CodeGenAction::embedOffloadObjects() {
}
}
+void CodeGenAction::linkBuiltinBCLibs() {
jsjodin wrote:
It does come from clang but it is distributed across a few classes with a lot
more options, so there was not simple way to simp
@@ -302,7 +302,7 @@ void MangleContext::mangleBlock(const DeclContext *DC,
const BlockDecl *BD,
assert((isa(DC) || isa(DC)) &&
"expected a NamedDecl or BlockDecl");
if (isa(DC))
- for (; DC && isa(DC); DC = DC->getParent())
+ for (; isa_and_nonnull
jsjodin wrote:
Yes, it should be possible to add a command to generate the .bc file, that is
probably a better option anyway since we would avoid specifying the triple.
https://github.com/llvm/llvm-project/pull/94763
___
@@ -1199,7 +1244,9 @@ llvm::Function *CGNVCUDARuntime::finalizeModule() {
}
return nullptr;
}
- if (CGM.getLangOpts().OffloadingNewDriver && RelocatableDeviceCode)
+ if (CGM.getLangOpts().OffloadViaLLVM)
+createOffloadingEntries();
+ else if (CGM.getLangOpts().
https://github.com/jsjodin edited
https://github.com/llvm/llvm-project/pull/94763
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -302,7 +302,7 @@ void MangleContext::mangleBlock(const DeclContext *DC,
const BlockDecl *BD,
assert((isa(DC) || isa(DC)) &&
"expected a NamedDecl or BlockDecl");
if (isa(DC))
- for (; DC && isa(DC); DC = DC->getParent())
+ for (; isa_and_nonnull
jplehr wrote:
Should the NFCI changes (like initializing struct fields) be put into a
separate PR?
https://github.com/llvm/llvm-project/pull/94549
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
https://github.com/zyn0217 updated
https://github.com/llvm/llvm-project/pull/94889
>From 217c00f47aaa65b113d1c1cfd93a9c4e1d235c1a Mon Sep 17 00:00:00 2001
From: Younan Zhang
Date: Sun, 9 Jun 2024 11:49:18 +0800
Subject: [PATCH 1/3] [Clang] Fix two issues of CTAD for aggregates
---
clang/lib/S
@@ -506,14 +508,19 @@ class InitListChecker {
Sema &S, const InitializedEntity &Entity, InitListExpr *IL, QualType &T,
bool VerifyOnly, bool TreatUnavailableAsInvalid,
bool InOverloadResolution = false,
- SmallVectorImpl *AggrDeductionCandidateParamTypes
Author: Mike Rice
Date: 2024-06-10T08:12:12-07:00
New Revision: cb3ff9a3936d31c986fbfa5757c09e585669dc1e
URL:
https://github.com/llvm/llvm-project/commit/cb3ff9a3936d31c986fbfa5757c09e585669dc1e
DIFF:
https://github.com/llvm/llvm-project/commit/cb3ff9a3936d31c986fbfa5757c09e585669dc1e.diff
LOG
https://github.com/mikerice1969 closed
https://github.com/llvm/llvm-project/pull/94680
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vgvassilev wrote:
Ah, that’s well spotted. The current test should not require plugins. Would
dropping this conditional make it work for your case?
https://github.com/llvm/llvm-project/pull/89811
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Author: David Spickett
Date: 2024-06-10T16:14:53+01:00
New Revision: c32428151678f05fa3a9b5996e07f1e67094626c
URL:
https://github.com/llvm/llvm-project/commit/c32428151678f05fa3a9b5996e07f1e67094626c
DIFF:
https://github.com/llvm/llvm-project/commit/c32428151678f05fa3a9b5996e07f1e67094626c.diff
zmodem wrote:
> Would dropping this conditional make it work for your case?
I don't have a mac handy to try it right now. If you commit it, our system will
test it automatically :) Or maybe @thakis could give it a try?
https://github.com/llvm/llvm-project/pull/89811
___
vgvassilev wrote:
> > Would dropping this conditional make it work for your case?
>
> I don't have a mac handy to try it right now. If you commit it, our system
> will test it automatically :) Or maybe @Thakis could give it a try?
I am on my phone in the next couple of hours and I am not sure
Author: Mike Rice
Date: 2024-06-10T08:24:03-07:00
New Revision: 65d300989b1cdf84f518af1f46b6c7ba93ecd17c
URL:
https://github.com/llvm/llvm-project/commit/65d300989b1cdf84f518af1f46b6c7ba93ecd17c
DIFF:
https://github.com/llvm/llvm-project/commit/65d300989b1cdf84f518af1f46b6c7ba93ecd17c.diff
LOG
https://github.com/mikerice1969 closed
https://github.com/llvm/llvm-project/pull/94777
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shiltian updated
https://github.com/llvm/llvm-project/pull/94830
>From b468736cad511e36c7ac22d2ad86ba01ca21d8a5 Mon Sep 17 00:00:00 2001
From: Shilei Tian
Date: Mon, 10 Jun 2024 11:37:17 -0400
Subject: [PATCH] [Clang][AMDGPU] Add a new builtin type for buffer rsrc
---
clang
@@ -2201,6 +2207,9 @@ TypeInfo ASTContext::getTypeInfoImpl(const Type *T) const
{
Align = 8;
\
break;
#include "clang/Basic/WebAssemblyReferenceTypes.def"
+case BuiltinType::AMDGPUBufferRsrc:
+ W
@@ -0,0 +1,11 @@
+// REQUIRES: amdgpu-registered-target
+// RUN: %clang_cc1 -fsyntax-only -verify -triple amdgcn -Wno-unused-value %s
+
+void foo() {
+ int n = 100;
+ __buffer_rsrc_t v = 0; // expected-error {{cannot initialize a variable of
type '__buffer_rsrc_t' with an rvalu
@@ -0,0 +1,11 @@
+// REQUIRES: amdgpu-registered-target
+// RUN: %clang_cc1 -fsyntax-only -verify -triple amdgcn -Wno-unused-value %s
+
+void foo() {
+ int n = 100;
+ __buffer_rsrc_t v = 0; // expected-error {{cannot initialize a variable of
type '__buffer_rsrc_t' with an rvalu
@@ -2201,6 +2207,9 @@ TypeInfo ASTContext::getTypeInfoImpl(const Type *T) const
{
Align = 8;
\
break;
#include "clang/Basic/WebAssemblyReferenceTypes.def"
+case BuiltinType::AMDGPUBufferRsrc:
+ W
AtariDreams wrote:
@jroelofs Can we please merge this?
https://github.com/llvm/llvm-project/pull/92600
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jroelofs wrote:
> @jroelofs Can we please merge this?
It's up to @tmatheson-arm, it's their patch. I've already given an 'LGTM'.
https://github.com/llvm/llvm-project/pull/92600
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm
@@ -2201,6 +2207,9 @@ TypeInfo ASTContext::getTypeInfoImpl(const Type *T) const
{
Align = 8;
\
break;
#include "clang/Basic/WebAssemblyReferenceTypes.def"
+case BuiltinType::AMDGPUBufferRsrc:
+ W
https://github.com/balazske created
https://github.com/llvm/llvm-project/pull/95003
The checker was renamed at some time ago but the documentation was not updated.
The section is now just moved and renamed. The documentation is still very
simple and needs improvement.
From 6bcef3384ed986447b1
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Balázs Kéri (balazske)
Changes
The checker was renamed at some time ago but the documentation was not updated.
The section is now just moved and renamed. The documentation is still very
simple and needs improvement.
---
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Balázs Kéri (balazske)
Changes
The checker was renamed at some time ago but the documentation was not updated.
The section is now just moved and renamed. The documentation is still very
simple and needs improvement.
---
Full diff: https:
https://github.com/fhahn created https://github.com/llvm/llvm-project/pull/95004
This adds an attribute version of -fpass-by-value-is-noalias (added in
a874d63344)
Still needs proper docs.
>From 34e880a20b0b36218a1307e52ccb71092bfd64f7 Mon Sep 17 00:00:00 2001
From: Florian Hahn
Date: Mon, 10
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Florian Hahn (fhahn)
Changes
This adds an attribute version of -fpass-by-value-is-noalias (added in
a874d63344)
Still needs proper docs.
---
Full diff: https://github.com/llvm/llvm-project/pull/95004.diff
4 Files Affected:
- (
@@ -3328,3 +3300,63 @@ void clang::checkUnsafeBufferUsage(const Decl *D,
}
}
}
+
+void clang::checkUnsafeBufferUsage(const Decl *D,
+ UnsafeBufferUsageHandler &Handler,
+ bool EmitSuggestions) {
+#ifndef
https://github.com/fhahn edited https://github.com/llvm/llvm-project/pull/95004
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Tomas Matheson
Date: 2024-06-10T17:04:54+01:00
New Revision: 39f09e8dcd9ceff5c5030ede6393155782b7cdad
URL:
https://github.com/llvm/llvm-project/commit/39f09e8dcd9ceff5c5030ede6393155782b7cdad
DIFF:
https://github.com/llvm/llvm-project/commit/39f09e8dcd9ceff5c5030ede6393155782b7cdad.diff
https://github.com/tmatheson-arm closed
https://github.com/llvm/llvm-project/pull/92600
___
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/94560
___
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/94560
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3328,3 +3300,63 @@ void clang::checkUnsafeBufferUsage(const Decl *D,
}
}
}
+
+void clang::checkUnsafeBufferUsage(const Decl *D,
+ UnsafeBufferUsageHandler &Handler,
+ bool EmitSuggestions) {
+#ifndef
Author: c8ef
Date: 2024-06-10T09:10:30-07:00
New Revision: d9593c1edd7add74564cf950cd209fc35f5261b6
URL:
https://github.com/llvm/llvm-project/commit/d9593c1edd7add74564cf950cd209fc35f5261b6
DIFF:
https://github.com/llvm/llvm-project/commit/d9593c1edd7add74564cf950cd209fc35f5261b6.diff
LOG: [cl
https://github.com/owenca closed https://github.com/llvm/llvm-project/pull/94560
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/danakj edited https://github.com/llvm/llvm-project/pull/91991
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/steakhal commented:
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/94987
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/steakhal approved this pull request.
https://github.com/llvm/llvm-project/pull/95003
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hdoc wrote:
@cor3ntin yes, please merge this as we do not have commit access.
https://github.com/llvm/llvm-project/pull/84726
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MitalAshok updated
https://github.com/llvm/llvm-project/pull/78112
>From 92f8720e3d21521b589d5291f086a2f32b87bfe0 Mon Sep 17 00:00:00 2001
From: Mital Ashok
Date: Sun, 14 Jan 2024 19:52:31 +
Subject: [PATCH 1/9] [clang] [SemaCXX] Implement CWG2627 Bit-fields and
narrowin
jmorse wrote:
G'day -- we've got some tests for -O0 that, with this patch applied, generate
different code with-and-without the `-g` flag, if `-fdebug-info-for-profiling`
is enabled. Example godbolt: https://godbolt.org/z/qooo5Eah1 .
It seems the intention of this patch is generating additiona
@@ -3328,3 +3300,63 @@ void clang::checkUnsafeBufferUsage(const Decl *D,
}
}
}
+
+void clang::checkUnsafeBufferUsage(const Decl *D,
+ UnsafeBufferUsageHandler &Handler,
+ bool EmitSuggestions) {
+#ifndef
ldionne wrote:
CC @philnik777
https://github.com/llvm/llvm-project/pull/95004
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3328,3 +3300,63 @@ void clang::checkUnsafeBufferUsage(const Decl *D,
}
}
}
+
+void clang::checkUnsafeBufferUsage(const Decl *D,
+ UnsafeBufferUsageHandler &Handler,
+ bool EmitSuggestions) {
+#ifndef
@@ -196,12 +196,14 @@ bool AArch64FunctionInfo::needsAsyncDwarfUnwindInfo(
const MachineFunction &MF) const {
if (!NeedsAsyncDwarfUnwindInfo) {
const Function &F = MF.getFunction();
+const AArch64FunctionInfo *AFI = MF.getInfo();
// The check got "minsize" i
zmodem wrote:
I was able to confirm on a local Mac that removing the
`if(CLANG_PLUGIN_SUPPORT)` fixes the test, so I'll go ahead and commit that.
https://github.com/llvm/llvm-project/pull/89811
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
h
Author: Hans Wennborg
Date: 2024-06-10T18:43:58+02:00
New Revision: a30ef38cd455d500680fea8afdc824b4983fa981
URL:
https://github.com/llvm/llvm-project/commit/a30ef38cd455d500680fea8afdc824b4983fa981
DIFF:
https://github.com/llvm/llvm-project/commit/a30ef38cd455d500680fea8afdc824b4983fa981.diff
https://github.com/danakj updated
https://github.com/llvm/llvm-project/pull/91991
>From 273f93fa4aecc0017b5c7172bc296ced1bc6d5f5 Mon Sep 17 00:00:00 2001
From: danakj
Date: Mon, 13 May 2024 12:11:41 -0400
Subject: [PATCH 1/3] Generate -Wunsafe-buffer-usage warnings in ctor and field
initialize
https://github.com/hokein approved this pull request.
https://github.com/llvm/llvm-project/pull/94937
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -214,7 +232,8 @@ declare double @za_shared_callee(double) "aarch64_inout_za"
define double @za_new_caller_to_za_shared_callee(double %x) nounwind noinline
optnone "aarch64_new_za"{
; CHECK-COMMON-LABEL: za_new_caller_to_za_shared_callee:
; CHECK-COMMON: // %bb.0: //
@@ -3328,3 +3300,63 @@ void clang::checkUnsafeBufferUsage(const Decl *D,
}
}
}
+
+void clang::checkUnsafeBufferUsage(const Decl *D,
+ UnsafeBufferUsageHandler &Handler,
+ bool EmitSuggestions) {
+#ifndef
https://github.com/sayhaan created
https://github.com/llvm/llvm-project/pull/95008
Refactor RangeListsWritersByCU so we don't store them all in an array and
instead create an instance per DWO.
>From b2fe35ae825dc757ea1daaf49142e789c4a560fc Mon Sep 17 00:00:00 2001
From: Amir Ayupov
Date: Tue,
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-backend-x86
@llvm/pr-subscribers-lldb
Author: Sayhaan Siddiqui (sayhaan)
Changes
Refactor RangeListsWritersByCU so we don't store them all in an array and
instead create an instance per DWO.
---
Patch is 33.08 KiB, truncated
@@ -3328,3 +3300,63 @@ void clang::checkUnsafeBufferUsage(const Decl *D,
}
}
}
+
+void clang::checkUnsafeBufferUsage(const Decl *D,
+ UnsafeBufferUsageHandler &Handler,
+ bool EmitSuggestions) {
+#ifndef
dwblaikie wrote:
> G'day -- we've got some tests for -O0 that, with this patch applied, generate
> different code with-and-without the `-g` flag, if
> `-fdebug-info-for-profiling` is enabled. Example godbolt:
> https://godbolt.org/z/qooo5Eah1 .
>
> It seems the intention of this patch is gene
https://github.com/nga888 updated
https://github.com/llvm/llvm-project/pull/94664
>From d230adc33f362f5da64ad46da0ac6a969fd4582d Mon Sep 17 00:00:00 2001
From: Andrew Ng
Date: Fri, 31 May 2024 17:17:18 +0100
Subject: [PATCH] [clang] Fix loss of `dllexport` for exported template
specialization
https://github.com/NagyDonat approved this pull request.
LGTM as well, with two optional remarks.
https://github.com/llvm/llvm-project/pull/94987
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
https://github.com/NagyDonat edited
https://github.com/llvm/llvm-project/pull/94987
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2772,7 +2772,7 @@ fixVariable(const VarDecl *VD, FixitStrategy::Kind K,
// also covers call-operator of lamdas
isa(FD) ||
// skip when the function body is a try-block
-(FD->hasBody() && isa(FD->getBody())) ||
+isa_and_nonnull(FD->get
@@ -8510,7 +8510,8 @@ class MappableExprsHandler {
assert(VDecl == VD && "We got information for the wrong
declaration??");
assert(!Components.empty() &&
"Not expecting declaration with no component lists.");
-if (VD && E && VD->getType()
michaelmaitland wrote:
I want to circle back to a comment made by @asb
[here](https://github.com/llvm/llvm-project/pull/70294#issuecomment-1782282361):
> it's obvious that commercial designs with active support should go in
Since this is in BPi-F3, I think that it constitutes as a commercial d
101 - 200 of 456 matches
Mail list logo
