[PATCH] D74958: [ARM][MVE] Combine more extending masked loads

Fri Feb 21 06:42:37 PST 2020

dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

Sounds good to me

================
Comment at: llvm/test/CodeGen/Thumb2/LowOverheadLoops/extending-loads.ll:3
+
+; CHECK-NOT: vmovlb
+define dso_local arm_aapcs_vfpcc void @sext_i8(i16* noalias nocapture %a, i8* nocapture readonly %b, i32 %N) {
----------------
I wouldn't be against just using the update script for this kind of file. The results shouldn't be too long and it can be useful to show other inefficiencies and improvements. A single check-not line has the tendency to end up testing absolutely nothing over time. Up to you what you think though.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74958/new/

https://reviews.llvm.org/D74958