[clang] 50581ef - [NFC] Fix warning in recent commit

Author: Chris Bieneman Date: 2025-02-15T14:19:31-06:00 New Revision: 50581ef1ee45815b9230043319de5ae93680d4ad URL: https://github.com/llvm/llvm-project/commit/50581ef1ee45815b9230043319de5ae93680d4ad DIFF: https://github.com/llvm/llvm-project/commit

[clang] 5b95931 - [NFC] Delete unintentionally added file

Author: Chris Bieneman Date: 2024-04-03T11:24:48-05:00 New Revision: 5b959310b0fae723bd119ed8815bf1cb1a8c67d4 URL: https://github.com/llvm/llvm-project/commit/5b959310b0fae723bd119ed8815bf1cb1a8c67d4 DIFF: https://github.com/llvm/llvm-project/commit/5b959310b0fae723bd119ed8815bf1cb1a8c67d4

[clang] f119a4f - [HLSL] Fix broken spir-v test

Author: Chris Bieneman Date: 2024-04-02T15:53:17-05:00 New Revision: f119a4ffb885ed588c46de1d51f4185572142ca2 URL: https://github.com/llvm/llvm-project/commit/f119a4ffb885ed588c46de1d51f4185572142ca2 DIFF: https://github.com/llvm/llvm-project/commit/f119a4ffb885ed588c46de1d51f4185572142ca2

[clang] 0065161 - Remove assert introduced in #71098

Author: Chris Bieneman Date: 2024-02-15T18:56:35-06:00 New Revision: 0065161c720c37e8ab545979aed6a03d944a3176 URL: https://github.com/llvm/llvm-project/commit/0065161c720c37e8ab545979aed6a03d944a3176 DIFF: https://github.com/llvm/llvm-project/commit/0065161c720c37e8ab545979aed6a03d944a3176

[clang] a1155f6 - [NFC] Clang-format const array declarations

Author: Chris Bieneman Date: 2024-02-15T13:16:21-06:00 New Revision: a1155f68f5d5c7013ca1deb312a2e1e4f71ef544 URL: https://github.com/llvm/llvm-project/commit/a1155f68f5d5c7013ca1deb312a2e1e4f71ef544 DIFF: https://github.com/llvm/llvm-project/commit/a1155f68f5d5c7013ca1deb312a2e1e4f71ef544

[clang] c2fd5b7 - [NFC] Remove trailing whitespace

Author: Chris Bieneman Date: 2024-01-11T12:34:51-06:00 New Revision: c2fd5b738e9700a515f1730c714897eeec064157 URL: https://github.com/llvm/llvm-project/commit/c2fd5b738e9700a515f1730c714897eeec064157 DIFF: https://github.com/llvm/llvm-project/commit/c2fd5b738e9700a515f1730c714897eeec064157

[clang] ff96567 - [NFC] Fix failing test introduced in #67700

Author: Chris Bieneman Date: 2023-11-29T12:02:44-06:00 New Revision: ff96567bd0fb8ed7a4739afa27a3cf3908115021 URL: https://github.com/llvm/llvm-project/commit/ff96567bd0fb8ed7a4739afa27a3cf3908115021 DIFF: https://github.com/llvm/llvm-project/commit/ff96567bd0fb8ed7a4739afa27a3cf3908115021

[clang] ce73177 - Fix clang Attribute doc build

Author: Chris Bieneman Date: 2023-11-28T15:30:15-06:00 New Revision: ce731770f869e8e61afa2dccb772e8367105a1a6 URL: https://github.com/llvm/llvm-project/commit/ce731770f869e8e61afa2dccb772e8367105a1a6 DIFF: https://github.com/llvm/llvm-project/commit/ce731770f869e8e61afa2dccb772e8367105a1a6

[PATCH] D159247: [HLSL] Cleanup support for `this` as an l-value

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG400d3261a0da: [HLSL] Cleanup support for `this` as an l-value (authored by beanz). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 400d326 - [HLSL] Cleanup support for `this` as an l-value

Author: Chris Bieneman Date: 2023-09-05T19:38:50-05:00 New Revision: 400d3261a0da56554aee8e5a2fbc27eade9d05db URL: https://github.com/llvm/llvm-project/commit/400d3261a0da56554aee8e5a2fbc27eade9d05db DIFF: https://github.com/llvm/llvm-project/commit

[PATCH] D159247: [HLSL] Cleanup support for `this` as an l-value

beanz updated this revision to Diff 555380. beanz added a comment. Updating based on PR feedback. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D159247/new/ https://reviews.llvm.org/D159247 Files: clang/include/clang/AST/ExprCXX.h clang/lib/AST

[PATCH] D159247: [HLSL] Cleanup support for `this` as an l-value

beanz updated this revision to Diff 555118. beanz added a comment. Updating based on review feedback from @core3ntin. Thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D159247/new/ https://reviews.llvm.org/D159247 Files: clang/include/clan

[PATCH] D159247: [HLSL] Cleanup support for `this` as an l-value

beanz added a comment. Yea, we could do that approach. It will mean filtering about a few `getNonReferenceType()` calls, which is what I was trying to avoid. That said, it might be the better solution since it can be unconditional and will work for both deducing `this` and HLSL. I'll test it o

[PATCH] D159247: [HLSL] Cleanup support for `this` as an l-value

beanz added a comment. Full disclosure here, I need to write some more tests for this. It should be NFC for all languages other than HLSL, but I want to spend some time making sure that the tests adequately exercise the HLSL paths (which should also be mostly NFC). Repository: rG LLVM Githu

[PATCH] D159247: [HLSL] Cleanup support for `this` as an l-value

beanz created this revision. beanz added reviewers: cor3ntin, aaron.ballman, bogner. Herald added subscribers: Anastasia, ChuanqiXu, martong. Herald added a reviewer: shafik. Herald added a reviewer: NoQ. Herald added a project: All. beanz requested review of this revision. Herald added a project:

[PATCH] D159126: [Clang] Add captures to the instantiation scope of lambda call operators

beanz added a comment. Yea, the gist of it is that in HLSL `this` is a reference not a pointer, which means the `CXXThisExpr` is always an LValue. I think the right fix for this is to cleanup the `CXXThisExpr` creation code and create a `CXXThisExpr::Create` method like the other AST nodes. The

[PATCH] D159126: [Clang] Add captures to the instantiation scope of lambda call operators

beanz added a comment. @cor3ntin, I know what the problem is and I think I can put up a review for a fix tonight or (more likely) tomorrow. Is that okay? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D159126/new/ https://reviews.llvm.org/D159126 _

[PATCH] D159126: [Clang] Add captures to the instantiation scope of lambda call operators

beanz added a comment. I'll apply this patch and debug the issue today. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D159126/new/ https://reviews.llvm.org/D159126 ___ cfe-commits mailing list cfe-commits

[PATCH] D158820: [Sema][HLSL] Fix naming of anyhit/closesthit shaders

beanz added inline comments. Comment at: clang/test/SemaHLSL/shader_type_attr.hlsl:30 // expected-error@+1 {{'shader' attribute parameters do not match the previous declaration}} +[shader("pixel")] bob80905 wrote: > bogner wrote: > > bob80905 wrote: > > > I d

[PATCH] D157149: [Option] Add "Visibility" field and clone the OptTable APIs to use it

beanz accepted this revision. beanz added a comment. LGTM. This is definitely an improvement over the awfulness we were doing. Thanks @bogner! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157149/new/ https://reviews.llvm.org/D157149

[PATCH] D156178: [HLSL] add pow library function

beanz added inline comments. Comment at: clang/test/CodeGenHLSL/builtins/pow.hlsl:6 +// RUN: dxil-pc-shadermodel6.3-library %s -emit-llvm -disable-llvm-passes \ +// RUN: -D__HLSL_ENABLE_16_BIT -o - | FileCheck %s --check-prefix=NO_HALF + Does this need to set

[PATCH] D151683: [clang] Enable C++11-style attributes in all language modes

beanz added inline comments. Comment at: clang/test/ParserHLSL/group_shared.hlsl:14 -// expected-error@+1 {{expected expression}} float groupshared [[]] i = 12; philnik wrote: > beanz wrote: > > aaron.ballman wrote: > > > philnik wrote: > > > > Should this als

[PATCH] D151683: [clang] Enable C++11-style attributes in all language modes

beanz added inline comments. Comment at: clang/test/ParserHLSL/group_shared.hlsl:14 -// expected-error@+1 {{expected expression}} float groupshared [[]] i = 12; aaron.ballman wrote: > philnik wrote: > > Should this also get an extension warning/should attribut

[PATCH] D143553: [Clang][CMake] Use perf-training for Clang-BOLT

beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM. Sorry for the delays reviewing! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143553/new/ https://reviews.llvm.org/D143553 ___

[PATCH] D149119: [CMake] Use LLVM own tools in extract_symbols.py

beanz added a comment. One potential area of concern here: If `llvm-driver` is ever extended to work as a plugin loader (thus exporting its symbols), removing support for the pre-installed host tools could cause a cyclic dependency. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149119/

[clang] c380156 - [NFC] [CMake] Add misssing dependencies

Author: Chris Bieneman Date: 2023-03-10T17:13:09-06:00 New Revision: c3801568ef2b7f923f9116f72a6b0bb382bcaa41 URL: https://github.com/llvm/llvm-project/commit/c3801568ef2b7f923f9116f72a6b0bb382bcaa41 DIFF: https://github.com/llvm/llvm-project/commit/c3801568ef2b7f923f9116f72a6b0bb382bcaa41

[clang] 326c135 - [NFC] [CMake] Cleanup clang tool dependencies

Author: Chris Bieneman Date: 2023-03-10T10:56:07-06:00 New Revision: 326c13531af32df8361cb4c98d2cc1355b0f8c48 URL: https://github.com/llvm/llvm-project/commit/326c13531af32df8361cb4c98d2cc1355b0f8c48 DIFF: https://github.com/llvm/llvm-project/commit/326c13531af32df8361cb4c98d2cc1355b0f8c48

[PATCH] D145270: Add codegen for llvm exp/exp2 elementwise builtins

beanz added inline comments. Comment at: clang/docs/LanguageExtensions.rst:642 T __builtin_elementwise_log10(T x) return the base 10 logarithm of x floating point types + T __builtin_elementwise_exp(T x)returns the base-e exp

[PATCH] D143208: Repair sphinx doc generation

beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D143208/new/ https://reviews.llvm.org/D143208 ___ c

[PATCH] D140489: Add builtin_elementwise_log

beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM. This looks pretty straightforward and similar to other changes you've been making. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140489/new/

[PATCH] D141705: [HLSL] [Dirver] add dxv as a Driver Action Job

beanz added inline comments. Comment at: clang/include/clang/Driver/Types.def:110 TYPE("api-information", API_INFO, INVALID, "json", phases::Precompile) +TYPE("dx-container", DX_CONTAINER, INVALID, "dxc", phases::Compile, phases::B

[PATCH] D141705: [HLSL] [Dirver] add dxv as a Driver Action Job

beanz added inline comments. Comment at: clang/include/clang/Basic/DiagnosticDriverKinds.td:687 +def warn_drv_dxc_missing_dxv : Warning<"dxv not found." +" Resulting DXIL will not be signed for use in release environments.">; Not all `dxv` binaries can sign

[PATCH] D136031: [DirectX backend] support ConstantBuffer to DXILResource.h

beanz added a comment. @python3kgae, this change introduced a bunch of warning spew because it is using an API that was deprecated shortly before the change merged. Can you please address this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136031/

[PATCH] D131052: [CMake] Allow setting the location of host tools with LLVM_NATIVE_TOOL_DIR

beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131052/new/ https://reviews.llvm.org/D131052 ___

[PATCH] D141705: [HLSL] [Dirver] add dxv as a VerifyDebug Job

beanz added inline comments. Comment at: clang/lib/Driver/Driver.cpp:4215 + // Call validator for dxc. + if (IsDXCMode()) { +Action *LastAction = Actions.back(); Shouldn't the validator only run if we are targeting DXIL? Also we should probably add the `-V

[PATCH] D141705: [HLSL] [Dirver] add dxv as a VerifyDebug Job

beanz added a comment. Re-using the `VerifyDebug` action really doesn't make sense. That's not what the DXIL validator does, and it will be a source of confusion forever. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141705/new/ https://reviews.ll

[PATCH] D131052: [CMake] Allow setting the location of host tools with LLVM_NATIVE_TOOL_DIR

beanz added a comment. The convention that `find_program` uses is to cache the variables, which causes them to be defined at global scope. That also avoids needing to recompute filesystem lookups in incremental builds, which is desirable. Repository: rG LLVM Github Monorepo CHANGES SINCE LA

[PATCH] D131052: [CMake] Allow setting the location of host tools with LLVM_NATIVE_TOOL_DIR

beanz added inline comments. Comment at: llvm/cmake/modules/AddLLVM.cmake:2401 + +macro(setup_host_tool tool_name setting_name exe_var_name target_var_name) + cmake_parse_arguments(ARG "" "SCOPE" "" ${ARGN}) Please make this a `function` instead of a `macro`. In

[PATCH] D136031: [DirectX backend] support ConstantBuffer to DXILResource.h

beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136031/new/ https://reviews.llvm.org/D136031 ___ c

[PATCH] D136031: [DirectX backend] support ConstantBuffer to DXILResource.h

beanz added a comment. It is probably worth adding some unit tests to test the `CBufferDataLayout` class. I think the meat of this change is fine. This code mixes `unsigned` and `uint32_t` interchangeably. They aren't required by the language to be the same. I have a general preference toward

[PATCH] D139137: add floor library function

beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM. @bob80905 if you haven't already, you should follow the steps here (https://www.llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access) to get commit access. Repository: rG LLVM Gi

[PATCH] D135721: [HLSL] Added HLSL this as a reference

beanz added inline comments. Comment at: clang/test/AST/HLSL/RWBuffer-AST.hlsl:49-50 // CHECK-NEXT: ArraySubscriptExpr 0x{{[0-9A-Fa-f]+}} <> 'element_type' lvalue -// CHECK-NEXT: MemberExpr 0x{{[0-9A-Fa-f]+}} <> 'element_type *' lvalue ->h 0x{{[0-9A-Fa-f]+}} -// CHECK-NEXT: CX

[PATCH] D135721: [HLSL] Added HLSL this as a reference

beanz added inline comments. Comment at: clang/lib/Sema/HLSLExternalSemaSource.cpp:179 +AST, SourceLocation(), +Constructor->getThisType().getTypePtr()->getPointeeType(), true); +This->setValueKind(ExprValueKind::VK_LValue); aaron.ballman wrot

[PATCH] D136271: [HLSL] Remove unused frontend-generated ID

This revision was automatically updated to reflect the committed changes. Closed by commit rG4c7218e77026: [HLSL] Remove unused frontend-generated ID (authored by beanz). Changed prior to commit: https://reviews.llvm.org/D136271?vs=468961&id=469681#toc Repository: rG LLVM Github Monorepo CH

[clang] 4c7218e - [HLSL] Remove unused frontend-generated ID

Author: Chris Bieneman Date: 2022-10-21T12:41:09-05:00 New Revision: 4c7218e7702651bb8dd2ef7734ca05ac0ada32fd URL: https://github.com/llvm/llvm-project/commit/4c7218e7702651bb8dd2ef7734ca05ac0ada32fd DIFF: https://github.com/llvm/llvm-project/commit

[PATCH] D136134: [NFC] [DirectX backend] move ResourceClass into llvm.

beanz added inline comments. Comment at: clang/lib/Sema/HLSLExternalSemaSource.cpp:20 #include "clang/Sema/Sema.h" +#include "llvm/Frontend/HLSL/HLSLResource.h" python3kgae wrote: > python3kgae wrote: > > beanz wrote: > > > You need to add FrontendHLSL to the

[PATCH] D136134: [NFC] [DirectX backend] move ResourceClass into llvm.

beanz added inline comments. Comment at: clang/lib/Sema/HLSLExternalSemaSource.cpp:20 #include "clang/Sema/Sema.h" +#include "llvm/Frontend/HLSL/HLSLResource.h" You need to add FrontendHLSL to the Sema/CMakeLists.txt file too. Repository: rG LLVM Github Mo

[PATCH] D136271: [HLSL] Remove unused frontend-generated ID

beanz created this revision. beanz added a reviewer: python3kgae. Herald added subscribers: Anastasia, hiraditya. Herald added a project: All. beanz requested review of this revision. Herald added projects: clang, LLVM. Herald added a subscriber: llvm-commits. As @python3kgae pointed out we're goi

[clang] d146a52 - Move HLSL builtins into hlsl namespace

Author: Chris Bieneman Date: 2022-10-19T10:59:31-05:00 New Revision: d146a5241c5039fd25e8bf5a4ae1200ad201d06c URL: https://github.com/llvm/llvm-project/commit/d146a5241c5039fd25e8bf5a4ae1200ad201d06c DIFF: https://github.com/llvm/llvm-project/commit

[PATCH] D135973: Move HLSL builtins into hlsl namespace

This revision was automatically updated to reflect the committed changes. Closed by commit rGd146a5241c50: Move HLSL builtins into hlsl namespace (authored by beanz). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135973/new/ https://reviews.llvm.org

[PATCH] D135429: [HLSL] [DirectX backend] Move generateGlobalCtorDtorCalls into DirectX backend.

beanz added inline comments. Comment at: clang/lib/CodeGen/CGHLSLRuntime.cpp:193 - -void CGHLSLRuntime::generateGlobalCtorDtorCalls() { - llvm::Module &M = CGM.getModule(); I think you've been a bit too aggressive about what code you're moving. The code that ge

[PATCH] D135429: [HLSL] [DirectX backend] Move generateGlobalCtorDtorCalls into DirectX backend.

beanz added a comment. In addition to restoring the test cases that you deleted, I think you should also reduce and simplify the test cases you're adding. These new tests are very large blobs of IR generated by clang. They include a lot of extra instructions that aren't needed to exercise the c

[PATCH] D135429: [HLSL] [DirectX backend] Move generateGlobalCtorDtorCalls into DirectX backend.

beanz added a comment. I don’t think you should be deleting all those frontend tests. Those tests existed before your change to remove the global constructor global variable, they provide valuable coverage of the frontend constructor generation. Repository: rG LLVM Github Monorepo CHANGES S

[PATCH] D133983: [HLSL] Add SV_DispatchThreadID

beanz accepted this revision. beanz added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133983/new/ https://reviews.llvm.org/D133983 ___ cfe-commits mailing list cfe-commits@lists.llvm.or

[PATCH] D135110: [NFC] [HLSL] Move common metadata to LLVMFrontend

beanz added a comment. The llvm-config test issue should be resolved in rGa4b010034f57 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135110/new/ https://reviews.llvm.org/D135110

[PATCH] D135110: [NFC] [HLSL] Move common metadata to LLVMFrontend

beanz added a comment. I think I have a fix for this and I'll get it up today. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135110/new/ https://reviews.llvm.org/D135110 ___ cfe-commits mailing list cfe-

[PATCH] D130951: [HLSL] CodeGen hlsl resource binding.

beanz accepted this revision. beanz added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/CodeGen/CGHLSLRuntime.h:73 CodeGenModule &CGM; uint32_t ResourceCounters[static_cast( hlsl::ResourceClass::NumClasses)] = {0}; --

[PATCH] D130951: [HLSL] CodeGen hlsl resource binding.

beanz added inline comments. Comment at: clang/lib/CodeGen/CGHLSLRuntime.h:73 CodeGenModule &CGM; uint32_t ResourceCounters[static_cast( hlsl::ResourceClass::NumClasses)] = {0}; The `ResourceCounters` here was a stand-in for allocating resource indic

[PATCH] D130951: [HLSL] CodeGen hlsl resource binding.

beanz added inline comments. Comment at: clang/include/clang/Basic/HLSLRuntime.h:31 +// NOTE: keep sync with ResourceBase::Kinds in DirectX backend. +enum class ResourceKind : uint32_t { + Invalid = 0, If this is only used in the clangCodeGen library, we can mov

[PATCH] D135110: [NFC] [HLSL] Move common metadata to LLVMFrontend

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG911d2dc23035: [NFC] [HLSL] Move common metadata to LLVMFrontend (authored by beanz). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTIO

[clang] 911d2dc - [NFC] [HLSL] Move common metadata to LLVMFrontend

Author: Chris Bieneman Date: 2022-10-14T13:40:04-05:00 New Revision: 911d2dc23035454cb85422922c19259855e33bba URL: https://github.com/llvm/llvm-project/commit/911d2dc23035454cb85422922c19259855e33bba DIFF: https://github.com/llvm/llvm-project/commit

[PATCH] D135973: Move HLSL builtins into hlsl namespace

beanz created this revision. beanz added reviewers: python3kgae, pow2clk, bob80905. Herald added a subscriber: Anastasia. Herald added a project: All. beanz requested review of this revision. Herald added a project: clang. Should have done this from the start. Since all the injected AST types are

[PATCH] D135110: [NFC] [HLSL] Move common metadata to LLVMFrontend

beanz added inline comments. Comment at: llvm/lib/Target/DirectX/DXILResource.h:46 // can only be added to the end, and not removed. enum class Kinds : uint32_t { Invalid = 0, python3kgae wrote: > Could we move ResourceBase::Kinds to Frontend/HLSL/HLSL

[PATCH] D135110: [NFC] [HLSL] Move common metadata to LLVMFrontend

beanz updated this revision to Diff 467606. beanz added a comment. Rebasing Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135110/new/ https://reviews.llvm.org/D135110 Files: clang/lib/CodeGen/CGHLSLRuntime.cpp clang/lib/CodeGen/CMakeLists.txt

[PATCH] D135060: [HLSL] Add groupshare address space.

beanz added inline comments. Comment at: clang/test/ParserHLSL/group_shared.hlsl:11 +// expected-warning@+1 {{'auto' type specifier is a C++11 extension}} +auto l = []() groupshared {}; + What happens to this if you set the language standard to hlsl 202x? I setu

[PATCH] D135060: [HLSL] Add groupshare address space.

beanz added inline comments. Comment at: clang/include/clang/Basic/Attr.td:4063 + let Spellings = [Keyword<"groupshared">]; + let Documentation = [HLSLGroupSharedAddressSpaceDocs]; +} Shouldn't this have `let Subjects = SubjectList<[Var]>;`? I don't think we s

[PATCH] D135595: [HLSL] Add utility to convert environment to stage

This revision was automatically updated to reflect the committed changes. Closed by commit rG19a0a5674911: [HLSL] Add utility to convert environment to stage (authored by beanz). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm

[clang] 19a0a56 - [HLSL] Add utility to convert environment to stage

Author: Chris Bieneman Date: 2022-10-12T16:31:30-05:00 New Revision: 19a0a56749110dc92b39f583e46779ff23aceeba URL: https://github.com/llvm/llvm-project/commit/19a0a56749110dc92b39f583e46779ff23aceeba DIFF: https://github.com/llvm/llvm-project/commit

[PATCH] D135595: [HLSL] Add utility to convert environment to stage

beanz added inline comments. Comment at: clang/include/clang/Basic/HLSLRuntime.h:39 + return static_cast(Pipeline); +} + bogner wrote: > You're not actually introducing the dependency here (it was already there), > but neither `ShaderStage` in LangOptions.h nor

[PATCH] D135721: [HLSL] Added HLSL this as a reference

beanz added a reviewer: aaron.ballman. beanz added a subscriber: aaron.ballman. beanz added a comment. Looping in @aaron.ballman here too because this is a bit of a fun one... I know Aaron loves when we show him the best of HLSL 😄 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D135595: [HLSL] Add utility to convert environment to stage

beanz added a comment. We should absolutely add utility functions when we have uses for them. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135595/new/ https://reviews.llvm.org/D135595 ___ cfe-commits ma

[PATCH] D133668: [HLSL] Use _BitInt(16) for int16_t to avoid promote to int.

beanz added a comment. In D133668#3847871 , @rjmccall wrote: > But that's purely on the implementation level, right? Everything is > implicitly vectorized and you're just specifying the computation of a single > lane, but as far as that lane-wise compu

[PATCH] D133668: [HLSL] Use _BitInt(16) for int16_t to avoid promote to int.

beanz added a comment. In D133668#3847163 , @rjmccall wrote: > Sure, but it's extremely easy to unpromote that arithmetic for most > operators, and I'm sure LLVM already has a pass that will do that. Okay... but HLSL explicitly doesn't promote. Having t

[PATCH] D135595: [HLSL] Add utility to convert environment to stage

beanz added a comment. Worth noting, we do have a similar set of static_asserts in Triple.cpp to validate the ordering of enum cases and that the subtraction results in the appropriate values: https://github.com/llvm/llvm-project/blob/main/llvm/lib/Support/Triple.cpp#L1942 Repository: rG LL

[PATCH] D135595: [HLSL] Add utility to convert environment to stage

beanz added a comment. In D135595#3847056 , @python3kgae wrote: > Backend needs the same thing. > Is it possible to move this to llvm and share it between frontend and backend? This translates the triple to a clang-defined enum... so strictly speaking n

[PATCH] D135595: [HLSL] Add utility to convert environment to stage

beanz created this revision. beanz added reviewers: aaron.ballman, MaskRay, klimek, python3kgae, pow2clk. Herald added subscribers: Anastasia, StephenFan. Herald added a project: All. beanz requested review of this revision. Herald added a project: clang. We had a bunch of places in the code where

[PATCH] D133668: [HLSL] Use _BitInt(16) for int16_t to avoid promote to int.

beanz added a comment. Avoiding argument promotion is one part of what we need, but not all of it. For example if you take this trial code: const RWBuffer In; RWBuffer Out; [numthreads(1,1,1)] void main(uint GI : SV_GroupIndex) { Out[GI] = In[GI].x + In[GI].y; } Following C rul

[PATCH] D133983: [HLSL] Add SV_DispatchThreadID

beanz added inline comments. Comment at: clang/lib/CodeGen/CGHLSLRuntime.cpp:135 +llvm::Function *DxThreadID = CGM.getIntrinsic(Intrinsic::dx_thread_id); +// dx_thread_id +return buildVectorInput(B, DxThreadID, Ty); nit: this comment doesn't add value

[clang] bb46022 - [CMake] Small fix to HLSL header install paths

Author: Chris Bieneman Date: 2022-10-07T09:45:11-05:00 New Revision: bb46022abe78497d3d5a8c6a6cf8599f7ec0d2ec URL: https://github.com/llvm/llvm-project/commit/bb46022abe78497d3d5a8c6a6cf8599f7ec0d2ec DIFF: https://github.com/llvm/llvm-project/commit

[PATCH] D133993: [HLSL] Remove global ctor/dtor variable for non-lib profile.

beanz added a comment. @efriedma, I think that's a great suggestion. @python3kgae, if you don't adjust this patch, can you file an issue for that so that we don't lose that feedback? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133993/new/ https:

[PATCH] D133993: [HLSL] Remove global ctor/dtor variable for non-lib profile.

beanz added a comment. In D133993#3816526 , @efriedma wrote: > Why are we using different mechanisms for global constructors in "libraries" > vs. other code? If we need a mechanism in LLVM already, we might as well use > it all the time? To elaborate

[PATCH] D130131: [HLSL] CodeGen hlsl cbuffer/tbuffer.

beanz added a reviewer: asl. beanz added a subscriber: asl. beanz added a comment. +@asl for codegen owner perspective. This LGTM too. The changes here are well isolated to HLSL, so they should have no adverse impact on other language support. @efriedma, @asl & @rjmccall any feedback? Reposit

[PATCH] D134330: [Docs] [HLSL] Add note about PCH support

beanz added inline comments. Comment at: clang/docs/HLSL/HLSLSupport.rst:98 +``HLSLExternalSemaSource`` will create new decls and use the old decls as +argument for setPreviousDecl. + We can probably generalize this to something like: ``` When precompiled headers

[PATCH] D135110: [NFC] [HLSL] Move common metadata to LLVMFrontend

beanz added a comment. In D135110#3832728 , @tschuett wrote: > The OpenMP frontend is mainly an IRBuilder. It is a different layering than > for HLSL. Are there plans for an HLSL(IR)Builder? HLSL and OpenMP are different in a lot of ways. HLSL's code ge

[PATCH] D135110: [NFC] [HLSL] Move common metadata to LLVMFrontend

beanz created this revision. beanz added reviewers: python3kgae, tschuett, jdoerfert, bogner, pow2clk, tex3d, Anastasia, efriedma. Herald added a subscriber: hiraditya. Herald added a project: All. beanz requested review of this revision. Herald added projects: clang, LLVM. This change pulls some

[PATCH] D133983: [HLSL] Add SV_DispatchThreadID

beanz added inline comments. Comment at: clang/lib/CodeGen/CGHLSLRuntime.cpp:167 + for (const auto &Param : Fn->args()) { +if (Param.hasStructRetAttr()) { + // FIXME: support output. python3kgae wrote: > beanz wrote: > > I might be missing something, bu

[PATCH] D133983: [HLSL] Add SV_DispatchThreadID

beanz added inline comments. Comment at: clang/lib/CodeGen/CGHLSLRuntime.cpp:167 + for (const auto &Param : Fn->args()) { +if (Param.hasStructRetAttr()) { + // FIXME: support output. I might be missing something, but I'm not seeing a test that exercises

[PATCH] D134637: clang-tblgen build: avoid duplicate inclusion of libLLVMSupport

beanz added a comment. I think this approach mostly looks sane to me. @phosek, and @Ericson2314 may have different feedback. Comment at: clang/lib/Support/CMakeLists.txt:23 + # libLLVM-*.so). + llvm_add_library(clangSupport_tablegen +STATIC Unless there

[clang] e432108 - [CMake] Add `CLANG_ENABLE_HLSL` CMake option

Author: Chris Bieneman Date: 2022-09-27T12:33:22-05:00 New Revision: e432108bf25456f428a3956fd79d29ea7fc196ac URL: https://github.com/llvm/llvm-project/commit/e432108bf25456f428a3956fd79d29ea7fc196ac DIFF: https://github.com/llvm/llvm-project/commit

[PATCH] D134693: [CMake] Add `CLANG_ENABLE_HLSL` CMake option

This revision was automatically updated to reflect the committed changes. Closed by commit rGe432108bf254: [CMake] Add `CLANG_ENABLE_HLSL` CMake option (authored by beanz). Changed prior to commit: https://reviews.llvm.org/D134693?vs=463079&id=463275#toc Repository: rG LLVM Github Monorepo

[PATCH] D134693: [CMake] Add `CLANG_ENABLE_HLSL` CMake option

beanz created this revision. beanz added reviewers: phosek, smeenai, compnerd. Herald added a subscriber: Anastasia. Herald added a project: All. beanz requested review of this revision. Herald added a project: clang. The HLSL support in clang is in proress and not fully functioning. As such we do

[clang] d20f9f8 - [Docs] [HLSL] Add IR reference for HLSL

Author: Chris Bieneman Date: 2022-09-23T10:29:54-05:00 New Revision: d20f9f8d2177cd02184bb2c2794af398ba355f61 URL: https://github.com/llvm/llvm-project/commit/d20f9f8d2177cd02184bb2c2794af398ba355f61 DIFF: https://github.com/llvm/llvm-project/commit/d20f9f8d2177cd02184bb2c2794af398ba355f61

[PATCH] D134304: [Docs] [HLSL] Add IR reference for HLSL

This revision was automatically updated to reflect the committed changes. Closed by commit rGd20f9f8d2177: [Docs] [HLSL] Add IR reference for HLSL (authored by beanz). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134304/new/ https://reviews.llvm.or

[PATCH] D134304: [Docs] [HLSL] Add IR reference for HLSL

beanz updated this revision to Diff 462299. beanz added a comment. Updating documentation for `hlsl.uavs`, thanks @python3kgae! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134304/new/ https://reviews.llvm.org/D134304 Files: clang/docs/HLSL/HLS

[PATCH] D131799: [HLSL] clang codeGen for HLSLNumThreadsAttr

beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131799/new/ https://reviews.llvm.org/D131799 ___ c

[PATCH] D130131: [HLSL] CodeGen hlsl cbuffer/tbuffer.

beanz added a comment. This is looking great. Thank you for all your work iterating on this. Please update the description to reflect the changes in implementation. Comment at: clang/lib/CodeGen/CGHLSLRuntime.cpp:120 + +void addResourceBinding(GlobalVariable *GV, CGHLSLRuntime

[PATCH] D134326: [HLSL] Allow SV_GroupIndex for lib profile.

beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134326/new/ https://reviews.llvm.org/D134326 ___

[PATCH] D134326: [HLSL] Allow SV_GroupIndex for lib profile.

beanz added inline comments. Comment at: clang/lib/Sema/SemaDeclAttr.cpp:6901 +// FIXME: it is OK for a compute shader entry and pixel shader entry live in +// same HLSL file. uint32_t Pipeline = I think the underlying issue here is that some attrib

[clang] bc97751 - [NFC] Add GitHub issues to HLSL FIXME comments

Author: Chris Bieneman Date: 2022-09-21T10:31:25-05:00 New Revision: bc97751a236e619dc33f81e9fc9b3fe6bd0ba7b6 URL: https://github.com/llvm/llvm-project/commit/bc97751a236e619dc33f81e9fc9b3fe6bd0ba7b6 DIFF: https://github.com/llvm/llvm-project/commit/bc97751a236e619dc33f81e9fc9b3fe6bd0ba7b6

[PATCH] D134319: [HLSL] add ceil library function

beanz added a comment. For this builtin you should be able to use `__builtin_elementwise_ceil` for all the overload cases, and you should be able to add the vector cases too. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134319/new/ https://review

  1   2   3   4   5   6   7   8   9   10   >