[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