[llvm-bugs] [Bug 36082] Merge r322016 into the 6.0 branch

[via llvm-bugs]

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>