[PATCH] D51537: Extend hasStoreToStackSlot with list of FI accesses.

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 31 13:35:28 PDT 2018


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

Good idea! LGTM.



================
Comment at: lib/CodeGen/TargetInstrInfo.cpp:345
+
+  auto StartSize = Accesses.size();
   for (MachineInstr::mmo_iterator o = MI.memoperands_begin(),
----------------
the type isn't obvious from immediate context, so I wouldn't use `auto`. Same in `hasStoreToStackSlot`.


https://reviews.llvm.org/D51537





More information about the llvm-commits mailing list