[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