[PATCH] D149422: [ValueTracking] Add tests for checking whether `div`/`rem` is safe to speculate; NFC

Noah Goldstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 29 11:23:42 PDT 2023


goldstein.w.n added a comment.

In D149422#4307697 <https://reviews.llvm.org/D149422#4307697>, @nikic wrote:

> What do you think about this test structure? https://llvm.godbolt.org/z/nMxMGoz1r Seems clearer to me than going through speculative hoisting.

Sure, where would that test belong? simplifycfg still?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149422



More information about the llvm-commits mailing list