[PATCH] D40696: Enable aggressive FMA on T99 and provide AArch64 option for other micro-arch's

[Joel Jones via Phabricator via llvm-commits]

joelkevinjones added a comment.

It looks like the latest changes address the earlier concerns expressed in the review comments:

- Use target feature: Present in AArch64.td, AArch64ISelLowering.h, AArch64ISelLowering.cpp, and AArch64Subtarget.h
- Simplified test case: Present in test/CodeGen/AArch64/fma-simple.ll
- Combined optimized/missed in .ll tests: Present in fma-aggressive.ll and fma-simple.ll

Is there anything else that is needed?


Repository:
  rL LLVM

https://reviews.llvm.org/D40696