[PATCH] D6970: R600/SI: Try to form (f64 [s|u]int_to_fp i1)

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 28 12:52:16 PST 2022


rampitec added a comment.

In D6970#3954790 <https://reviews.llvm.org/D6970#3954790>, @arsenm wrote:

> In D6970#3954786 <https://reviews.llvm.org/D6970#3954786>, @rampitec wrote:
>
>> Looks reasonable, but description mentions R600/SI while it affects other targets. I.e. it needs to be retitled.
>
> It's slower assuming quarter rate conversions, which is what the scheduler thinks is the case. I was thinking of just abandoning this

Actually GFX11SpeedModel suggests it is even slower...


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

https://reviews.llvm.org/D6970



More information about the llvm-commits mailing list