[PATCH] D158250: [IR] Add more details to StructuralHash

Aiden Grossman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 23 11:07:10 PDT 2023


aidengrossman added inline comments.


================
Comment at: llvm/lib/IR/StructuralHash.cpp:51
+    else if (ValueType->isDoubleTy())
+      hash(1002);
+  }
----------------
nikic wrote:
> Aren't these cases covered by the type ID hash?
Yes. For some reason I was thinking that there was.a generic floating point type in the enum so did them separately. Thanks for the catch!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158250



More information about the llvm-commits mailing list