https://github.com/foxtran updated
https://github.com/llvm/llvm-project/pull/127546
>From 80f3250eadc887d76abf9c2edbec6285aef6d5d2 Mon Sep 17 00:00:00 2001
From: "Igor S. Gerasimov"
Date: Mon, 17 Feb 2025 22:39:31 +0100
Subject: [PATCH 1/8] Merge {Maybe,Always}FallThrough_ReturnsNonVoid into
F
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/127623
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide commented:
I think the changes are fine now, but we should add one more:
```c++
// NO warning here. This could be a regular function returning a `Coro` object.
[[noreturn]] Coro test();
```
Putting the check in `CheckCompletedCoroutineBody()` means that this should
al
jhuber6 wrote:
The reproducer I posted above still crashes, just in a different place.
```
clang-21:
/home/jhuber/Documents/llvm/llvm-project/llvm/lib/IR/Instructions.cpp:744: void
llvm::CallInst::init(llvm::FunctionType*, llvm::Value*,
llvm::ArrayRef,
llvm::ArrayRef >, const llvm::Twine&):
https://github.com/michael-kenzel edited
https://github.com/llvm/llvm-project/pull/72040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/michael-kenzel edited
https://github.com/llvm/llvm-project/pull/72040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/127623
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jodelek updated
https://github.com/llvm/llvm-project/pull/127187
From a55c76bac1bd70878c777b5930553fac114d2fd5 Mon Sep 17 00:00:00 2001
From: Sebastian Jodlowski
Date: Fri, 14 Feb 2025 01:34:16 -0800
Subject: [PATCH 1/4] Add support for sm101 target architecture (Tegra
Black
https://github.com/Nerixyz updated
https://github.com/llvm/llvm-project/pull/127623
>From 3014c72978494df2a77f4b660fcea5fd20802221 Mon Sep 17 00:00:00 2001
From: Nerixyz
Date: Tue, 18 Feb 2025 12:56:05 +0100
Subject: [PATCH] [Clang] Warn about `[[noreturn]]` on coroutines
---
.../clang/Basic/
Sirraide wrote:
> I think the changes are fine now, but we should add one more test case:
>
> ```c++
> // NO warning here. This could be a regular function returning a `Coro`
> object.
> [[noreturn]] Coro test2();
> ```
>
> Putting the check in `CheckCompletedCoroutineBody()` means that this s
https://github.com/Sirraide approved this pull request.
https://github.com/llvm/llvm-project/pull/127623
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -544,25 +544,17 @@ static ControlFlowKind
CheckFallThrough(AnalysisDeclContext &AC) {
namespace {
struct CheckFallThroughDiagnostics {
- unsigned diag_MaybeFallThrough_HasNoReturn;
- unsigned diag_MaybeFallThrough_ReturnsNonVoid;
- unsigned diag_AlwaysFallThrough_HasNoR
https://github.com/foxtran updated
https://github.com/llvm/llvm-project/pull/127546
>From 80f3250eadc887d76abf9c2edbec6285aef6d5d2 Mon Sep 17 00:00:00 2001
From: "Igor S. Gerasimov"
Date: Mon, 17 Feb 2025 22:39:31 +0100
Subject: [PATCH 1/9] Merge {Maybe,Always}FallThrough_ReturnsNonVoid into
F
Sirraide wrote:
Also, just a heads-up, #127623 is likely going to be merged soon and introduce
some merge conflicts for you in `CheckFallThroughForBody()`. The code that that
pr touches is just deleted here, so that’s fine I think, but maybe make sure
that none of the diag ids are `0` if the f
jodelek wrote:
> LGTM.
>
> Do you need help merging the patch?
This is my first one, so would appreciate the help, thanks!
https://github.com/llvm/llvm-project/pull/127187
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org
@@ -223,6 +221,41 @@
} while (0)
#endif
+#define _LIBUNWIND_TRACING_COMPACT_UNWIND (0)
+#if !_LIBUNWIND_TRACING_COMPACT_UNWIND
+#define _LIBUNWIND_TRACE_COMPACT_UNWIND0(msg)
+#define _LIBUNWIND_TRACE_COMPACT_UNWIND(msg, ...)
+#else
+#define _LIBUNWIND_TRACE_COMPACT_UNWIND0
https://github.com/michael-kenzel edited
https://github.com/llvm/llvm-project/pull/72040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mxms0 created
https://github.com/llvm/llvm-project/pull/127698
Add support for absl Span and string_view types to warn on their unsafe usage.
>From 5f8bbc6abd98948186c4682035299be47e1550bc Mon Sep 17 00:00:00 2001
From: mxms
Date: Tue, 18 Feb 2025 16:21:18 -0500
Subject: [P
https://github.com/jodelek updated
https://github.com/llvm/llvm-project/pull/127187
From a55c76bac1bd70878c777b5930553fac114d2fd5 Mon Sep 17 00:00:00 2001
From: Sebastian Jodlowski
Date: Fri, 14 Feb 2025 01:34:16 -0800
Subject: [PATCH 1/4] Add support for sm101 target architecture (Tegra
Black
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
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Max (mxms0)
Changes
Add support for absl Span and string_view types to warn on their unsafe usage.
---
Full diff: https://github.com/llvm/llvm-project/pull/127698.diff
1 Files Affected:
- (modified) clang/lib/Analysis/UnsafeBufferUsage
llvmbot wrote:
@llvm/pr-subscribers-clang-analysis
Author: Max (mxms0)
Changes
Add support for absl Span and string_view types to warn on their unsafe usage.
---
Full diff: https://github.com/llvm/llvm-project/pull/127698.diff
1 Files Affected:
- (modified) clang/lib/Analysis/UnsafeBu
https://github.com/mxms0 closed https://github.com/llvm/llvm-project/pull/117370
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mxms0 wrote:
cc @jkorous-apple
https://github.com/llvm/llvm-project/pull/127698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide approved this pull request.
https://github.com/llvm/llvm-project/pull/127546
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -10069,6 +10069,17 @@ static std::optional
TryGetExprRange(ASTContext &C, const Expr *E,
case UO_AddrOf: // should be impossible
return IntRange::forValueOfType(C, GetExprType(E));
+case UO_Not: {
+ std::optional SubRange = TryGetExprRange(
+ C,
Author: nerix
Date: 2025-02-18T22:30:39+01:00
New Revision: db5bc8e9d07729ce4994cc908275722a093e5c0c
URL:
https://github.com/llvm/llvm-project/commit/db5bc8e9d07729ce4994cc908275722a093e5c0c
DIFF:
https://github.com/llvm/llvm-project/commit/db5bc8e9d07729ce4994cc908275722a093e5c0c.diff
LOG: [C
Sirraide wrote:
#127623 has been merged.
https://github.com/llvm/llvm-project/pull/127546
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide closed
https://github.com/llvm/llvm-project/pull/127623
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
efriedma-quic wrote:
Adding metadata to an instruction should never be required for correctness:
optimizations are not required to preserve metadata on instructions, so this
"fix" will just break again later.
If we need two different kinds of alloca in coroutine functions, we need a
different
https://github.com/Flandini created
https://github.com/llvm/llvm-project/pull/127700
Fixes #116444.
### Current vs expected behavior
Previously, the result of a `CXXNewExpr` was not always list initialized when
using an initializer list.
In this example:
```
struct S { int x; };
void F() {
https://github.com/foxtran updated
https://github.com/llvm/llvm-project/pull/127546
>From d09db5fc8ca7f58813f917c3726d09f0e4cfdd81 Mon Sep 17 00:00:00 2001
From: "Igor S. Gerasimov"
Date: Mon, 17 Feb 2025 22:39:31 +0100
Subject: [PATCH 1/9] Merge {Maybe,Always}FallThrough_ReturnsNonVoid into
F
https://github.com/Flandini edited
https://github.com/llvm/llvm-project/pull/127700
___
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: Michael Flanders (Flandini)
Changes
Fixes #116444.
### Current vs expected behavior
Previously, the result of a `CXXNewExpr` was not always list initialized when
using an initializer list.
In this example:
```
struct S { int x; };
void
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Michael Flanders (Flandini)
Changes
Fixes #116444.
### Current vs expected behavior
Previously, the result of a `CXXNewExpr` was not always list initialized when
using an initializer list.
In this example:
```
struct
Prabhuk wrote:
>
> Builder link:
> https://ci.chromium.org/ui/p/fuchsia/builders/prod/clang-windows-x64/b8722951897240189057/overview
> Logs:
> https://logs.chromium.org/logs/fuchsia/buildbucket/cr-buildbucket/8722951897240189057/+/u/clang/build/stdout
>
> ```
> FAILED:
> C:/b/s/w/ir/x/w/l
foxtran wrote:
@Sirraide, I still do not see how to call EmitDiag with coroutines :)
https://github.com/llvm/llvm-project/pull/127546
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -696,12 +661,12 @@ static void CheckFallThroughForBody(Sema &S, const Decl
*D, const Stmt *Body,
if (CD.checkDiagnostics(Diags, ReturnsVoid, HasNoReturn))
return;
SourceLocation LBrace = Body->getBeginLoc(), RBrace = Body->getEndLoc();
- auto EmitDiag = [&](Sourc
@@ -696,12 +661,12 @@ static void CheckFallThroughForBody(Sema &S, const Decl
*D, const Stmt *Body,
if (CD.checkDiagnostics(Diags, ReturnsVoid, HasNoReturn))
return;
SourceLocation LBrace = Body->getBeginLoc(), RBrace = Body->getEndLoc();
- auto EmitDiag = [&](Sourc
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/127546
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/s-perron updated
https://github.com/llvm/llvm-project/pull/125718
>From a65ccd00e3201af94059a2431f29da6ebfa6 Mon Sep 17 00:00:00 2001
From: Steven Perron
Date: Tue, 4 Feb 2025 11:47:42 -0500
Subject: [PATCH 1/2] [HLSL] Fix resrouce wrapper declaration
The resource wrappe
@@ -726,7 +713,11 @@ ABIArgInfo RISCVABIInfo::classifyArgumentType(QualType Ty,
bool IsFixed,
return ABIArgInfo::getDirect();
}
- if (const VectorType *VT = Ty->getAs()) {
+ // TODO: _BitInt is not handled yet in VLS calling convention since _BitInt
+ // ABI is also
@@ -0,0 +1,48 @@
+.. title:: clang-tidy - readability-ambiguous-smartptr-reset-call
+
+readability-ambiguous-smartptr-reset-call
+=
+
+Finds potentially erroneous calls to ``reset`` method on smart pointers when
+the pointee type also has a
@@ -0,0 +1,48 @@
+.. title:: clang-tidy - readability-ambiguous-smartptr-reset-call
+
+readability-ambiguous-smartptr-reset-call
+=
+
+Finds potentially erroneous calls to ``reset`` method on smart pointers when
+the pointee type also has a
https://github.com/tarunprabhu approved this pull request.
LGTM. Thanks.
https://github.com/llvm/llvm-project/pull/119718
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tarunprabhu edited
https://github.com/llvm/llvm-project/pull/119718
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tarunprabhu wrote:
LGTM. Thanks.
https://github.com/llvm/llvm-project/pull/126026
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3986,11 +3986,15 @@ defm assumptions : BoolFOption<"assumptions",
"Disable codegen and compile-time checks for C++23's [[assume]]
attribute">,
PosFlag>;
+
+let Visibility = [ClangOption, FlangOption] in {
tarunprabhu wrote:
> You'd hope that
Sirraide wrote:
> I can not find corresponding type to save enum values into specific container
> (expected `diag::FunModes`).
Ah, that’s because it emits an anonymous enum inside a namespace; just use
`unsigned` I’d say.
https://github.com/llvm/llvm-project/pull/127546
__
foxtran wrote:
https://godbolt.org/z/4EjMGoqTn
So, That is what I'd like to expect :D
https://github.com/llvm/llvm-project/pull/127546
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mizvekov commented:
Sorry that took a while, I am still coming back from WG21, so I can't look into
this throughly yet.
I would look into / want to know about the very specific requirements on
hitting the bug, described in the bug report, such as the lambda vs struct
thing.
Sirraide wrote:
> https://godbolt.org/z/4EjMGoqTn
>
> So, That is what I'd like to expect :D
What we’re currently emitting is more or less:
```c++
namespace diag {
namespace FunModes {
enum {
Function,
// ...
};
}
}
```
https://github.com/llvm/llvm-project/pull/127546
_
@@ -300,6 +306,10 @@ void NVPTXTargetInfo::getTargetDefines(const LangOptions
&Opts,
Builder.defineMacro("__CUDA_ARCH_FEAT_SM90_ALL", "1");
if (GPU == OffloadArch::SM_100a)
Builder.defineMacro("__CUDA_ARCH_FEAT_SM100_ALL", "1");
+if (GPU == OffloadArch::SM_
https://github.com/foxtran updated
https://github.com/llvm/llvm-project/pull/127546
>From 80f3250eadc887d76abf9c2edbec6285aef6d5d2 Mon Sep 17 00:00:00 2001
From: "Igor S. Gerasimov"
Date: Mon, 17 Feb 2025 22:39:31 +0100
Subject: [PATCH 1/6] Merge {Maybe,Always}FallThrough_ReturnsNonVoid into
F
@@ -3910,8 +3910,13 @@ StmtResult Sema::BuildReturnStmt(SourceLocation
ReturnLoc, Expr *RetValExp,
FnRetType = FD->getReturnType();
if (FD->hasAttrs())
Attrs = &FD->getAttrs();
-if (FD->isNoReturn())
- Diag(ReturnLoc, diag::warn_noreturn_function_has_ret
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 5ecce45ea2980aff35d1283d4dd3feb8f74de16c
973dde560d5ae499ffacf3cdd1d18d42465057fb --e
@@ -3910,8 +3910,13 @@ StmtResult Sema::BuildReturnStmt(SourceLocation
ReturnLoc, Expr *RetValExp,
FnRetType = FD->getReturnType();
if (FD->hasAttrs())
Attrs = &FD->getAttrs();
-if (FD->isNoReturn())
- Diag(ReturnLoc, diag::warn_noreturn_function_has_ret
https://github.com/foxtran updated
https://github.com/llvm/llvm-project/pull/127546
>From fbdd55feb53bfbcc07af39dc9fa1bf77311f5e45 Mon Sep 17 00:00:00 2001
From: "Igor S. Gerasimov"
Date: Mon, 17 Feb 2025 22:39:31 +0100
Subject: [PATCH 1/6] Merge {Maybe,Always}FallThrough_ReturnsNonVoid into
F
https://github.com/Nerixyz updated
https://github.com/llvm/llvm-project/pull/127623
>From 6ba42659e7350a893273b22f011166fa58c05e89 Mon Sep 17 00:00:00 2001
From: Nerixyz
Date: Tue, 18 Feb 2025 12:56:05 +0100
Subject: [PATCH] [Clang] Warn about `[[noreturn]]` on coroutines
---
.../clang/Basic/
foxtran wrote:
Yep, I checked #122505
https://github.com/llvm/llvm-project/pull/127546
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/foxtran updated
https://github.com/llvm/llvm-project/pull/127546
>From 80f3250eadc887d76abf9c2edbec6285aef6d5d2 Mon Sep 17 00:00:00 2001
From: "Igor S. Gerasimov"
Date: Mon, 17 Feb 2025 22:39:31 +0100
Subject: [PATCH 1/7] Merge {Maybe,Always}FallThrough_ReturnsNonVoid into
F
foxtran wrote:
@Sirraide, CI should be happy now. I have merged `invalid-noreturn warnings` as
well as now `%enum_select` is used.
https://github.com/llvm/llvm-project/pull/127546
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.l
@@ -94,6 +94,11 @@ bool Qualifiers::isTargetAddressSpaceSupersetOf(LangAS A,
LangAS B,
(A == LangAS::Default &&
(B == LangAS::cuda_constant || B == LangAS::cuda_device ||
B == LangAS::cuda_shared)) ||
+ // In HLSL, the this pointer for mem
Author: Sarah Spall
Date: 2025-02-18T10:49:26-08:00
New Revision: 8e16e5ca23d70b6d21346abfbe292aa7e91dcb2e
URL:
https://github.com/llvm/llvm-project/commit/8e16e5ca23d70b6d21346abfbe292aa7e91dcb2e
DIFF:
https://github.com/llvm/llvm-project/commit/8e16e5ca23d70b6d21346abfbe292aa7e91dcb2e.diff
L
https://github.com/spall closed https://github.com/llvm/llvm-project/pull/127670
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Sirraide wrote:
> Maybe we can just emit the enum once and check that every diagnostic that
> uses `%enum_select` w/ the same enum has the same number of cases
That also reminds me: A while ago I ran into a bug where someone had added a
few cases to one diagnotic but forgot to update another d
@@ -6585,6 +6585,18 @@ void InitializationSequence::InitializeFrom(Sema &S,
}
}
+if (S.getLangOpts().HLSL && Initializer && isa(DestAT))
{
llvm-beanz wrote:
Looking at this a bit. `canPerformArrayCopy` fails in cases that would
otherwise be va
https://github.com/llvm-beanz updated
https://github.com/llvm/llvm-project/pull/127557
>From 54ba5ffea82a9613fd343750ec36242e1494f5e4 Mon Sep 17 00:00:00 2001
From: Chris Bieneman
Date: Mon, 17 Feb 2025 20:41:18 -0600
Subject: [PATCH 1/2] [HLSL] Allow arrays to copy-initialize
This change allo
@@ -94,6 +94,11 @@ bool Qualifiers::isTargetAddressSpaceSupersetOf(LangAS A,
LangAS B,
(A == LangAS::Default &&
(B == LangAS::cuda_constant || B == LangAS::cuda_device ||
B == LangAS::cuda_shared)) ||
+ // In HLSL, the this pointer for mem
@@ -2927,6 +2898,53 @@ void CastOperation::CheckCXXCStyleCast(bool
FunctionalStyle,
}
}
+// CheckHLSLCStyleCast - Returns `true` ihe cast is handled or errored as an
+// HLSL-specific cast. Returns false if the cast should be checked as a CXX
+// C-Style cast.
+bool CastOpe
@@ -0,0 +1,25 @@
+#include "llvm/Support/LibCXXABI.h"
+
+namespace llvm {
+
+std::unique_ptr CXXABI::Create(Triple &TT) {
+ if (TT.getOS() == Triple::Linux)
+return std::make_unique();
teresajohnson wrote:
This is overly restrictive, however, since Linux is
https://github.com/bogner approved this pull request.
https://github.com/llvm/llvm-project/pull/127557
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1920,6 +1924,18 @@ Error LTO::runThinLTO(AddStreamFn AddStream, FileCache
Cache,
ThinLTO.CombinedIndex, WholeProgramVisibilityEnabledInLTO,
DynamicExportSymbols, VisibleToRegularObjSymbols);
+ Triple TT(getTargetTriple());
+ DeadRTTIElimIndex(ThinLTO.Combine
@@ -911,6 +916,63 @@ static void setLiveRoot(ModuleSummaryIndex &Index,
StringRef Name) {
Summary->setLive(true);
}
+// Return true if the User U is reachable from a non-vtable user
+// through the use-def chain.
+static bool hasNonVTableUsers(const User *U, CXXABI *ABI
https://github.com/MacDue edited
https://github.com/llvm/llvm-project/pull/121763
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/andykaylor commented:
This is an improvement. I have a few minor comments.
https://github.com/llvm/llvm-project/pull/127621
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
https://github.com/s-perron converted_to_draft
https://github.com/llvm/llvm-project/pull/125718
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -94,6 +94,11 @@ bool Qualifiers::isTargetAddressSpaceSupersetOf(LangAS A,
LangAS B,
(A == LangAS::Default &&
(B == LangAS::cuda_constant || B == LangAS::cuda_device ||
B == LangAS::cuda_shared)) ||
+ // In HLSL, the this pointer for mem
https://github.com/s-perron updated
https://github.com/llvm/llvm-project/pull/125718
>From 3ba76b95757a43002d8f799651d7846658011e37 Mon Sep 17 00:00:00 2001
From: Steven Perron
Date: Tue, 4 Feb 2025 11:47:42 -0500
Subject: [PATCH 1/2] [HLSL] Fix resrouce wrapper declaration
The resource wrappe
Sirraide wrote:
> Will it have redefinition problem since I need to share this enum between two
> diagnostics?
Hmm, that’s a good point. I think in that case, for now just use `%enum_select`
for one of the two and make the other one a regular `%select` (you can still
use the enum for both the
@@ -1817,5 +1817,9 @@ def ext_hlsl_access_specifiers : ExtWarn<
InGroup;
def err_hlsl_unsupported_component : Error<"invalid component '%0' used;
expected 'x', 'y', 'z', or 'w'">;
def err_hlsl_packoffset_invalid_reg : Error<"invalid resource class specifier
'%0' for packoff
haoNoQ wrote:
We could also go with something like "uninitialized or inaccessible memory" so
that to technically cover the OOB case without triggering an immediate visceral
reaction to buffer overruns. But it'll probably still be net-negative in
confusion compared to a simple "uninitialized".
https://github.com/bogner edited
https://github.com/llvm/llvm-project/pull/127675
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Nerixyz updated
https://github.com/llvm/llvm-project/pull/127623
>From a7924a05880d6f96c27b169a709327275d3eb9f3 Mon Sep 17 00:00:00 2001
From: Nerixyz
Date: Tue, 18 Feb 2025 12:56:05 +0100
Subject: [PATCH] [Clang] Warn about `[[noreturn]]` on coroutines
---
.../clang/Basic/
Nerixyz wrote:
> Probably where we create the FunctionDecl?
The first warning is emitted when creating the return value. So I'm checking
for a coroutine there. To fix the crash in the AnalysisBasedWarnings, I'm
checking if a diagnostic with ID 0 would've been emitted.
https://github.com/llvm/
@@ -94,6 +94,11 @@ bool Qualifiers::isTargetAddressSpaceSupersetOf(LangAS A,
LangAS B,
(A == LangAS::Default &&
(B == LangAS::cuda_constant || B == LangAS::cuda_device ||
B == LangAS::cuda_shared)) ||
+ // In HLSL, the this pointer for mem
foxtran wrote:
Looks like I cannot have some variable with proper type generated from
`%enum_select`.
So, for
```
"%enum_select{%Function{function}|%Block{block}|%Lambda{lambda}|%Coroutine{coroutine}}0"
```
I can not find corresponding type to save enum values into specific container
(expected
https://github.com/ElvinaYakubova updated
https://github.com/llvm/llvm-project/pull/127620
>From 4c29499a36117ab9681f99a16e017b33c2322317 Mon Sep 17 00:00:00 2001
From: Elvina Yakubova
Date: Thu, 13 Feb 2025 05:45:21 -0800
Subject: [PATCH] [AArch64] Add optional extensions enabled on Grace
Ena
https://github.com/ykhatav updated
https://github.com/llvm/llvm-project/pull/127654
>From 2714184615ef06027941c7bccee3a2453d76c24b Mon Sep 17 00:00:00 2001
From: "Khatavkar, Yashasvi"
Date: Tue, 18 Feb 2025 06:44:49 -0800
Subject: [PATCH 1/3] Support template as type
---
clang/include/clang/B
@@ -944,6 +944,15 @@ def ProcessorFeatures {
list Falkor = [HasV8_0aOps, FeatureCRC, FeatureSHA2,
FeatureAES,
FeatureFPARMv8, FeatureNEON,
FeaturePerfMon,
FeatureRDM];
+ list Grace= [HasV9_0aO
https://github.com/Flandini updated
https://github.com/llvm/llvm-project/pull/127700
>From 2c094dc9d9eab60e94ea018a854a1eae48dd894b Mon Sep 17 00:00:00 2001
From: Michael Flanders
Date: Tue, 18 Feb 2025 12:27:18 -0600
Subject: [PATCH 1/2] [analyzer] Do list init for CXX new expr with initialize
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Michael Flanders (Flandini)
Changes
Fixes #116444.
Closed #127700 because I accidentally updated it in github UI.
### Current vs expected behavior
Previously, the result of a `CXXNewExpr` was not always list initialized
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Michael Flanders (Flandini)
Changes
Fixes #116444.
Closed #127700 because I accidentally updated it in github UI.
### Current vs expected behavior
Previously, the result of a `CXXNewExpr` was not always list initialized when
using an in
https://github.com/Flandini updated
https://github.com/llvm/llvm-project/pull/127702
>From 12791f2c89f7e42bd261ac573c2497857c42b6f3 Mon Sep 17 00:00:00 2001
From: Michael Flanders
Date: Tue, 18 Feb 2025 15:56:13 -0600
Subject: [PATCH] [analyzer] Do list initialization for CXXNewExpr with
initi
llvmbot wrote:
@llvm/pr-subscribers-libc
Author: Joseph Huber (jhuber6)
Changes
Summary:
The scan operation implemented here only works if there are contiguous
ones in the executation mask that can be used to propagate the result.
There are two solutions to this, one is to enter 'whole-wav
https://github.com/jhuber6 created
https://github.com/llvm/llvm-project/pull/127703
Summary:
The scan operation implemented here only works if there are contiguous
ones in the executation mask that can be used to propagate the result.
There are two solutions to this, one is to enter 'whole-wave-
https://github.com/Flandini closed
https://github.com/llvm/llvm-project/pull/127700
___
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: Joseph Huber (jhuber6)
Changes
Summary:
The scan operation implemented here only works if there are contiguous
ones in the executation mask that can be used to propagate the result.
There are two solutions to this, one is to enter 'whole-wa
https://github.com/jhuber6 milestoned
https://github.com/llvm/llvm-project/pull/127504
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/127703
>From bf81a9280b0fda28862fb56f633614ce9bae9840 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Tue, 18 Feb 2025 15:08:05 -0600
Subject: [PATCH] [Clang] Fix cross-lane scan when given divergent lanes
Summary:
301 - 400 of 537 matches
Mail list logo
