[llvm] [EarlyCSE] De-Duplicate callsites with differing attrs (PR #110929)

Yingwei Zheng via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 2 20:03:52 PDT 2024


================
@@ -621,7 +621,7 @@ bool DenseMapInfo<GEPValue>::isEqual(const GEPValue &LHS, const GEPValue &RHS) {
     return false;
   if (LHS.ConstantOffset.has_value() && RHS.ConstantOffset.has_value())
     return LHS.ConstantOffset.value() == RHS.ConstantOffset.value();
-  return LGEP->isIdenticalToWhenDefined(RGEP);
+  return LGEP->isIdenticalToWhenDefined(RGEP, /*IntersectAttrs=*/true);
----------------
dtcxzyw wrote:

It is unnecessary for comparison of GEPs.


https://github.com/llvm/llvm-project/pull/110929


More information about the llvm-commits mailing list