[PATCH] D62101: NewGVN: Handle addrspacecast

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 3 08:52:03 PDT 2019


fhahn added a comment.

LGTM. Could you also add a test where we need to pass in the leader of the operand? Something like having 2 loads from the same address would probably be enough.


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

https://reviews.llvm.org/D62101





More information about the llvm-commits mailing list