[PATCH] D98706: [SCEV] Add false->any implication to save compile time

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 16 08:04:03 PDT 2021


lebedev.ri added a comment.

In D98706#2628952 <https://reviews.llvm.org/D98706#2628952>, @mkazantsev wrote:

> The cost is spread over all SCEV I guess. If we want to improve it, we should make it sitch by sitch. Apparently this sitution is rare. :( Well, it was worth trying.

While not a compile time win, this seems like an improvement on it's own to me.
The change itself is rather cheap (a plain single comparison).

That being said, i think i'm failing to understand why it implies *anything*?


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

https://reviews.llvm.org/D98706



More information about the llvm-commits mailing list