[PATCH] D31238: [ScalarEvolution] Re-enable Predicate implication from operations

[Maxim Kazantsev via llvm-commits]

Hello Daniel,

Thanks for your letter. The failure is caused by my patch, I have already reverted it and working on a fix.

Regards,
Max

From: Daniel Berlin [mailto:dberlin at dberlin.org]
Sent: Friday, March 24, 2017 5:05 PM
To: reviews+D31238+public+3404398fbfb9b9e2 at reviews.llvm.org; Max Kazantsev via Phabricator <reviews at reviews.llvm.org>
Cc: Maxim Kazantsev <max.kazantsev at azul.com>; Sanjoy Das <sanjoy at playingwithpointers.com>; Philip Reames <listmail at philipreames.com>; Artur Pilipenko <apilipenko at azul.com>; Anna Thomas <anna at azul.com>; Junbum Lim <junbuml at codeaurora.org>; llvm-commits <llvm-commits at lists.llvm.org>
Subject: Re: [PATCH] D31238: [ScalarEvolution] Re-enable Predicate implication from operations

Note: At least two buildbots i saw were also complaining about significant compile time regressions (IE tests not compiling in some mandated timeout)

Not sure if it is just an artifact of something, but probably worth going through the failures to see what happened.


On Fri, Mar 24, 2017 at 2:27 AM, Max Kazantsev via Phabricator via llvm-commits <llvm-commits at lists.llvm.org<mailto:llvm-commits at lists.llvm.org>> wrote:
mkazantsev updated this revision to Diff 92906.

https://reviews.llvm.org/D31238

Files:
  include/llvm/Analysis/ScalarEvolution.h
  lib/Analysis/ScalarEvolution.cpp
  test/Analysis/ScalarEvolution/implied-via-addition.ll
  test/Analysis/ScalarEvolution/implied-via-division.ll


_______________________________________________
llvm-commits mailing list
llvm-commits at lists.llvm.org<mailto:llvm-commits at lists.llvm.org>
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170324/44650558/attachment.html>