[all-commits] [llvm/llvm-project] e6f332: [IndVarSimplify] Fix Modified status for removal o...

David via All-commits all-commits at lists.llvm.org
Tue Sep 29 04:22:02 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e6f332ef1e414ec41a188217d7547a371ed975de
      https://github.com/llvm/llvm-project/commit/e6f332ef1e414ec41a188217d7547a371ed975de
  Author: David Stenberg <david.stenberg at ericsson.com>
  Date:   2020-09-29 (Tue, 29 Sep 2020)

  Changed paths:
    M llvm/lib/Transforms/Utils/SimplifyIndVar.cpp
    A llvm/test/Transforms/IndVarSimplify/eliminate-overflow-modified.ll

  Log Message:
  -----------
  [IndVarSimplify] Fix Modified status for removal of overflow intrinsics

When removing an overflow intrinsic the Changed status in SimplifyIndvar
was not set, leading to the IndVarSimplify pass returning an incorrect
status.

This was caught using the check introduced by D80916.

As pointed out in the code review, a similar bug may exist for
eliminateTrunc().

Reviewed By: reames

Differential Revision: https://reviews.llvm.org/D85971




More information about the All-commits mailing list