<html>
<head>
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
</head>
<body>
<p>This isn't a regression in this change, we're exposing an
existing issue at higher frequency. I reduced a test which
doesn't involve this change, and filed a bug for it.
<a class="moz-txt-link-freetext" href="https://bugs.llvm.org/show_bug.cgi?id=49900">https://bugs.llvm.org/show_bug.cgi?id=49900</a></p>
<p>Florian, I'm willing to take a shot at fixing this, but need a
bit of help. Would you have a couple minutes to debate ideas on
how the vectorizer should be structured to avoid querying SCEV
over partially constructed IR?</p>
<p>Philip<br>
</p>
<div class="moz-cite-prefix">On 4/8/21 8:05 AM, Florian Hahn wrote:<br>
</div>
<blockquote type="cite"
cite="mid:DD193E3F-425E-4840-8FF7-0F9759947DB8@apple.com">
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
Hi Philip,<br class="">
<div><br class="">
<blockquote type="cite" class="">
<div class="">On Dec 28, 2020, at 18:15, Philip Reames via
llvm-commits <<a
href="mailto:llvm-commits@lists.llvm.org" class=""
moz-do-not-send="true">llvm-commits@lists.llvm.org</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div class=""><br class="">
Author: Philip Reames<br class="">
Date: 2020-12-28T10:13:28-08:00<br class="">
New Revision: 4b33b2387787aef5020450cdcc8dde231eb0a5fc<br
class="">
<br class="">
URL: <a
href="https://github.com/llvm/llvm-project/commit/4b33b2387787aef5020450cdcc8dde231eb0a5fc"
class="" moz-do-not-send="true">https://github.com/llvm/llvm-project/commit/4b33b2387787aef5020450cdcc8dde231eb0a5fc</a><br
class="">
DIFF: <a
href="https://github.com/llvm/llvm-project/commit/4b33b2387787aef5020450cdcc8dde231eb0a5fc.diff"
class="" moz-do-not-send="true">https://github.com/llvm/llvm-project/commit/4b33b2387787aef5020450cdcc8dde231eb0a5fc.diff</a><br
class="">
<br class="">
LOG: Reapply "[LV] Vectorize (some) early and multiple
exit loops"" w/fix for builder<br class="">
<br class="">
This reverts commit
4ffcd4fe9ac2ee948948f732baa16663eb63f1c7 thus restoring
e4df6a40dad.<br class="">
<br class="">
The only change from the original patch is to add "llvm::"
before the call to empty(iterator_range). This is a
speculative fix for the ambiguity reported on some
builders.</div>
</div>
</blockquote>
</div>
<div class=""><br class="">
</div>
<div class="">It looks like the following failure started with the
patch: <a href="https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=33029" style="font-family: arial, sans-serif; white-space: pre;" class="" moz-do-not-send="true">https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=33029</a></div>
<div class=""><br class="">
</div>
<div class="">It would be great if you could take a look.</div>
<div class=""><br class="">
</div>
<div class="">Cheers,</div>
<div class="">Florian</div>
</blockquote>
</body>
</html>