[llvm] [DXIL] Add DXIL SubArch to correspond to version number (PR #89125)

Damyan Pepper via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 18 12:04:24 PDT 2024


damyanp wrote:

> In the clang DXC driver mode we translate target profiles to triples (i.e. `ps_6_6` to `dxil-pc-shadermodel6.6-pixel`). Clang needs two changes to utilize this:

> * translate target profile to dxil version and shader model version.

In TripleTest.cpp I don't see any test cases that exercise a specific shadermodel being specified.  These are all of the form:

* dxil-unknown-shadermodel-amplification
* dxilv1.0-unknown-unknown

> * In the non-DXC-mode driver, we should propagate shader model version to dxil sub arch if the DXIL sub arch is not set.

I can't see anything in this change that does that?

Is this something that should be done in this PR, or is this happening elsewhere?


https://github.com/llvm/llvm-project/pull/89125


More information about the llvm-commits mailing list