[PATCH] D45541: [globalisel] Update GlobalISel emitter to match new representation of extending loads

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 8 09:24:08 PDT 2018


dsanders added a comment.

> I can see some optimizations for G_SEXTLOAD/G_ZEXTLOAD since they're only legal when they're an extend so they could just drop the check in the optimized table.

Actually, we do still need to check the size most of the time (the exception being when there's only one legal size). However, we could still drop the check from a G_LOAD that only handles non-extending loads since there would only be one legal size.


Repository:
  rL LLVM

https://reviews.llvm.org/D45541





More information about the llvm-commits mailing list