[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