[PATCH] D12698: DAGCombiner: Replace store of FP constant after attemping store merges

[Sanjay Patel via llvm-commits]

spatel added a comment.

This appears to solve PR24654:
https://llvm.org/bugs/show_bug.cgi?id=24654

Can you split this patch into an NFC refactoring to add replaceStoreOfFPConstant() and then submit the logic changes as a separate patch? That would make it easier to read.

The AMDGPU FIXME test addition could be a separate commit too.


http://reviews.llvm.org/D12698