[PATCH] D104874: [AMDGPU] Add S_MOV_B64_IMM_PSEUDO for fp constants
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 24 15:17:49 PDT 2021
rampitec added a comment.
In D104874#2839691 <https://reviews.llvm.org/D104874#2839691>, @arsenm wrote:
> Do you have a MIR example that fails to rematerialize as is? We split so many 64-bit operations it might be worthwhile just teaching the coalescer how to handle it
We fail to rematerialize this newly added test remat-fp64-constants.ll, it shall be fairly easy to stop it before RA.
I agree if we can teach RA to materialize subregs it would be the best, but honestly I have failed.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104874/new/
https://reviews.llvm.org/D104874
More information about the llvm-commits
mailing list