[PATCH] D110752: [SimpleLoopUnswitch] Don't unswitch constant conditions

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 30 11:55:02 PDT 2021


lebedev.ri added a comment.

Is this falling into the traditional trap of the fact that constants and constant expressions are both represented by the `Constant` class?
Presumably while we obviously shouldn't unswitch immediate constants, constant expressions are a fair game still?


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

https://reviews.llvm.org/D110752



More information about the llvm-commits mailing list