================
@@ -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:

That's a good point. This should still happen to work since we treat "not 
overloaded" and "overloaded on return type" the same for the purposes of 
overloadKind in tryCreateOp, and I've updated the comment to say so.

The right thing to do, which I think should be done in a follow up, is to 
generate a table of the function signatures per opcode, which can then be 
queried by a simple switch over those records. I think we'll need that soon any 
way so I'll do that shortly.

https://github.com/llvm/llvm-project/pull/101250
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to