[PATCH] D110587: [SCEV] Extend trip count to avoid overflow by default

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 24 05:59:27 PST 2022


SjoerdMeijer added a comment.

> I did leave the legacy behavior as an option since a) loop-flatten stops triggering if I extend due to weirdly specific pattern matching I didn't understand and b) we could reasonably use the mode if we'd externally established a lock of overflow.

I wasn't aware of this until I noticed the FIXME and have committed NFC patches rGf6ac8088b0e8 <https://reviews.llvm.org/rGf6ac8088b0e890765974fee5f5820a340736f9bf> and rGada6d78a7802 <https://reviews.llvm.org/rGada6d78a7802f8057f1ab7cee0bed25f91fcc4b4> to try and clarify this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110587



More information about the llvm-commits mailing list