s-perron wrote:

> LGTM! I think changing the mangling for Vulkan is the best solution here.
> 
> One small comment, later on if/when you decide to use SPIRVBuiltins.cpp/.td 
> for lowering builtins, there might be additional changes needed. The lookup 
> function expects Itanium demangled strings.

Thanks we will keep that in mind.

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

Reply via email to