| Issue |
102963
|
| Summary |
[DXIL][Validator] Validator Version Specification
|
| Labels |
new issue
|
| Assignees |
|
| Reporter |
bharadwajy
|
My understanding is that validator version is derived from the shader model / DXIL version, and that the only time that it isn’t is if it’s overridden on the command line. There was some discussion recently about why you would ever want to override it other than specifically for preview versions and what it even means for the validator version to differ significantly from the shader model version. Currently the metadata you’re reading is only ever set by the command line override.
Presumably the “right” thing to do here is to derive the validator version from the other versions. However, if we want to maintain the current behaviour in clang it looks like it just defaults to 1.0 if the command line argument isn’t set (this seems wrong). I think there’s a valid argument to leaving this out of this change and having a discussion about what the correct behaviour here actually is before adding it.
_Originally posted by @bogner in https://github.com/llvm/llvm-project/pull/102079#discussion_r1713477403_
![]()
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
