[PATCH] [AArch64] Select lower fsub, fabs pattern to fabd on AArch64

Jyoti Allur jyoti.allur at samsung.com
Tue Dec 30 00:47:33 PST 2014


I would use a more generic name for the testcase as mentioned before. You could probably club the testcases of sadb, fabd into a single generic file something like arm64-neon-simd-vabs.ll. For the scalar versions you should find an appropriate place though.

We should probably check for complete machine instructions in the test output rather than just pneumonic, since we have added different patterns to handle various types of data.

Could you please modify to test these?

[PS] I would still get a go from either of reviewers before committing.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D6791

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list