[PATCH] D129743: [GlobalISel] Change widenScalar of G_FCONSTANT to mutate into G_CONSTANT.

[Amara Emerson via Phabricator via llvm-commits]

aemerson added a comment.

In D129743#3651111 <https://reviews.llvm.org/D129743#3651111>, @foad wrote:

>> Widening a G_FCONSTANT by extending and then generating G_FPTRUNC doesn't produce the same result all the time.
>
> That's only a problem if you care about the bit pattern of NaNs, right? It should be OK for all non-NaN values.

Yeah, but it seems better to handle this as a G_CONSTANT rather than have value dependent legalization.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129743