[PATCH] D68844: [SCEV] Compute exit count for simple floating point IVs

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 14 12:37:42 PDT 2019


reames added a comment.

In D68844#1707027 <https://reviews.llvm.org/D68844#1707027>, @nikic wrote:

> I'm wondering if we can't extend Float2Int to convert these to operations on integers. I'm assuming it currently doesn't due to a `FIXME: Handle select and phi nodes`.


I'd really rather not continue extending a dedicated transform pass specifically for floats when we have a generic framework it fits naturally within which drives a number of existing optimizations.  That feels like a much cleaner factoring.

Or to say it differently, we have a generic framework for reasoning about phis which happen to be induction variables.  That's SCEV.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D68844





More information about the llvm-commits mailing list