[PATCH] D120121: [StableHashing] Hash vregs with multiple defs

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineStableHash.cpp:69
+      for (auto &Def : MRI.def_instructions(MO.getReg()))
+        DefOpcodes.push_back(Def.getOpcode());
+      return hash_combine_range(DefOpcodes.begin(), DefOpcodes.end());
----------------
If this only uses the opcode, won't it collide if you have multiple defs of the same register by the same opcode?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120121