[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