llvmbot wrote:
@llvm/pr-subscribers-llvm-transforms
Author: Thurston Dang (thurstond)
Changes
This is glue code to convert LowerAllowCheckPass from a FUNCTION_PASS to
FUNCTION_PASS_WITH_PARAMS. The parameters are currently unused.
Future work will plumb `-fsanitize-skip-hot-cutoff` (intr
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Thurston Dang (thurstond)
Changes
This is glue code to convert LowerAllowCheckPass from a FUNCTION_PASS to
FUNCTION_PASS_WITH_PARAMS. The parameters are currently unused.
Future work will plumb `-fsanitize-skip-hot-cutoff` (introduced in
https://github.com/thurstond ready_for_review
https://github.com/llvm/llvm-project/pull/122765
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/thurstond updated
https://github.com/llvm/llvm-project/pull/122765
>From 6c244a083b458637f35149547270ba25675ca417 Mon Sep 17 00:00:00 2001
From: Thurston Dang
Date: Mon, 13 Jan 2025 18:38:14 +
Subject: [PATCH 1/2] [sanitizer][NFCI] Add Options to LowerAllowCheckPass
This
@@ -1060,238 +1061,331 @@
CodeGenFunction::evaluateOrEmitBuiltinObjectSize(const Expr *E, unsigned Type,
return ConstantInt::get(ResType, ObjectSize, /*isSigned=*/true);
}
-const FieldDecl *CodeGenFunction::FindFlexibleArrayMemberFieldAndOffset(
-ASTContext &Ctx, const
@@ -852,6 +874,58 @@ static bool areExprsMacroAndNonMacro(const Expr *&LhsExpr,
return LhsLoc.isMacroID() != RhsLoc.isMacroID();
}
+
+static bool areExprsSameMacroOrLiteral(const BinaryOperator *BinOp,
+ const ASTContext *Context) {
+
+
@@ -747,6 +747,28 @@ static bool areSidesBinaryConstExpressions(const
BinaryOperator *&BinOp, const A
return false;
}
+static bool areSidesBinaryConstExpressionsOrDefinesOrIntegerConstant(
+const BinaryOperator *&BinOp, const ASTContext *AstCtx) {
+ if (areSidesBinaryC
@@ -3494,6 +3494,9 @@ def fno_struct_path_tbaa : Flag<["-"],
"fno-struct-path-tbaa">, Group;
def fno_strict_enums : Flag<["-"], "fno-strict-enums">, Group;
def fno_strict_overflow : Flag<["-"], "fno-strict-overflow">, Group,
Visibility<[ClangOption, FlangOption]>;
+def fno_z
necto wrote:
> LGTM. Do you need me to merge that for you?
Yes, please
https://github.com/llvm/llvm-project/pull/109430
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/2LoS updated https://github.com/llvm/llvm-project/pull/122680
>From 3109461716e5e78b23bea7a2eb6aac3d34348612 Mon Sep 17 00:00:00 2001
From: LoS
Date: Mon, 13 Jan 2025 11:21:46 +0100
Subject: [PATCH 1/3] Fixed some warn-override tests in SemaCXX
---
...e => warn-inconsistent-
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/106321
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
@alexfh It wouldn't fix your issue, see this comment
https://github.com/llvm/llvm-project/pull/122423#issuecomment-2585390531 (there
were 2 issues, a bug in the PR implementation, and an unfortunate breakage
caused by some libraries relying on non-conforming behavior)
https://
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/106321
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -8192,6 +8192,14 @@ ExprResult InitializationSequence::Perform(Sema &S,
Kind.getRange().getEnd());
} else {
CurInit = new (S.Context) ImplicitValueInitExpr(Step->Type);
+// Note the return value isn't used to return early
+// to pres
@@ -19,13 +19,12 @@ struct on_member_pointer_complete_ty {
};
struct on_member_pointer_incomplete_ty {
- struct size_unknown * buf __counted_by(count); //
expected-error{{'counted_by' cannot be applied to a pointer with pointee of
unknown size because 'struct size_unknown'
cor3ntin wrote:
We probably want some test for this change
https://github.com/llvm/llvm-project/pull/115248
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
alexfh wrote:
FYI, we've also seen problems compiling code that includes certain boost
headers (version 1.84, as far as I can tell) after
https://github.com/llvm/llvm-project/pull/102857:
```
boost/asio/any_io_executor.hpp:161:13: error: satisfaction of constraint
'::boost::asio::execution::ex
Author: CarolineConcatto
Date: 2025-01-13T16:34:33Z
New Revision: 9256485043fe5cc3a24dba649deef8ae69e6d702
URL:
https://github.com/llvm/llvm-project/commit/9256485043fe5cc3a24dba649deef8ae69e6d702
DIFF:
https://github.com/llvm/llvm-project/commit/9256485043fe5cc3a24dba649deef8ae69e6d702.diff
L
@@ -8207,6 +8207,14 @@ ExprResult InitializationSequence::Perform(Sema &S,
Kind.getRange().getEnd());
} else {
CurInit = new (S.Context) ImplicitValueInitExpr(Step->Type);
+// Note the return value isn't used to return early
+// to pres
https://github.com/CarolineConcatto closed
https://github.com/llvm/llvm-project/pull/121947
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -8192,6 +8192,14 @@ ExprResult InitializationSequence::Perform(Sema &S,
Kind.getRange().getEnd());
} else {
CurInit = new (S.Context) ImplicitValueInitExpr(Step->Type);
+// Note the return value isn't used to return early
+// to pres
@@ -8836,13 +8836,22 @@ void FixedPointValueToString(SmallVectorImpl
&Str, llvm::APSInt Val,
unsigned Scale);
inline FunctionEffectsRef FunctionEffectsRef::get(QualType QT) {
+ const Type *TypePtr = QT.getTypePtr();
while (true) {
-QualTyp
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `premerge-monolithic-linux`
running on `premerge-linux-1` while building `clang,llvm` at step 7
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/153/builds/19558
He
Sirraide wrote:
> @Sirraide hi are you still working on this? We observed similar issue
> recently. #121723
Ah yeah, I was planning to get back to that last week but then I got sick; I
should have time to look at it again sometime later this week.
https://github.com/llvm/llvm-project/pull/110
@@ -8836,13 +8836,22 @@ void FixedPointValueToString(SmallVectorImpl
&Str, llvm::APSInt Val,
unsigned Scale);
inline FunctionEffectsRef FunctionEffectsRef::get(QualType QT) {
+ const Type *TypePtr = QT.getTypePtr();
while (true) {
-QualTyp
@@ -8836,13 +8836,22 @@ void FixedPointValueToString(SmallVectorImpl
&Str, llvm::APSInt Val,
unsigned Scale);
inline FunctionEffectsRef FunctionEffectsRef::get(QualType QT) {
+ const Type *TypePtr = QT.getTypePtr();
while (true) {
-QualTyp
Author: Vitaly Buka
Date: 2025-01-13T09:55:44-08:00
New Revision: 409ca49feb6659e279358f77b393f7955fb1bbda
URL:
https://github.com/llvm/llvm-project/commit/409ca49feb6659e279358f77b393f7955fb1bbda
DIFF:
https://github.com/llvm/llvm-project/commit/409ca49feb6659e279358f77b393f7955fb1bbda.diff
L
https://github.com/farzonl created
https://github.com/llvm/llvm-project/pull/122772
In the below code B varies over the arg list via a loop. However, the
diagnostics do not vary with the loop.
Fix so that diagnostics can vary with B.
>From 9df164c835e7df5bf4ea863bd9735bb0e456cb6c Mon Sep 17 0
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Farzon Lotfi (farzonl)
Changes
In the below code B varies over the arg list via a loop. However, the
diagnostics do not vary with the loop.
Fix so that diagnostics can vary with B.
---
Full diff: https://github.com/llvm/llvm-project/pull
https://github.com/vitalybuka created
https://github.com/llvm/llvm-project/pull/122773
None
>From b6abdc748225c29544d3c82bfafb5fac3dfd67a7 Mon Sep 17 00:00:00 2001
From: Vitaly Buka
Date: Mon, 13 Jan 2025 11:30:49 -0800
Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Vitaly Buka (vitalybuka)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/122773.diff
2 Files Affected:
- (modified) clang/lib/CodeGen/BackendUtil.cpp (+2-1)
- (modified) llvm/lib/Passes/PassBuilder.cpp (+1-1)
`
sarnex wrote:
Sorry, I kind of got lost in the above discussion. Are additional changes
required in Clang to parse builtins unsupported on the current target to make
sure we don't break anything or is only changing the result of `__has_builtin`
acceptable?
https://github.com/llvm/llvm-project
Author: Kazu Hirata
Date: 2025-01-13T11:38:00-08:00
New Revision: 8ed99689038bcc89dca92a4a3a13a4ede166bd7e
URL:
https://github.com/llvm/llvm-project/commit/8ed99689038bcc89dca92a4a3a13a4ede166bd7e
DIFF:
https://github.com/llvm/llvm-project/commit/8ed99689038bcc89dca92a4a3a13a4ede166bd7e.diff
L
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/122651
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Kazu Hirata
Date: 2025-01-13T11:38:20-08:00
New Revision: 19c0a6b5eb3a0c0619fccc140740500737fdcd47
URL:
https://github.com/llvm/llvm-project/commit/19c0a6b5eb3a0c0619fccc140740500737fdcd47
DIFF:
https://github.com/llvm/llvm-project/commit/19c0a6b5eb3a0c0619fccc140740500737fdcd47.diff
L
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/122652
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jhuber6 wrote:
> Sorry, I kind of got lost in the above discussion. Are additional changes
> required in Clang to parse builtins unsupported on the current target to make
> sure we don't break anything or is only changing the result of
> `__has_builtin` acceptable?
I believe the conclusion wa
@@ -0,0 +1,584 @@
+// RUN: %clang_cc1 -fsyntax-only -verify %s
+// RUN: %clang_cc1 -fexperimental-late-parse-attributes -fsyntax-only -verify
%s
+
+#define __counted_by(f) __attribute__((counted_by(f)))
+
+//
=
@@ -0,0 +1,56 @@
+; RUN: llc -verify-machineinstrs -O0 -mtriple=spirv32v1.3-vulkan-unknown %s -o
- | FileCheck %s
+; RUN: %if spirv-tools %{ llc -O0 -mtriple=spirv32v1.3-vulkan-unknown %s -o -
-filetype=obj | spirv-val %}
s-perron wrote:
https://github.com/llvm
cor3ntin wrote:
Thanks for spotting that!
Can you add a description to your commit message?
https://github.com/llvm/llvm-project/pull/122680
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
@@ -186,4 +218,216 @@ bool Sema::CheckCountedByAttrOnField(FieldDecl *FD, Expr
*E, bool CountInBytes,
return false;
}
+SourceRange Sema::BoundsSafetySourceRangeFor(const CountAttributedType *CATy) {
+ // This is an approximation that's not quite right. This points to the
+
@@ -2223,6 +2225,8 @@ def err_destructor_not_member : Error<
def err_destructor_cannot_be : Error<"destructor cannot be declared '%0'">;
def err_invalid_qualified_destructor : Error<
"'%0' qualifier is not allowed on a destructor">;
+def err_invalid_destructor_decl : Error<
-
@@ -3181,136 +3193,250 @@ Register
SPIRVInstructionSelector::buildPointerToResource(
return AcReg;
}
-bool SPIRVInstructionSelector::selectFirstBitHigh16(Register ResVReg,
-const SPIRVType *ResType,
-
shiltian wrote:
> Is there a way for us to still support 3-field triples and generic targets?
I can't come up with a better solution because we don't know how many `-` could
exist in the generic target id. With that, fixating the "known" part (aka
triple) would be reasonable. Do you have any s
https://github.com/Icohedron updated
https://github.com/llvm/llvm-project/pull/122202
>From 5610b225e76b046e911c1a7a0c1e4ccc128d35a1 Mon Sep 17 00:00:00 2001
From: Icohedron
Date: Thu, 9 Jan 2025 01:14:52 +
Subject: [PATCH 1/3] [HLSL] Implement the D3DCOLORtoUBYTE4 intrinsic
---
clang/lib
@@ -246,6 +246,17 @@ void PTMFTester::convert() [[clang::nonblocking]]
(this->*mConvertFunc)();
}
+// Allow implicit conversion from array to pointer.
+void nb14(unsigned idx) [[clang::nonblocking]]
+{
+ using FP = void (*)() [[clang::nonblocking]];
+ using
@@ -8836,13 +8836,22 @@ void FixedPointValueToString(SmallVectorImpl
&Str, llvm::APSInt Val,
unsigned Scale);
inline FunctionEffectsRef FunctionEffectsRef::get(QualType QT) {
+ const Type *TypePtr = QT.getTypePtr();
while (true) {
-QualTyp
https://github.com/frasercrmck created
https://github.com/llvm/llvm-project/pull/122751
Having the fp16 pragmas enabled in the header file is risky, as we have some
headers that also disable pragmas. We want to be as immune to changes in the
order of included header files as possible.
This pa
lamb-j wrote:
Is there a way for us to still support 3-field triples and generic targets?
In the past we worked to allow 3-field triples as bundler inputs. I would be
surprised if adding a 4-field restriction didn't break something internally.
https://github.com/llvm/llvm-project/pull/122629
_
@@ -96,3 +96,49 @@ namespace PR38286 {
template struct C; // expected-note {{non-type declaration found}}
template C::~C() {} // expected-error {{identifier 'C' after
'~' in destructor name does not name a type}}
}
+
+namespace GH121706 {
a-tarasyuk wrote:
@@ -2223,6 +2225,8 @@ def err_destructor_not_member : Error<
def err_destructor_cannot_be : Error<"destructor cannot be declared '%0'">;
def err_invalid_qualified_destructor : Error<
"'%0' qualifier is not allowed on a destructor">;
+def err_invalid_destructor_decl : Error<
-
https://github.com/AidanGoldfarb created
https://github.com/llvm/llvm-project/pull/122754
This update resolves #121235
>From b6c576fb90362640b2fd4e41bd7f13dfee95d04d Mon Sep 17 00:00:00 2001
From: Aidan
Date: Mon, 13 Jan 2025 11:53:39 -0500
Subject: [PATCH] initial template arg fix push
---
@@ -41,6 +41,14 @@ constexpr enable_if_t bit_cast(T
F) {
return __builtin_bit_cast(U, F);
}
+constexpr vector d3d_color_to_ubyte4_impl(vector V) {
+ // Use the same scaling factor used by FXC (i.e., 255.001953)
+ // Excerpt from stackoverflow discussion:
+ // "Built-in r
@@ -0,0 +1,140 @@
+//===-- AArch64BuildAttributes.cpp - AArch64 Build Attributes
-===//
+//
+// 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: Ap
https://github.com/sivan-shani edited
https://github.com/llvm/llvm-project/pull/118771
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shafik commented:
Thank you for the fix.
At a first glance this seems to make sense to me, I added some more reviewers.
Please add a release note.
https://github.com/llvm/llvm-project/pull/121854
___
cfe-commits mailing list
cfe-c
https://github.com/sivan-shani deleted
https://github.com/llvm/llvm-project/pull/118771
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
401 - 457 of 457 matches
Mail list logo
