[PATCH] D60601: [DAGCombiner] Exploiting more about the transformation of TransformFPLoadStorePair function

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 12 12:47:34 PDT 2019


efriedma added a comment.

I can't find any previous discussion of this particular check.  I agree it isn't necessary for correctness.

Do you have any benchmarks showing this actually helps?  I'm a little concerned that this might not be a performance win in practice due to register pressure.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60601





More information about the llvm-commits mailing list