@@ -697,45 +699,21 @@ class Qualifiers {
/// every address space is a superset of itself.
/// CL2.0 adds:
/// __generic is a superset of any address space except for __constant.
- static bool isAddressSpaceSupersetOf(LangAS A, LangAS B) {
-// Address spaces must
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/115777
>From 23a8d5af0ab181814885bca6ab6494be9d71f59b Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Wed, 13 Nov 2024 18:14:05 -0600
Subject: [PATCH 1/4] use ASTContext
---
.../bugprone/VirtualNearMissCheck.cpp
@@ -697,45 +699,21 @@ class Qualifiers {
/// every address space is a superset of itself.
/// CL2.0 adds:
/// __generic is a superset of any address space except for __constant.
- static bool isAddressSpaceSupersetOf(LangAS A, LangAS B) {
-// Address spaces must
https://github.com/SixWeining closed
https://github.com/llvm/llvm-project/pull/115832
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/SixWeining edited
https://github.com/llvm/llvm-project/pull/115832
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jhuber6 wrote:
> I'm not sure what unrelated code you're saying would need to pulled into a
> .cpp file. It looks like there's only one actual call to
> `TI.isAddressSpaceSupersetOf`, so if you just pass around an `ASTContext &`
> to that point, nothing else will need to drill into it. And fra
https://github.com/SixWeining approved this pull request.
LGTM. Tips: It's not necessary to use force-push to address review comments.
Add new commits and they will be squashed when merge.
https://github.com/llvm/llvm-project/pull/115832
___
cfe-commi
https://github.com/tangaac updated
https://github.com/llvm/llvm-project/pull/115832
>From 206bfe8e5e504620c1cd089298a2fedf8c4f3d23 Mon Sep 17 00:00:00 2001
From: tangaac
Date: Tue, 12 Nov 2024 16:31:57 +0800
Subject: [PATCH 1/2] add la v1.1 features for sys::getHostCPUFeatures
---
clang/lib/D
@@ -14,6 +14,7 @@
#include "clang/Driver/Options.h"
#include "llvm/TargetParser/Host.h"
#include "llvm/TargetParser/LoongArchTargetParser.h"
+#include
tangaac wrote:
It's added automatically by the lsp clangd, I will delete it.
https://github.com/llvm/llvm-p
@@ -14,6 +14,7 @@
#include "clang/Driver/Options.h"
#include "llvm/TargetParser/Host.h"
#include "llvm/TargetParser/LoongArchTargetParser.h"
+#include
SixWeining wrote:
Is this header necessary?
https://github.com/llvm/llvm-project/pull/115832
__
https://github.com/c8ef closed https://github.com/llvm/llvm-project/pull/114637
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 approved this pull request.
https://github.com/llvm/llvm-project/pull/114544
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: c8ef
Date: 2024-11-14T09:28:51+08:00
New Revision: 6e614e11df6a152082b51a1b18332cb8730a4032
URL:
https://github.com/llvm/llvm-project/commit/6e614e11df6a152082b51a1b18332cb8730a4032
DIFF:
https://github.com/llvm/llvm-project/commit/6e614e11df6a152082b51a1b18332cb8730a4032.diff
LOG: [cl
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/115180
>From 428283c7b61ca50d40ffd3ddc5c08aca39f39533 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 7 Nov 2024 00:35:47 +0800
Subject: [PATCH 1/2] [clang-tidy] fix false positive when detecting templated
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/115180
>From 428283c7b61ca50d40ffd3ddc5c08aca39f39533 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 7 Nov 2024 00:35:47 +0800
Subject: [PATCH 1/2] [clang-tidy] fix false positive when detecting templated
@@ -8732,6 +8732,18 @@ void Sema::CheckVariableDeclarationType(VarDecl *NewVD) {
}
}
+ // zero sized static arrays are not allowed in HIP device functions
+ if (getLangOpts().HIP &&
+ DeclAttrsMatchCUDAMode(getLangOpts(), getCurFunctionDecl())) {
@@ -8732,6 +8732,18 @@ void Sema::CheckVariableDeclarationType(VarDecl *NewVD) {
}
}
+ // zero sized static arrays are not allowed in HIP device functions
+ if (getLangOpts().HIP &&
+ DeclAttrsMatchCUDAMode(getLangOpts(), getCurFunctionDecl())) {
https://github.com/tangaac updated
https://github.com/llvm/llvm-project/pull/115832
>From 206bfe8e5e504620c1cd089298a2fedf8c4f3d23 Mon Sep 17 00:00:00 2001
From: tangaac
Date: Tue, 12 Nov 2024 16:31:57 +0800
Subject: [PATCH 1/2] add la v1.1 features for sys::getHostCPUFeatures
---
clang/lib/D
@@ -0,0 +1,31 @@
+// RUN: %clang_analyze_cc1 %s \
+// RUN: -analyzer-checker=debug.ExprInspection \
+// RUN: -verify
+
+void clang_analyzer_eval(int);
+
+void test_derived_sym_simplification_on_assume(int s0, int s1) {
+ int elem = s0 + s1 + 1;
+ if (elem-- == 0) // elem = s
rniwa wrote:
Thanks for the review!
https://github.com/llvm/llvm-project/pull/115695
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -345,18 +344,15 @@ TEST_F(OpenMPIRBuilderTest, CreateBarrier) {
IRBuilder<> Builder(BB);
- OpenMPIRBuilder::InsertPointOrErrorTy BarrierIP1 =
- OMPBuilder.createBarrier({IRBuilder<>::InsertPoint()}, OMPD_for);
- assert(BarrierIP1 && "unexpected error");
+ ASSERT_
https://github.com/rniwa closed https://github.com/llvm/llvm-project/pull/115695
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1875,13 +1873,17 @@ Instruction *InstCombinerImpl::visitFPTrunc(FPTruncInst
&FPT) {
// fptrunc (select Cond, (fpext X), Y --> select Cond, X, (fptrunc Y)
Value *NarrowY = Builder.CreateFPTrunc(Y, Ty);
Value *Sel = Builder.CreateSelect(Cond, X, NarrowY, "n
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Nicolas van Kempen (nicovank)
Changes
In one of the cases recently added to this check in #110448, the error
message is no longer accurate as the comparison is not with zero. #116033 brought my attention to this as it may add
@@ -697,45 +699,21 @@ class Qualifiers {
/// every address space is a superset of itself.
/// CL2.0 adds:
/// __generic is a superset of any address space except for __constant.
- static bool isAddressSpaceSupersetOf(LangAS A, LangAS B) {
-// Address spaces must
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/115777
>From 23a8d5af0ab181814885bca6ab6494be9d71f59b Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Wed, 13 Nov 2024 18:14:05 -0600
Subject: [PATCH 1/3] use ASTContext
---
.../bugprone/VirtualNearMissCheck.cpp
https://github.com/danix800 updated
https://github.com/llvm/llvm-project/pull/115579
>From 19b47c6ad25453c6be74bfd4cbdb7bc7eeed401c Mon Sep 17 00:00:00 2001
From: dingfei
Date: Sat, 9 Nov 2024 10:28:59 +0800
Subject: [PATCH 1/3] [StaticAnalyzer] early return if sym is concrete on
assuming
Thi
https://github.com/rjmccall edited
https://github.com/llvm/llvm-project/pull/115777
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vitalybuka wrote:
> @vitalybuka any idea how lldb-server is failing with `UNRESOLVED`? How in the
> world are any of the changes here affecting lldb-server?
flake?
https://github.com/llvm/llvm-project/pull/107332
___
cfe-commits mailing list
cfe-comm
@@ -17,34 +17,34 @@
#include "clang/Basic/TargetInfo.h"
#include "clang/Basic/TargetOptions.h"
#include "llvm/Support/Compiler.h"
+#include "llvm/Support/NVPTXAddrSpace.h"
#include "llvm/TargetParser/Triple.h"
#include
namespace clang {
namespace targets {
static cons
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/115777
>From 23a8d5af0ab181814885bca6ab6494be9d71f59b Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Wed, 13 Nov 2024 18:14:05 -0600
Subject: [PATCH 1/2] use ASTContext
---
.../bugprone/VirtualNearMissCheck.cpp
https://github.com/nicovank created
https://github.com/llvm/llvm-project/pull/116132
In one of the cases recently added to this check in #110448, the error message
is no longer accurate as the comparison is not with zero. #116033 brought my
attention to this as it may add another comparison w
@@ -31,6 +31,7 @@ namespace clang {
template class CanProxy;
template struct CanProxyAdaptor;
class CXXRecordDecl;
+class ASTContext;
rjmccall wrote:
Minor nit: these are alphabetized.
https://github.com/llvm/llvm-project/pull/115777
_
https://github.com/rjmccall commented:
Broadly LGTM. A few structural requests.
https://github.com/llvm/llvm-project/pull/115777
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/arsenm updated
https://github.com/llvm/llvm-project/pull/111976
>From 7391edf32b0d745a7d57bee5ee2cec8520b646a5 Mon Sep 17 00:00:00 2001
From: Matt Arsenault
Date: Fri, 11 Oct 2024 14:33:32 +0400
Subject: [PATCH 01/10] clang: Remove requires system-linux from some driver
test
@@ -235,10 +235,9 @@ class OpenMPIRBuilderTest : public testing::Test {
return Error::success();
};
-Expected LoopResult =
-OMPBuilder.createCanonicalLoop(Loc, LoopBodyGenCB, CastedTripCount);
-assert(LoopResult && "unexpected error");
-CanonicalL
Author: Ryosuke Niwa
Date: 2024-11-13T16:28:01-08:00
New Revision: 73b577cc8c8a8ceeac87de5953a2c643e125d43e
URL:
https://github.com/llvm/llvm-project/commit/73b577cc8c8a8ceeac87de5953a2c643e125d43e
DIFF:
https://github.com/llvm/llvm-project/commit/73b577cc8c8a8ceeac87de5953a2c643e125d43e.diff
https://github.com/t-rasmud approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/115695
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jhuber6 wrote:
> Oh, you can just forward-declare `class ASTContext` at the top of that file.
> It's funny that that isn't already there.
Done, and a sema test.
https://github.com/llvm/llvm-project/pull/115777
___
cfe-commits mailing list
cfe-commits
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/115777
>From 23a8d5af0ab181814885bca6ab6494be9d71f59b Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Wed, 13 Nov 2024 18:14:05 -0600
Subject: [PATCH] use ASTContext
---
.../bugprone/VirtualNearMissCheck.cpp
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/115777
>From bc76c323aefe52b019375cf3a3227223e2d97133 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Wed, 13 Nov 2024 18:14:05 -0600
Subject: [PATCH] use ASTContext
---
.../bugprone/VirtualNearMissCheck.cpp
@@ -5322,6 +5324,11 @@ RValue CodeGenFunction::EmitCall(const CGFunctionInfo
&CallInfo,
IRCallArgs[FirstIRArg] = Val;
break;
}
+ } else if (I->getType()->isArrayParameterType()) {
+// use the tmp created by the HLSLOutArgExpr
+/
brad0 wrote:
cc @MaskRay
https://github.com/llvm/llvm-project/pull/115462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Richard Smith
Date: 2024-11-13T15:38:44-08:00
New Revision: adfa6b762dc53bc53377785d824264a3311e829d
URL:
https://github.com/llvm/llvm-project/commit/adfa6b762dc53bc53377785d824264a3311e829d
DIFF:
https://github.com/llvm/llvm-project/commit/adfa6b762dc53bc53377785d824264a3311e829d.diff
@@ -8732,6 +8732,18 @@ void Sema::CheckVariableDeclarationType(VarDecl *NewVD) {
}
}
+ // zero sized static arrays are not allowed in HIP device functions
+ if (getLangOpts().HIP &&
+ DeclAttrsMatchCUDAMode(getLangOpts(), getCurFunctionDecl())) {
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/115777
>From 1e400acbd574703adcebd704c53991427815b090 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Tue, 12 Nov 2024 11:20:19 -0600
Subject: [PATCH 1/2] [Clang] Use TargetInfo when deciding is an address space
is
rjmccall wrote:
Oh, you can just forward-declare `class ASTContext` at the top of that file.
It's funny that that isn't already there.
https://github.com/llvm/llvm-project/pull/115777
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://
https://github.com/zygoloid closed
https://github.com/llvm/llvm-project/pull/116100
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 edited
https://github.com/llvm/llvm-project/pull/115051
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 requested changes to this pull request.
I don't think it is a good way to suppress all warning when we find gtest macro.
1. it will confuse user who really want to check even in test file.
2. the user can disable this rules in test folder or even use nolint the test
a-tarasyuk wrote:
@AaronBallman should the release notes be updated for such a minor change?
https://github.com/llvm/llvm-project/pull/116120
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Oleksandr T. (a-tarasyuk)
Changes
Fixes https://github.com/llvm/llvm-project/issues/116118
---
Full diff: https://github.com/llvm/llvm-project/pull/116120.diff
2 Files Affected:
- (modified) clang/include/clang/Basic/DiagnosticSemaKinds
https://github.com/a-tarasyuk created
https://github.com/llvm/llvm-project/pull/116120
Fixes https://github.com/llvm/llvm-project/issues/116118
>From d93fd0c7e9bceb7c4725cebc01375dd16d9ef58d Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Thu, 14 Nov 2024 01:35:28 +0200
Subject: [PATCH] [Clan
https://github.com/5chmidti requested changes to this pull request.
I don't think we can ignore a TU simply because it has the Google Test header
included, which this will do.
This would ignore real problems, such as
```c++
#include
#include
void issue(const absl::optional &opt) {
if (!opt.
rjmccall wrote:
I'm not sure what you're saying would need to pulled into a .cpp file. It
looks like there's only one actual call to `TI.isAddressSpaceSupersetOf`, so if
you just pass around an `ASTContext &` to that point, nothing else will need to
drill into it. And frankly that entire exp
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/115777
>From 1e400acbd574703adcebd704c53991427815b090 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Tue, 12 Nov 2024 11:20:19 -0600
Subject: [PATCH] [Clang] Use TargetInfo when deciding is an address space is
comp
jhuber6 wrote:
> > Okay the problem with using `ASTContext` here is that it creates some
> > recursive includes. ~I can do this by moving the check into `Type.cpp`
> > instead, so this will be function call instead of being inlined.~ This
> > would require a lot of extra stuff so I'm going to
carlosgalvezp wrote:
Apologies for the late response, I don't have much time to perform in-depth
reviews. Tagging additional reviewers @5chmidti @HerrCai0907
https://github.com/llvm/llvm-project/pull/115051
___
cfe-commits mailing list
cfe-commits@l
rjmccall wrote:
> Okay the problem with using `ASTContext` here is that it creates some
> recursive includes. ~I can do this by moving the check into `Type.cpp`
> instead, so this will be function call instead of being inlined.~ This would
> require a lot of extra stuff so I'm going to defer i
jhuber6 wrote:
Okay the problem with using `ASTContext` here is that it creates some recursive
includes. I can do this by moving the check into `Type.cpp` instead, so this
will be function call instead of being inlined.
https://github.com/llvm/llvm-project/pull/115777
_
https://github.com/nicovank edited
https://github.com/llvm/llvm-project/pull/116033
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/SpencerAbson edited
https://github.com/llvm/llvm-project/pull/116026
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jhuber6 wrote:
> Address spaces from language dialects generally have required relationships
> and behaviors in the language, and that really shouldn't be overridden by
> targets. However, targets do need to be able to decide how target-specific
> address spaces work, including how they intera
jhuber6 wrote:
So the main reason I want this is so that I don't need to keep passing
`-nogpulib` because targeting plain C/C++ should not require the ROCm device
libs.
https://github.com/llvm/llvm-project/pull/99687
___
cfe-commits mailing list
cfe-
rjmccall wrote:
Address spaces from language dialects generally have required relationships and
behaviors in the language, and that really shouldn't be overridden by targets.
However, targets do need to be able to decide how target-specific address
spaces work, including how they interact with
https://github.com/bob80905 updated
https://github.com/llvm/llvm-project/pull/115045
>From ef4a7eea3eacce4f77b628aebe7f2838733971d0 Mon Sep 17 00:00:00 2001
From: Joshua Batista
Date: Tue, 5 Nov 2024 10:35:59 -0800
Subject: [PATCH 1/6] add empty struct test cases
---
.../SemaHLSL/Types/Traits
https://github.com/t-rasmud updated
https://github.com/llvm/llvm-project/pull/114606
>From cc19550fdbaca4b77e90de57c472a31a8c3f8293 Mon Sep 17 00:00:00 2001
From: Rashmi Mudduluru
Date: Fri, 1 Nov 2024 14:10:50 -0700
Subject: [PATCH 1/6] [Webkit Checkers] Introduce a Webkit checker for memory
@@ -272,6 +277,13 @@ class RocmInstallationDetector {
return Loc->second;
}
+ void init(bool DetectHIPRuntime = true, bool DetectDeviceLib = false) {
jhuber6 wrote:
I don't understand why we need this, isn't it fine just to let it fail? Let the
detect
@@ -4063,6 +4063,11 @@ defm threadsafe_statics :
BoolFOption<"threadsafe-statics",
NegFlag,
PosFlag>;
+defm tls_guards : BoolFOption<"tls-guards",
+ LangOpts<"TlsGuards">, DefaultTrue,
efriedma-quic wrote:
My understanding is that if a tls variable is def
@@ -0,0 +1,117 @@
+//===- MemoryUnsafeCastChecker.cpp -*- C++ -*-==//
+//
+// 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: Apach
https://github.com/efriedma-quic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/113830
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1,109 +1,28 @@
// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.6-library
-finclude-default-header -fnative-half-type -verify %s
// expected-no-diagnostics
-struct oneInt {
-int i;
-};
-
-struct twoInt {
- int aa;
- int ab;
-};
-
-struct threeInts {
- oneInt o;
-
@@ -6440,7 +6440,8 @@ const ToolChain &Driver::getToolChain(const ArgList &Args,
TC = std::make_unique(*this, Target, Args);
break;
case llvm::Triple::AMDHSA:
- TC = std::make_unique(*this, Target, Args);
+ TC = std::make_unique(*this, Target, Args,
+
https://github.com/jhuber6 closed
https://github.com/llvm/llvm-project/pull/112248
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jhuber6 wrote:
ping
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
@@ -0,0 +1,22 @@
+// RUN: %clang_cc1 -finclude-default-header -triple
dxil-pc-shadermodel6.6-library %s -fnative-half-type -verify
+
+
+void test_arg_missing() {
+ __builtin_hlsl_elementwise_clip();
+ // expected-error@-1 {{too few arguments to function call, expected 1, have
0
@@ -483,10 +583,103 @@ static BuiltinTypeDeclBuilder
setupBufferType(CXXRecordDecl *Decl, Sema &S,
.addDefaultHandleConstructor(S, RC);
}
+BinaryOperator *constructSizeOfLEQ16Expr(ASTContext &Context,
+ SourceLocation NameLoc,
+
cyndyishida wrote:
ping
https://github.com/llvm/llvm-project/pull/115048
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -27,3 +27,25 @@ struct ST2 {
static uint X : SV_DispatchThreadID;
uint s : SV_DispatchThreadID;
};
+
+[numthreads(8,8,1)]
+// expected-error@+1 {{attribute 'SV_GroupID' only applies to a field or
parameter of type 'uint/uint2/uint3'}}
+void CSMain_GID(float ID : SV_
https://github.com/lizhengxing edited
https://github.com/llvm/llvm-project/pull/115911
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,21 @@
+// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.3-library -x hlsl -emit-llvm
-finclude-default-header -disable-llvm-passes -o - %s | FileCheck %s
+
+// Make sure SV_GroupID translated into dx.group.id.
+
+// CHECK: define void @foo()
+// CHECK: %[[#ID:]] = cal
https://github.com/lizhengxing updated
https://github.com/llvm/llvm-project/pull/115911
>From 6418461717614d5879688d32a0ab9bf9d9137328 Mon Sep 17 00:00:00 2001
From: Zhengxing Li
Date: Tue, 1 Oct 2024 15:13:34 -0700
Subject: [PATCH 1/2] [HLSL] Implement SV_GroupID semantic
Support SV_GroupID a
https://github.com/farzonl approved this pull request.
Minor comments, but rest LGTM. Code correct as ticket is currently defined. We
may want to move the behavior you currently have in cgbuiltins to
hlsl_intrinsics.h. That way for spirv instead of a basic block we could do a
c++ if statement
@@ -483,10 +583,103 @@ static BuiltinTypeDeclBuilder
setupBufferType(CXXRecordDecl *Decl, Sema &S,
.addDefaultHandleConstructor(S, RC);
}
+BinaryOperator *constructSizeOfLEQ16Expr(ASTContext &Context,
+ SourceLocation NameLoc,
+
@@ -770,6 +770,15 @@ def CheckAccessFullyMapped : DXILOp<71,
checkAccessFullyMapped> {
let stages = [Stages];
}
+def Discard : DXILOp<82, discard> {
+ let Doc = "discard the current pixel";
+ let LLVMIntrinsic = int_dx_discard;
+ let arguments = [Int1Ty];
---
@@ -483,10 +583,103 @@ static BuiltinTypeDeclBuilder
setupBufferType(CXXRecordDecl *Decl, Sema &S,
.addDefaultHandleConstructor(S, RC);
}
+BinaryOperator *constructSizeOfLEQ16Expr(ASTContext &Context,
+ SourceLocation NameLoc,
+
@@ -323,30 +324,129 @@ struct TemplateParameterListBuilder {
S.Context, Builder.Record->getDeclContext(), SourceLocation(),
SourceLocation(), /* TemplateDepth */ 0, Position,
&S.Context.Idents.get(Name, tok::TokenKind::identifier),
-/* Typename *
@@ -19180,6 +19221,11 @@ case Builtin::BI__builtin_hlsl_elementwise_isinf: {
"asuint operands types mismatch");
return handleHlslSplitdouble(E, this);
}
+ case Builtin::BI__builtin_hlsl_elementwise_clip:
+
farzonl wrote:
delete the space
htt
https://github.com/efriedma-quic updated
https://github.com/llvm/llvm-project/pull/113506
>From 7f259d75f18422fff05ac11526b5a0a1b73312fd Mon Sep 17 00:00:00 2001
From: Eli Friedman
Date: Wed, 23 Oct 2024 16:12:27 -0700
Subject: [PATCH] [clang codegen] Add CreateRuntimeFunction overload that tak
@@ -1,109 +1,28 @@
// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.6-library
-finclude-default-header -fnative-half-type -verify %s
// expected-no-diagnostics
-struct oneInt {
-int i;
-};
-
-struct twoInt {
- int aa;
- int ab;
-};
-
-struct threeInts {
- oneInt o;
-
@@ -1,109 +1,28 @@
// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.6-library
-finclude-default-header -fnative-half-type -verify %s
// expected-no-diagnostics
-struct oneInt {
-int i;
-};
-
-struct twoInt {
- int aa;
- int ab;
-};
-
-struct threeInts {
- oneInt o;
-
https://github.com/vvereschaka approved this pull request.
@justinfargnoli ,
I have checked the latest changes locally on the ubuntu expensive builder. They
works fine there, thank you. LGTM.
https://github.com/llvm/llvm-project/pull/113780
___
cfe-co
omjavaid wrote:
@5chmidti Sorry for the delay. I have tested this and it seems to compile on
windows msvc without any regressions.
https://github.com/llvm/llvm-project/pull/112168
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.l
@@ -3060,6 +3060,9 @@ Generic_GCC::Generic_GCC(const Driver &D, const
llvm::Triple &Triple,
: ToolChain(D, Triple, Args), GCCInstallation(D),
CudaInstallation(D, Triple, Args), RocmInstallation(D, Triple, Args) {
getProgramPaths().push_back(getDriver().Dir);
+ llv
https://github.com/Prabhuk updated
https://github.com/llvm/llvm-project/pull/111719
>From 05eab970b59c61b22e8afa1ed9381906ae925c03 Mon Sep 17 00:00:00 2001
From: prabhukr
Date: Wed, 9 Oct 2024 10:16:58 -0700
Subject: [PATCH 1/3] [clang] Introduce a new UEFI target predefine.
---
clang/lib/Bas
justinfargnoli wrote:
> Is it possible/easy to split
> https://github.com/llvm/llvm-project/commit/ef6a751a794c3e2d266cac0242629043ddd0d2aa
> into a new PR?
Certainly, @boomanaiden154! Please see #116108 :)
https://github.com/llvm/llvm-project/pull/113780
_
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/115545
>From be73e1600846f6026c03d2e3107b4237f54c51ac Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Fri, 8 Nov 2024 15:42:04 -0600
Subject: [PATCH 1/5] [Clang] Add support for scoped atomic thread fence
Summary:
P
https://github.com/malavikasamak closed
https://github.com/llvm/llvm-project/pull/115797
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Malavika Samak
Date: 2024-11-13T13:08:52-08:00
New Revision: 0dcb0acf8265e1486f4f3715cef01987af1391cd
URL:
https://github.com/llvm/llvm-project/commit/0dcb0acf8265e1486f4f3715cef01987af1391cd
DIFF:
https://github.com/llvm/llvm-project/commit/0dcb0acf8265e1486f4f3715cef01987af1391cd.diff
Author: erichkeane
Date: 2024-11-13T13:02:13-08:00
New Revision: 95fa5f39a0506948bd3c81842c7828d7892023cd
URL:
https://github.com/llvm/llvm-project/commit/95fa5f39a0506948bd3c81842c7828d7892023cd
DIFF:
https://github.com/llvm/llvm-project/commit/95fa5f39a0506948bd3c81842c7828d7892023cd.diff
LO
501 - 600 of 145173 matches
Mail list logo
