================
@@ -948,15 +948,16 @@ void CGHLSLRuntime::emitEntryFunction(const FunctionDecl 
*FD,
     OB.emplace_back("convergencectrl", bundleArgs);
   }
 
-  llvm::DenseMap<const DeclaratorDecl *, llvm::Value *> OutputSemantic;
+  llvm::DenseMap<const DeclaratorDecl *, std::pair<llvm::Value *, llvm::Type 
*>>
----------------
vtjnash wrote:

There appears to be 0-2 or more entries (having sret does not imply that 
returning a value, or marking more than one argument as sret, is allowed to 
crash llvm).

I don't know why someone didn't use SmallDenseMap optimization, but that sort 
of change is not really fit to get bundled in this PR

https://github.com/llvm/llvm-project/pull/181740
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to