[PATCH] D36656: [SCCP] Propagate integer range information in IPSCCP (WIP).

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 14 12:53:27 PDT 2017


efriedma added a comment.

There have been improvements, but we still have a lot of transforms that aren't assume-aware; essentially every call to hasOneUse() does the wrong thing, etc.

And this transform in particular is creating up to four new IR instructions per existing instruction in the IR, which is going to be a general drain on performance for anything examining the IR; we need some sort of rule for figuring out what's worth annotating.


https://reviews.llvm.org/D36656





More information about the llvm-commits mailing list