[llvm] [DirectX][DXIL] Distinguish return type for overload type resolution. (PR #85646)

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 18 10:58:48 PDT 2024


================
@@ -249,34 +249,39 @@ static FunctionType *getDXILOpFunctionType(const OpCodeProperty *Prop,
       ArgTys[0], ArrayRef<Type *>(&ArgTys[1], ArgTys.size() - 1), false);
 }
 
-static FunctionCallee getOrCreateDXILOpFunction(dxil::OpCode DXILOp,
-                                                Type *OverloadTy, Module &M) {
-  const OpCodeProperty *Prop = getOpCodeProperty(DXILOp);
+namespace llvm {
+namespace dxil {
+
+// Create an instruction that calls DXIL Op with return type, specified opcode,
+// and call arguments.
+// \param OpCode Opcode of the DXIL Op call constructed
+// \param ReturnTy Return type of the DXIL Op call constructed
+// \param OverloadTy Overload type of the DXIL Op call constructed
+// \ret DXIL Op call constructed
+CallInst *DXILOpBuilder::createDXILOpCall(dxil::OpCode OpCode, Type *ReturnTy,
----------------
bogner wrote:

Doc comments belong in the header rather than the implementation. Also, you'll want to use three slashes (`///`) so that doxygen picks it up.

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


More information about the llvm-commits mailing list