[PATCH] D50167: RFC: [SCEV] Add explicit representations of umin/smin

Keno Fischer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 5 18:20:52 PDT 2018


loladiro updated this revision to Diff 159239.
loladiro added a comment.

Fix a small bug discovered during production testing (
when expanding umin/umax, don't go to integers just because
types are unequal - the only reason to do is if one of the
operands in an integer and the other is not).


Repository:
  rL LLVM

https://reviews.llvm.org/D50167

Files:
  include/llvm/Analysis/ScalarEvolution.h
  include/llvm/Analysis/ScalarEvolutionExpander.h
  include/llvm/Analysis/ScalarEvolutionExpressions.h
  lib/Analysis/ScalarEvolution.cpp
  lib/Analysis/ScalarEvolutionExpander.cpp
  test/Analysis/LoopAccessAnalysis/memcheck-ni.ll
  test/Analysis/LoopAccessAnalysis/reverse-memcheck-bounds.ll
  test/Analysis/ScalarEvolution/2008-07-29-SMinExpr.ll
  test/Analysis/ScalarEvolution/min-max-exprs.ll
  test/Analysis/ScalarEvolution/pr28705.ll
  test/Analysis/ScalarEvolution/predicated-trip-count.ll
  test/Analysis/ScalarEvolution/trip-count14.ll
  test/Analysis/ScalarEvolution/trip-count3.ll
  test/Transforms/IRCE/conjunctive-checks.ll
  test/Transforms/IRCE/decrementing-loop.ll
  test/Transforms/IRCE/multiple-access-no-preloop.ll
  test/Transforms/IRCE/ranges_of_different_types.ll
  test/Transforms/IRCE/rc-negative-bound.ll
  test/Transforms/IRCE/single-access-no-preloop.ll
  test/Transforms/IRCE/single-access-with-preloop.ll
  test/Transforms/IndVarSimplify/eliminate-trunc.ll
  test/Transforms/LoopStrengthReduce/2013-01-14-ReuseCast.ll
  test/Transforms/LoopVectorize/X86/pr35432.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50167.159239.patch
Type: text/x-patch
Size: 78693 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180806/fdb38a7e/attachment.bin>


More information about the llvm-commits mailing list