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

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 21 06:51:37 PST 2020


samparker marked an inline comment as done.
samparker added inline comments.


================
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) {
----------------
dmgreen wrote:
> 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.
ok, will do


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74958





More information about the llvm-commits mailing list