[PATCH] D36656: [SCCP] Propagate integer range information in IPSCCP (WIP).
Matthew Simpson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 14 13:34:08 PDT 2017
mssimpso added a comment.
In https://reviews.llvm.org/D36656#841247, @efriedma wrote:
> Should we use range/nonnull metadata or assumptions to generate ranges?
>
> Should IPSCCP itself try to fold operations based on ranges?
>
> Inserting a bunch of "llvm.assume" calls is probably not a good idea; it's likely to be a big performance sink, and block other optimizations. (Not sure what the right answer looks like here, though; it's a hard problem.)
An alternative would be to just attach range metadata to arguments/calls, and then rely on LVI for the intraprocedural work.
https://reviews.llvm.org/D36656
More information about the llvm-commits
mailing list