[all-commits] [llvm/llvm-project] d428f0: [InstCombine] freeze operand in urem expansion

Sanjay Patel via All-commits all-commits at lists.llvm.org
Wed May 11 09:48:14 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d428f09b2c9d49f6a328ef636c02305a8be4ab61
      https://github.com/llvm/llvm-project/commit/d428f09b2c9d49f6a328ef636c02305a8be4ab61
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2022-05-11 (Wed, 11 May 2022)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp
    M llvm/test/Transforms/InstCombine/rem.ll
    M llvm/test/Transforms/InstCombine/vector-urem.ll

  Log Message:
  -----------
  [InstCombine] freeze operand in urem expansion

As discussed in issue #37809, this transform is not safe
if the input is an undefined value.

There is no difference in codegen on the basic examples,
but this could lead to regressions. We may need to
improve freeze analysis or lowering if that happens.




More information about the All-commits mailing list