[PATCH] D61525: AMDGPU] gfx1010: prefer V_MUL_LO_U32 over V_MUL_LO_I32

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 6 09:51:26 PDT 2019


rampitec added a comment.

In D61525#1490852 <https://reviews.llvm.org/D61525#1490852>, @rampitec wrote:

> In D61525#1490811 <https://reviews.llvm.org/D61525#1490811>, @arsenm wrote:
>
> > Can this just use the u32 version on all targets then?
>
>
> It's sp3 compatibility. I am bot sure u32 is also available in all ASICs.


OK, I have checked. All ASICs support u32 version. I will change it.


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

https://reviews.llvm.org/D61525





More information about the llvm-commits mailing list