[PATCH] D144306: [BOLT][NFC] Add hash computation for basic blocks

Amir Ayupov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 27 14:56:56 PDT 2023


Amir added a comment.

The current state looks good to me. I think @maksfb is suggesting the use of ADT/Hashing helpers instead of rolling up our own hashing primitives but that could be in a follow-up diff IMHO.

> I agree that not relying on specific opcodes is a good idea. Maybe that could be a follow-up diff?

I'll work on that as a follow-up.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144306



More information about the llvm-commits mailing list