[PATCH] D89479: [SimplifyCFG] Be more conservative when speculating in loops. (WIP)

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 15 09:23:59 PDT 2020


fhahn added a comment.

In D89479#2332577 <https://reviews.llvm.org/D89479#2332577>, @spatel wrote:

> Do you know if there is any overlap in this and what I am proposing with D89461 <https://reviews.llvm.org/D89461> (and also if we remove or adjust the isCheapToSpeculateCttz() calls that are still there)?

Hm, yes I think we could also solve this by just making CTLZ not cheap to speculate on X86 I guess? I think the current implementation just checks for `LZCNT`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89479



More information about the llvm-commits mailing list