[PATCH] D54585: [TTI] Reduction costs only need to include a single extract element cost
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 28 06:44:37 PST 2018
spatel added a comment.
In D54585#1310210 <https://reviews.llvm.org/D54585#1310210>, @spatel wrote:
> D54955 <https://reviews.llvm.org/D54955> proposes a fix for the root cause of the bug. I think the test case there shows that we can still crash even with this patch reverted.
Committed here:
rL347759 <https://reviews.llvm.org/rL347759>
After that has baked for a while, we should be able to re-apply this patch, confirm that:
https://bugs.llvm.org/show_bug.cgi?id=39774
is no longer a problem, and re-close:
https://bugs.llvm.org/show_bug.cgi?id=37731
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D54585/new/
https://reviews.llvm.org/D54585
More information about the llvm-commits
mailing list