[PATCH] D87310: [IRSim] Adding support for isomorphic predicates

Jon Roelofs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 10 14:31:32 PDT 2020


jroelofs added inline comments.


================
Comment at: llvm/lib/Analysis/IRSimilarityIdentifier.cpp:62
+  case CmpInst::ICMP_SGE:
+  case CmpInst::ICMP_UGE:
+    return CI->getSwappedPredicate();
----------------
Should equality and inequality also be canonicalized? Maybe your hash function can be tweaked to use an operation that's commutative specifically in that case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87310



More information about the llvm-commits mailing list