[PATCH] D103877: [SCEV] Keep common NUW flags when inlining Add operands.

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 8 21:56:13 PDT 2021


mkazantsev added a comment.

> Good point, I was not aware that the flags need to apply for any order of evaluation

Technically, SCEV's commutative operations don't have order of operand, we only sort them to make simplification easier. So I guess it's a reasonable thing to assume. Expander, for example, may use any order to expand it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103877



More information about the llvm-commits mailing list