[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