[PATCH] D48828: [InstSimplify] fold extracting from std::pair
Hiroshi Inoue via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 17 05:51:14 PDT 2018
inouehrs added a comment.
@lebedev.ri @spatel
Thak you so much for the advices. I avoid mentioning about C++ in the comment.
@lebedev.ri
> Use ./utils/update_test_checks.py please.
I updated the test cases using update_test_checks.py and moved them into existing ll files since `pair.ll` is no longer good file name.
@spatel
The simplified test cases can be optimized by current instcombine pass. But to enable jump threading for std::pair, which is the original motivation of the patch, we must apply this folding over a phi node as discussed in https://reviews.llvm.org/D44626.
Executing jump threading pass again after CFG simplification may catch this opportunity. But I think it is better to do jump threading early to help other optimizers.
https://reviews.llvm.org/D48828
More information about the llvm-commits
mailing list