[PATCH] D147732: [AMDGPU] Add f32 permlane{16, x16} builtin variants

[Jeffrey Byrnes via Phabricator via cfe-commits]

jrbyrnes added a comment.

In D147732#4249567 <https://reviews.llvm.org/D147732#4249567>, @rampitec wrote:

> Isn't it simpler to lower it to an existing int intrinsic and casts in clang?

Thanks for your comment Stas!

I think it would be ideal if clang inserted pure bitcasts for floats instead of fptoui when passed as operands to these builtins. My concern is -- Do you think we need to preserve the implicit casting behavior for compatibility?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147732