[PATCH] D98844: [PowerPC] Enable fast-isel on AIX subtarget
Nemanja Ivanovic via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 19 21:25:25 PDT 2021
nemanjai added a comment.
This patch is missing context as well as a description. Can you please add both? It would be good to understand why we want FISEL on AIX. Perhaps some compile-time measurement on some significantly sized code. Also, some notes about how well this was tested would be useful. A different instruction selector is a significant change so it would be good to do some thorough testing at `-O0` including some binary compatibility testing (with GCC and/or the system compiler).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98844/new/
https://reviews.llvm.org/D98844
More information about the llvm-commits
mailing list