[PATCH] D79351: [MCA] Fixed a bug where loads and stores were sometimes incorrectly marked as depedent (PR45793).

[Matt Davis via Phabricator via llvm-commits]

mattd added a comment.
mattd added inline comments.


================
Comment at: llvm/include/llvm/MCA/HardwareUnits/LSUnit.h:61
 
-  ArrayRef<MemoryGroup *> getSuccessors() const { return Succ; }
-  unsigned getNumSuccessors() const { return Succ.size(); }
+  unsigned getNumSuccessors() const {
+    return OrderSucc.size() + DataSucc.size();
----------------
s/unsigned/size_t/  because SmallVectorBase::size returns a size_t.


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

https://reviews.llvm.org/D79351