[PATCH] D51925: [AMDGPU] Fix issue for zext of f16 to i32

David Stuttard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 28 03:28:25 PST 2018


dstuttard added a comment.

In D51925#1309875 <https://reviews.llvm.org/D51925#1309875>, @arsenm wrote:

> In D51925#1309692 <https://reviews.llvm.org/D51925#1309692>, @dstuttard wrote:
>
> > ping
>
>
> What happens if you just drop the optimization entirely?


Not sure what you mean - I get a load of lit test failures (31), but that's what I'd expect since it no longer does the transform to use FP16_ZEXT - unless you mean something else?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D51925





More information about the llvm-commits mailing list