[PATCH] D81586: [ARM] Describe defs/uses of VLLDM and VLSTM

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 10 11:41:31 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMExpandPseudoInsts.cpp:1280
+  for (int I = 0; I < 7; ++I)
+    VLSTM.addReg(ARM::Q0 + I, RegState::Implicit | RegState::Undef);
 
----------------
RegState::Undef seems wrong: presumably the registers may actually contain data we need to preserve, or we wouldn't bother saving them in the first place.


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

https://reviews.llvm.org/D81586





More information about the llvm-commits mailing list