[PATCH] [ARM] Fix issue with SMLAL (Signed Multiply Accumulate Long) lowering

Tim Northover t.p.northover at gmail.com
Thu Jan 22 07:11:18 PST 2015


Hi Jyoti,

Thanks for updating the patch. The code looks good to me now.

> Should we include a new test case in which we have weird operation on prod as well ?


It would probably be a good idea; it's only one line different from the second one you're already adding I think. Feel free to commit with that test added.

Cheers.

Tim.


http://reviews.llvm.org/D6998

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






More information about the llvm-commits mailing list