[PATCH] D86364: [ValueTracking] Interpret GEPs as a series of adds multiplied by the related scaling factor

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 21 12:48:33 PDT 2020


qcolombet added a comment.

> Can your add also IR testcase from that PR? (To show that Instcombine can fold it)

This commit is actually not enough to fix that PR, instcombine will still not be able to simplify the IR because it doesn't understand that `icmp ugt float*, inttoptr (i64 523 to float*)` is a `icmp` with a compile time constant.
Let me see if I can massage the IR to work around this issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86364



More information about the llvm-commits mailing list