[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