[llvm-bugs] [Bug 36082] Merge r322016 into the 6.0 branch
via llvm-bugs
llvm-bugs at lists.llvm.org
Tue Jan 30 01:53:26 PST 2018
https://bugs.llvm.org/show_bug.cgi?id=36082
Hans Wennborg <hans at chromium.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|NEW |RESOLVED
--- Comment #3 from Hans Wennborg <hans at chromium.org> ---
(In reply to Sanjay Patel from comment #2)
> Yes, r322016 should be merged as that's a clear fix for the bug.
Merged in r323738.
> No, r322006 should not be merged.
>
> Based on the post-commit comments in:
> https://reviews.llvm.org/D41812
>
> r322006 needs changes for correctness, so it's not suitable for the branch
> yet. It may get or already be reverted from trunk based on a mailing list
> comment.
Sounds like it was pretty much backed out in r323630.
> If you merge:
> https://reviews.llvm.org/rL323331
> then you should not hit bug 35807 which is what r322006 was also solving.
>
> Note that merging r323331 is tracked by bug 36045.
That was merged.
--
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20180130/bc3a3ab0/attachment.html>
More information about the llvm-bugs
mailing list