[llvm-branch-commits] [llvm] [DirectX] Make DXILOpBuilder's API more useable (PR #101250)

[Justin Bogner via llvm-branch-commits]

================
@@ -151,7 +151,11 @@ DXILOperationDesc::DXILOperationDesc(const Record *R) {
         assert(knownType && "Specification of multiple differing overload "
                             "parameter types not yet supported");
       } else {
-        OverloadParamIndices.push_back(i);
+        // Skip the return value - nothing is overloaded on only return, and it
----------------
bogner wrote:

Thinking about this some more I don't think we can get away with this - if an op is overloaded on return type we simply need to provide the one we want. I think I'll add a `Type *` argument that's defaulted to null for "infer it" and return an error if we can't do so.

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