[PATCH] D74605: [ARM][RDA] add getUniqueReachingMIDef

[Sam Parker via Phabricator via llvm-commits]

samparker marked 2 inline comments as done.
samparker added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/ReachingDefAnalysis.h:222
+  /// nullptr if the id does not refer to the block.
+  MachineInstr *getInstFromId(MachineBasicBlock *MBB, int InstId) const;
+
----------------
SjoerdMeijer wrote:
> This function already exists. Why this change? 
Just making it private.


================
Comment at: llvm/lib/CodeGen/ReachingDefAnalysis.cpp:337
+  MachineInstr *LocalDef = getReachingLocalMIDef(MI, PhysReg);
+  if (InstIds.lookup(LocalDef) < InstIds.lookup(MI))
+    return LocalDef;
----------------
SjoerdMeijer wrote:
> nit: use shorter notation?
> 
>    if (InstIds[LocalDef] < InstIds[MI])
Can't use that notation in a const method. 


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

https://reviews.llvm.org/D74605