[PATCH] D62423: [globalisel][legalizer] Attempt to write down the minimal legalization rules

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 24 13:47:21 PDT 2019


arsenm added inline comments.


================
Comment at: llvm/docs/GlobalISel.rst:464-465
+
+There are no minimum rules for pointers since G_INTTOPTR and G_PTRTOINT can
+be lowered to COPY (or to nothing using replaceAllUses()) by the legalizer.
+
----------------
arsenm wrote:
> This ignores the existence of non-integral pointers
This should also probably say "selected" to copy, since IIRC COPY is not allowed to convert the type with generic virtual registers


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62423





More information about the llvm-commits mailing list