[PATCH] D22822: Adjust coercion of aggregates on RenderScript

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 27 07:26:11 PDT 2016


t.p.northover added a comment.

> I can talk to the rest of the RS team to see if they would mind passing the information differently going forward,


It seems like it would improve RenderScript if they could. I'm not sure how common arguments like this are, but pessimizing them would be a shame.

> but existing code/devices has to work this way.


If the RenderScript runtime changes that happens without putting this patch into Clang doesn't it?

If you've got IR already then Clang isn't involved, and trying to generate IR to run on an older runtime isn't going to work for many other reasons besides this (LLVM IR isn't compatible like that).


https://reviews.llvm.org/D22822





More information about the llvm-commits mailing list