[PATCH] D144306: [BOLT] adding hash computation for basic blocks

[Amir Ayupov via Phabricator via llvm-commits]

Amir added inline comments.


================
Comment at: bolt/lib/Core/HashUtilities.cpp:134-138
+    while (Opcode) {
+      uint8_t LSB = Opcode & 0xff;
+      HashString.push_back(LSB);
+      Opcode = Opcode >> 8;
+    }
----------------
Amir wrote:
> As a note: I want to switch to using instruction mnemonic instead of opcode value. It shouldn'
It shouldn't affect the matching, but will make hash stable wrt opcode value changes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144306