[PATCH] D29213: InferAddressSpaces: Handle icmp

[Justin Lebar via Phabricator via llvm-commits]

jlebar added inline comments.


================
Comment at: lib/Transforms/Scalar/InferAddressSpaces.cpp:747
+
+          if (Value *OtherNewV = ValueWithNewAddrSpace.lookup(OtherSrc)) {
+            Cmp->setOperand(OtherIdx, OtherNewV);
----------------
I don't see where we check that NewV and OtherNewV have the same AS.  Shouldn't it be right here?

Sorry if it's obvious; I'm breaking my rule against reviewing patches late at night, and may be embarrassed in the morning.


https://reviews.llvm.org/D29213