[PATCH] D90361: Prevent LICM and machineLICM from hoisting convergent operations

Xiaoqing Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 29 10:38:50 PDT 2020


xiaoqing_wu added a comment.

In D90361#2362068 <https://reviews.llvm.org/D90361#2362068>, @arsenm wrote:

> This is allowed by the current definition. See D85603 <https://reviews.llvm.org/D85603>

D85603 <https://reviews.llvm.org/D85603> requires changes to IR to model the dependency of convergent operations on their enclosing control flows. 
The change in this review solves a different problem, it is to prevent hoisting with the existing IR.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90361



More information about the llvm-commits mailing list