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

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 28 12:46:18 PST 2022


arsenm added a comment.

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


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

https://reviews.llvm.org/D6970



More information about the llvm-commits mailing list