[PATCH] D60010: [SCEV] Check the cache in get{S|U}MaxExpr before doing any work

[Sanjoy Das via Phabricator via llvm-commits]

sanjoy updated this revision to Diff 192908.
sanjoy added a comment.

Fix bug: when trying to look up the smax/umax SCEV again (after simplification) we can't use the older ID since it was computed based on the unsimplified operands.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60010

Files:
  include/llvm/Analysis/ScalarEvolution.h
  lib/Analysis/ScalarEvolution.cpp
  test/Analysis/ScalarEvolution/max-expr-cache.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60010.192908.patch
Type: text/x-patch
Size: 11264 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190329/bda175fd/attachment.bin>