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

Jyoti Allur jyoti.allur at samsung.com
Mon Dec 29 07:40:54 PST 2014


Again, same comment as for other patch.

FABD Vd.<T>, Vn.<T>, Vm.<T> 
Subtracts the elements of Vm from the corresponding
elements of Vn, and places the absolute values of the results in the elements of Vd

Rm, Rn should swap thier places i think.
I was also thinking if we should have a more generic name for the testcase to cover all vector arithmetic instructions involving abs.
Would like to know what James and Tim's thoughts are on this.
LGTM, otherwise.


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