https://github.com/rofirrim updated
https://github.com/llvm/llvm-project/pull/155849
From 365c3c5d18b0c466fee85075d6f6bc3c63267fef Mon Sep 17 00:00:00 2001
From: Roger Ferrer Ibanez
Date: Wed, 27 Aug 2025 08:18:02 +
Subject: [PATCH] [Clang][OpenMP] Add an additional class to hold data that
@@ -496,9 +496,8 @@ _mm256_andnot_si256(__m256i __a, __m256i __b)
/// \param __b
///A 256-bit integer vector.
/// \returns A 256-bit integer vector containing the result.
-static __inline__ __m256i __DEFAULT_FN_ATTRS256
-_mm256_avg_epu8(__m256i __a, __m256i __b)
-{
+static
@@ -1,4 +1,7 @@
-// RUN: %check_clang_tidy -std=c++11-or-later %s
readability-container-contains %t
+// RUN: %check_clang_tidy -std=c++11-or-later %s
readability-container-contains %t -- \
localspook wrote:
```suggestion
// RUN: %check_clang_tidy %s readability
@@ -6266,26 +6266,6 @@ SDValue
AArch64TargetLowering::LowerINTRINSIC_WO_CHAIN(SDValue Op,
case Intrinsic::aarch64_sve_clz:
return DAG.getNode(AArch64ISD::CTLZ_MERGE_PASSTHRU, DL, Op.getValueType(),
Op.getOperand(2), Op.getOperand(3), Op.getOperand(
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Balazs Benics (steakhal)
Changes
Tldr;
We can't unconditionally trivially copy empty classes because that would
clobber the stored entries in the object that the optimized empty class
overlaps with.
This regression was
@@ -379,6 +386,111 @@ void DereferenceChecker::checkBind(SVal L, SVal V, const
Stmt *S,
C.addTransition(State, this);
}
+void DereferenceChecker::checkPreStmt(const BinaryOperator *Op,
+ CheckerContext &C) const {
+ if (!Op->isAdditiveO
steakhal wrote:
> Sounds reasonable.
>
> I feel a bit guilty that I wasn't able to predict this regression when I
> reviewed the reverted commit as "LGTM, clean little patch", but I don't think
> that I could've done better without investing drastically more effort into
> the review.
You act
@@ -130,13 +143,17 @@ struct BuiltinTypeMethodBuilder {
BuiltinTypeMethodBuilder(BuiltinTypeDeclBuilder &DB, DeclarationName &Name,
QualType ReturnTy, bool IsConst = false,
- bool IsCtor = false)
+
@@ -223,6 +223,7 @@ Deprecated Compiler Flags
Modified Compiler Flags
---
- The `-gkey-instructions` compiler flag is now enabled by default when DWARF
is emitted for plain C/C++ and optimizations are enabled. (#GH149509)
+- The `-Wnrvo` compiler flag will
https://github.com/hekota edited
https://github.com/llvm/llvm-project/pull/155866
___
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
Author: None (flovent)
Changes
Closes [#156154](https://github.com/llvm/llvm-project/issues/156154)
---
Full diff: https://github.com/llvm/llvm-project/pull/157468.diff
4 Files Affected:
- (modified) clang-tools-extra/clang-tidy/.clang-tid
Sirraide wrote:
> IIRC the Clang behaviour for `-Wincompatible-pointer-types` differs from GCC
> as it includes qualifiers while GCC doesn't, so it may show up with some new
> failures. That's not a comment for or against changing it.
Sort of: while `-Werror=incompatible-pointer-types` *would*
@@ -6313,30 +6313,38 @@ static FunctionDecl *rewriteBuiltinFunctionDecl(Sema
*Sema, ASTContext &Context,
unsigned i = 0;
SmallVector OverloadParams;
- for (QualType ParamType : FT->param_types()) {
+ {
+// The lvalue conversions in this loop are only for type resolu
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
Fixes #157422
---
Full diff: https://github.com/llvm/llvm-project/pull/157490.diff
2 Files Affected:
- (modified) clang/lib/AST/ByteCode/InterpBuiltin.cpp (+3)
- (modified) clang/test/AST/ByteCode/builtin
https://github.com/mmha created https://github.com/llvm/llvm-project/pull/157471
This patch replaces all uses of `replaceAllUseWith` with `replaceOp`. This
fixes a number of regressions post #155244.
>From 9cc2c129d4f1cc805901897c6a11be50fe971675 Mon Sep 17 00:00:00 2001
From: Morris Hafner
Da
cyndyishida wrote:
> FYI, this change caused some kinda noisy warnings from GCC, and the warnings
> don't sound entirely bogus:
>
> ```
> In file included from
> ../../clang/include/clang/Serialization/ASTRecordReader.h:20,
> from ../../clang/lib/Serialization/ASTReaderDecl.cp
@@ -1168,15 +1168,20 @@ static bool isTrivialForMSVC(const CXXRecordDecl *RD,
QualType Ty,
}
bool MicrosoftCXXABI::classifyReturnType(CGFunctionInfo &FI) const {
- const CXXRecordDecl *RD = FI.getReturnType()->getAsCXXRecordDecl();
- if (!RD)
-return false;
-
- bool is
https://github.com/rniwa closed https://github.com/llvm/llvm-project/pull/155033
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?=
Message-ID:
In-Reply-To:
@@ -118,6 +118,47 @@ static void fixSeparateAttrArgAndNumber(StringRef ArgStr,
SourceLocation ArgLoc,
Slot = new (Ctx) IdentifierLoc(ArgLoc, PP.getIdentifierInfo(FixedArg
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?=
Message-ID:
In-Reply-To:
https://github.com/llvm-beanz edited
https://github.com/llvm/llvm-project/pull/152537
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?=
Message-ID:
In-Reply-To:
https://github.com/llvm-beanz commented:
My only big concern is around the forced capitalization of semantics in the
diagnostics.
I also left some notes about follow-up changes we should c
@@ -431,15 +464,31 @@ Expr *BuiltinTypeMethodBuilder::getResourceHandleExpr() {
OK_Ordinary);
}
+BuiltinTypeMethodBuilder &
+BuiltinTypeMethodBuilder::createLocalVar(StringRef Name, QualType Ty) {
+ ensureCompleteDecl();
+
+ ASTContext &AS
Author: Krzysztof Parzyszek
Date: 2025-09-08T09:55:37-05:00
New Revision: 68b98bb9b97b60c02d26611535b3876572c81fe7
URL:
https://github.com/llvm/llvm-project/commit/68b98bb9b97b60c02d26611535b3876572c81fe7
DIFF:
https://github.com/llvm/llvm-project/commit/68b98bb9b97b60c02d26611535b3876572c81fe7
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Donát Nagy (NagyDonat)
Changes
My recent commit a80c393a9c498279a1ec9fd630535b9ff139b49f accidentally added a
paragraph which does not belong to the new documentation of the checker
`security.VAList`; this commit corrects this mistake.
-
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?=
Message-ID:
In-Reply-To:
@@ -1,7 +1,41 @@
-// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.0-compute -x hlsl -ast-dump -o
- %s -verify
+// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.0-compute -x h
@@ -1,6 +1,8 @@
// shell is required for the "dirname" command
cmtice wrote:
Done.
https://github.com/llvm/llvm-project/pull/156950
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bi
@@ -1,4 +1,4 @@
-// REQUIRES: shell
+// UNSUPPORTED: system-windows
cmtice wrote:
My understanding is that the "REQUIRES: shell" meant that it never ran on
Windows.
https://github.com/llvm/llvm-project/pull/156950
___
thesamesam wrote:
IIRC the Clang behaviour for `-Wincompatible-pointer-types` differs from GCC as
it includes qualifiers while GCC doesn't, so it may show up with some new
failures. That's not a comment for or against changing it.
https://github.com/llvm/llvm-project/pull/157364
__
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 origin/main HEAD --extensions cl,c --
clang-tools-extra/test/clang-tidy/checkers/bugp
jeaye wrote:
I've scoured the latest Linux/Windows build+test logs and the failures in there
have seemingly nothing to do with this PR. Based on comparing these results
with other recently opened PRs, it's unclear to me if these are consistent.
@vgvassilev do you know?
https://github.com/llvm
@@ -676,6 +774,58 @@
BuiltinTypeDeclBuilder::addHandleConstructorFromImplicitBinding() {
.finalize();
}
+BuiltinTypeDeclBuilder &BuiltinTypeDeclBuilder::addCreateFromBinding() {
hekota wrote:
We do want these methods to be separate with different names
mmha wrote:
My understanding is that we are not supposed to erase an op that was replaced
anymore. `eraseOp` calls `replaceOp` internally which lead to the double
replacement assertion failure.
I also think in our case it doesn't really matter whether we use `replaceOp`
and `replaceAllUseWith
@@ -483,6 +532,55 @@ BuiltinTypeMethodBuilder
&BuiltinTypeMethodBuilder::dereference(T Ptr) {
return *this;
}
+template
+BuiltinTypeMethodBuilder &
+BuiltinTypeMethodBuilder::getResourceHandle(T ResourceRecord) {
hekota wrote:
I'll change the name to `acc
https://github.com/cmtice updated
https://github.com/llvm/llvm-project/pull/156950
>From 05fa2572ad23795ef821b81ecd5dd4ff4d0aea7a Mon Sep 17 00:00:00 2001
From: Caroline Tice
Date: Thu, 4 Sep 2025 12:00:44 -0700
Subject: [PATCH 1/2] [clang-tools-extra] Remove 'REQUIRES: shell' from lit
tests.
https://github.com/markbhasawut updated
https://github.com/llvm/llvm-project/pull/157464
>From 70b779e23b1b0866e9bdd5ff2b2f6cc86feebd31 Mon Sep 17 00:00:00 2001
From: Bhasawut Singhaphan
Date: Mon, 8 Sep 2025 15:51:33 +0700
Subject: [PATCH 1/5] [Headers][X86] Enable constexpr handling for
MMX/
https://github.com/erichkeane approved this pull request.
Release note + bug number is a good idea, else LGTM.
https://github.com/llvm/llvm-project/pull/157139
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailm
@@ -431,15 +464,31 @@ Expr *BuiltinTypeMethodBuilder::getResourceHandleExpr() {
OK_Ordinary);
}
+BuiltinTypeMethodBuilder &
+BuiltinTypeMethodBuilder::createLocalVar(StringRef Name, QualType Ty) {
+ ensureCompleteDecl();
+
+ ASTContext &AS
@@ -431,15 +464,31 @@ Expr *BuiltinTypeMethodBuilder::getResourceHandleExpr() {
OK_Ordinary);
}
+BuiltinTypeMethodBuilder &
+BuiltinTypeMethodBuilder::createLocalVar(StringRef Name, QualType Ty) {
+ ensureCompleteDecl();
+
+ ASTContext &AS
https://github.com/hekota edited
https://github.com/llvm/llvm-project/pull/155866
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shafik approved this pull request.
LGTM but we need a release note.
https://github.com/llvm/llvm-project/pull/157139
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?=
Message-ID:
In-Reply-To:
@@ -282,14 +327,17 @@ void Parser::ParseHLSLAnnotations(ParsedAttributes &Attrs,
return;
}
} break;
- case ParsedAttr::UnknownAttribute:
-Diag(Loc, diag::
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?=
Message-ID:
In-Reply-To:
@@ -3054,6 +3057,9 @@ static void emitAttributes(const RecordKeeper &Records,
raw_ostream &OS,
<< (R.getValueAsBit("InheritEvenIfAlreadyPresent") ? "true"
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?= ,
Nathan =?utf-8?q?Gauër?=
Message-ID:
In-Reply-To:
@@ -4943,13 +4932,35 @@ def HLSLResourceBinding: InheritableAttr {
}];
}
-def HLSLSV_Position : HLSLAnnotationAttr {
- let Spellings = [HLSLAnnotation<"sv_position">];
https://github.com/andykaylor approved this pull request.
This looks good, but it isn't clear to me why the crashes were occurring. I see
from the description of #155244 that replaceAllUsesWith now has a delayed
effect in some circumstances. Was the fact that we were erasing the replaced op
ca
@@ -620,6 +635,422 @@ void createRegisterFatbinFunction(Module &M,
GlobalVariable *FatbinDesc,
// Add this function to constructors.
appendToGlobalCtors(M, CtorFunc, /*Priority=*/101);
}
+
+/// SYCLWrapper helper class that creates all LLVM IRs wrapping given images.
+stru
https://github.com/erichkeane closed
https://github.com/llvm/llvm-project/pull/157477
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -379,6 +386,111 @@ void DereferenceChecker::checkBind(SVal L, SVal V, const
Stmt *S,
C.addTransition(State, this);
}
+void DereferenceChecker::checkPreStmt(const BinaryOperator *Op,
+ CheckerContext &C) const {
+ if (!Op->isAdditiveO
@@ -0,0 +1,127 @@
+//===--===//
+//
+// 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/fmayer approved this pull request.
https://github.com/llvm/llvm-project/pull/156543
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/157331
>From 8941ba8930c4487c958c8f4746570575bb976931 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Sat, 6 Sep 2025 18:38:46 +0200
Subject: [PATCH] [CIR] Upstream VisitOpaqueValueExpr support for Complex &
Sc
@@ -111,53 +118,52 @@ llvm::TargetExtType
*HLSLBufferLayoutBuilder::createLayoutType(
if (!PackOffsets || PO != -1) {
if (!layoutField(FD, EndOffset, FieldOffset, FieldType, PO))
return nullptr;
-Layout.push_back(FieldOffset);
-LayoutEle
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/157364
___
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 origin/main HEAD --extensions h,cpp --
clang/lib/Basic/Diagnostic.cpp clang/lib/Basic
NagyDonat wrote:
Sounds reasonable.
I feel a bit guilty that I wasn't able to predict this regression when I
reviewed the reverted commit as "LGTM, clean little patch", but I don't think
that I could've done better without investing drastically more effort into the
review.
By the way does th
@@ -173,9 +178,11 @@ static bool isDeclRefExprToReference(const Expr *E) {
return false;
}
-void DereferenceChecker::reportBug(const DerefBugType &BT,
- ProgramStateRef State, const Stmt *S,
- CheckerContext
https://github.com/frasercrmck approved this pull request.
https://github.com/llvm/llvm-project/pull/157002
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/157501
Resolves warnings triggered by `Woverloaded-virtual`
>From 54fee3ec74c6f7b75f2aba5fc796d034311df830 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Mon, 8 Sep 2025 09:00:27 -0700
Subject: [PATCH] [clang]
https://github.com/kparzysz closed
https://github.com/llvm/llvm-project/pull/157470
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/157501
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -57,9 +59,12 @@ Error SpecialCaseList::Matcher::insert(StringRef Pattern,
unsigned LineNumber,
auto Glob = std::make_unique();
Glob->Name = Pattern.str();
Glob->LineNo = LineNumber;
+ // Backslashes are valid in posix-style filenames.
+ bool IsSlashAgnostic = Triple
llvmbot wrote:
@llvm/pr-subscribers-clang-modules
Author: Cyndy Ishida (cyndyishida)
Changes
Resolves warnings triggered by `Woverloaded-virtual`
---
Full diff: https://github.com/llvm/llvm-project/pull/157501.diff
4 Files Affected:
- (modified) clang/include/clang/Serialization/ASTRea
https://github.com/DKLoehr updated
https://github.com/llvm/llvm-project/pull/149886
>From 028be83a047f7dc3e5ff7d89f62ce4bd7452ff1b Mon Sep 17 00:00:00 2001
From: Devon Loehr
Date: Mon, 21 Jul 2025 19:07:23 +
Subject: [PATCH 1/4] Make special case matcher slash-agnostic
---
clang/docs/Sani
@@ -1175,13 +1175,15 @@ static void InitializePredefinedMacros(const TargetInfo
&TI,
Builder.defineMacro("__UINTMAX_C_SUFFIX__", ConstSuffix);
Builder.defineMacro("__UINTMAX_C(c)",
ConstSuffix.size() ? Twine("c##") + ConstSuffix : "c");
- DefineType(
https://github.com/alexey-bataev approved this pull request.
https://github.com/llvm/llvm-project/pull/156004
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/alexey-bataev edited
https://github.com/llvm/llvm-project/pull/157025
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/alexey-bataev commented:
Update OpenMP.rst and whatsnew docs for clang
https://github.com/llvm/llvm-project/pull/157025
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
@@ -1497,6 +1497,11 @@ def err_omp_multiple_step_or_linear_modifier : Error<
"multiple %select{'step size'|'linear modifier'}0 found in linear clause">;
def err_omp_deprecate_old_syntax: Error<
"old syntax '%0' on '%1' clause was deprecated, use new syntax '%2'">;
+def warn
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/156373
>From 1ff0888cb712c69f5fe6f587b60649d03b0472cd Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 29 Aug 2025 09:42:59 -0700
Subject: [PATCH 01/17] add support for rootsig as an hlsl environment
---
clang/
https://github.com/qiongsiwu closed
https://github.com/llvm/llvm-project/pull/156377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
qiongsiwu wrote:
Since the original check is reverted
(https://github.com/llvm/llvm-project/pull/157154), I am closing this PR.
https://github.com/llvm/llvm-project/pull/156377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llv
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/156373
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AmrDeveloper created
https://github.com/llvm/llvm-project/pull/157496
Upstream support for FPToFPBuiltin ATanOp
>From baccdc42c8c4046788fa364dcc71780bfd6c9ab0 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Sun, 7 Sep 2025 22:30:39 +0200
Subject: [PATCH] [CIR] Upstream FPT
https://github.com/Lancern closed
https://github.com/llvm/llvm-project/pull/156253
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
antoniofrighetto wrote:
> It would probably be best to split up the LLVM and the Clang changes.
Thank you, moved AA changes separately, addressed reviews. Updated this PR as
well, pending an update on tests due to
https://github.com/llvm/llvm-project/pull/147670.
> (I do wonder whether we sho
https://github.com/erichkeane approved this pull request.
I think this makes sense, and if GCC hasn't gotten a huge amount of pushback,
then there is no reason for us not to do this.
https://github.com/llvm/llvm-project/pull/157364
___
cfe-commits m
@@ -0,0 +1,90 @@
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
UTC_ARGS: --version 5
+; RUN: opt -S -passes=instcombine < %s | FileCheck %s
+
+; sinf clobbering errno, but %p cannot alias errno per C/C++ strict aliasing
rules via TBAA.
+; Hence, can
https://github.com/antoniofrighetto edited
https://github.com/llvm/llvm-project/pull/125258
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/157350
>From 764cd36f0b1ff4405616c126ac799e602a182f89 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Sun, 7 Sep 2025 18:58:07 +0200
Subject: [PATCH] [CIR] Upstream FPToFPBuiltin ASinOp
---
clang/include/clang
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
Fixes #157428
---
Full diff: https://github.com/llvm/llvm-project/pull/157494.diff
2 Files Affected:
- (modified) clang/lib/AST/ByteCode/InterpBuiltin.cpp (+3)
- (added) clang/test/AST/ByteCode/builtins.c
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/157494
Fixes #157428
>From 720cbfe0b715c6a2f1c02631b96dc560fedab88b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Mon, 8 Sep 2025 17:36:52 +0200
Subject: [PATCH] [clang][bytecode] Check strlen im
https://github.com/mmha updated https://github.com/llvm/llvm-project/pull/157116
>From 4121aecbda6be80161eb8fe22c351fadd5bbe62a Mon Sep 17 00:00:00 2001
From: Morris Hafner
Date: Fri, 5 Sep 2025 16:27:28 +0200
Subject: [PATCH 1/2] [CIR] Add support for __builtin_alloca
This patch adds support f
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/154034
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/154034
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -258,3 +258,13 @@ void trap2() {
// LLVM: {{.+}}:
// LLVM-NEXT:call void @_Z2f1v()
// LLVM: }
+
+void *test_alloca(unsigned long n) {
+ return __builtin_alloca(n);
+}
+
+// CIR-LABEL: @_Z11test_allocam(
+// CIR: %{{.+}} = cir.alloca !u8i, !cir.ptr, %
https://github.com/kparzysz closed
https://github.com/llvm/llvm-project/pull/157481
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin approved this pull request.
LGTM but let's see if other people have opinions in the next few days.
That GCC did it first and did not get pushback is reassuring
@erichkeane
https://github.com/llvm/llvm-project/pull/157364
__
@@ -223,6 +223,7 @@ Deprecated Compiler Flags
Modified Compiler Flags
---
- The `-gkey-instructions` compiler flag is now enabled by default when DWARF
is emitted for plain C/C++ and optimizations are enabled. (#GH149509)
+- The `-Wnrvo` compiler flag will
github-actions[bot] wrote:
@localspook 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 bui
https://github.com/localspook closed
https://github.com/llvm/llvm-project/pull/147553
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Victor Chernyakin
Date: 2025-09-08T08:30:01-07:00
New Revision: ab898f32c60689d1d47d0b6de66c30d3476994bb
URL:
https://github.com/llvm/llvm-project/commit/ab898f32c60689d1d47d0b6de66c30d3476994bb
DIFF:
https://github.com/llvm/llvm-project/commit/ab898f32c60689d1d47d0b6de66c30d3476994bb.d
steakhal wrote:
FYI @ziqingluo-90
https://github.com/llvm/llvm-project/pull/157480
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/RKSimon edited
https://github.com/llvm/llvm-project/pull/157464
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/adrian-prantl approved this pull request.
https://github.com/llvm/llvm-project/pull/155485
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Michael137 wrote:
Personally i think the assert makes sense here (if a user can't ever create an
AST like that, which IIUC they can't). Although it is true that we try not to
assert on user input as Jonas mentioned, this was a bug in our LLDB's AST
creation. Malformed DWARF can already cause a
@@ -6007,10 +6007,9 @@ bool Parser::isConstructorDeclarator(bool IsUnqualified,
bool DeductionGuide,
// A C++11 attribute here signals that we have a constructor, and is an
// attribute on the first constructor parameter.
- if (getLangOpts().CPlusPlus11 &&
- isCXX11
@@ -1053,9 +1053,12 @@ mlir::LogicalResult
CIRToLLVMBaseClassAddrOpLowering::matchAndRewrite(
mlir::LogicalResult CIRToLLVMAllocaOpLowering::matchAndRewrite(
cir::AllocaOp op, OpAdaptor adaptor,
mlir::ConversionPatternRewriter &rewriter) const {
- assert(!cir::Missing
Author: Erich Keane
Date: 2025-09-08T14:59:31Z
New Revision: 2946879c49d2cb17d1b53ff398edd681baf49417
URL:
https://github.com/llvm/llvm-project/commit/2946879c49d2cb17d1b53ff398edd681baf49417
DIFF:
https://github.com/llvm/llvm-project/commit/2946879c49d2cb17d1b53ff398edd681baf49417.diff
LOG: [
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/157480
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/maryammo edited
https://github.com/llvm/llvm-project/pull/147383
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane created
https://github.com/llvm/llvm-project/pull/157477
We are checking the 'type' of 'pointer' before we are checking whether it is
null or not! This is clearly incorrect, so this patch reorders the asserts.
>From b0d937cc90e9797094248a7df1f5c5d8b373c015 Mon Se
1 - 100 of 179 matches
Mail list logo
