[llvm] [DirectX] Fix DXIL part header version encoding (PR #91506)

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Thu May 9 15:47:04 PDT 2024


================
@@ -1510,6 +1510,8 @@ VersionTuple Triple::getDXILVersion() const {
   if (getArch() != dxil || getOS() != ShaderModel)
     llvm_unreachable("invalid DXIL triple");
   StringRef Arch = getArchName();
+  if (Arch == "dxil")
----------------
bogner wrote:

I think you can say `getSubArch() == NoSubArch` here instead of using string comparison

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


More information about the llvm-commits mailing list