[PATCH] D34161: [AArch64] Add ARMv8.2-A FP16 vector intrinsics - Continuation

Abderrazek Zaafrani via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 16:49:52 PDT 2017


az added a comment.

Ok, I will upload the combination of the two patches here.

I am kind of undecided between your suggestion of using -O2 or my previous solution of  removing 'opt -S -mem2reg'. the former  makes the test more readable but unstable given that future changes to optimizations at -O2 level may result in updating the tests (but, I do no expect much changes given that they are small tests).


https://reviews.llvm.org/D34161





More information about the llvm-commits mailing list