[PATCH] D65605: Use MCRegister in MCRegisterInfo's interfaces

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 1 17:03:36 PDT 2019


arsenm added inline comments.


================
Comment at: llvm/include/llvm/MC/MCRegister.h:72
+  static inline unsigned getTombstoneKey() { return ~0U - 1; }
+  static unsigned getHashValue(const unsigned& Val) { return Val * 37U; }
+
----------------
dsanders wrote:
> arsenm wrote:
> > Why *37?
> No idea. It's copied directly from the DenseMapInfo<unsigned> implementation
Is it possible to implement this by calling the unsigned version?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65605





More information about the llvm-commits mailing list