arsenm added a comment.

In D86154#2224270 <https://reviews.llvm.org/D86154#2224270>, @nhaehnle wrote:

> Note that part of my motivation here over D84639 
> <https://reviews.llvm.org/D84639> is to support more general types on the 
> lane intrinsics, since they also express some semantic content which would be 
> interesting to be able to express e.g. on descriptors. I wasn't able to bend 
> the SelectionDAG type legalization to my will, so that's why I instead 
> "legalize" the intrinsics in the AMDGPUCodeGenPrepare pass.

Don't you just need to handle this in ReplaceNodeResults the same way?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86154/new/

https://reviews.llvm.org/D86154

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to