LLVM Bugzilla is read-only and represents the historical archive of all LLVM issues filled before November 26, 2021. Use github to submit LLVM bugs

Bug 31622 - [meta] 4.0.0 Release Blockers
Summary: [meta] 4.0.0 Release Blockers
Status: RESOLVED FIXED
Alias: None
Product: new-bugs
Classification: Unclassified
Component: new bugs (show other bugs)
Version: 4.0
Hardware: All All
: P normal
Assignee: Hans Wennborg
URL:
Keywords:
Depends on: 27685 30256 30284 30775 31186 31187 31199 31257 31325 31381 31606 31626 31628 31638 31644 31672 31673 31685 31688 31689 31692 31710 31711 31723 31727 31729 31733 31740 31744 31745 31751 31761 31763 31769 31773 31780 31782 31804 31808 31823 31825 31832 31833 31843 31845 31881 31889 31961 31963 31965 31983 31998 32013
Blocks:
  Show dependency tree
 
Reported: 2017-01-12 16:58 PST by Hans Wennborg
Modified: 2017-03-13 13:48 PDT (History)
16 users (show)

See Also:
Fixed By Commit(s):


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Hans Wennborg 2017-01-12 16:58:16 PST
This is the tracking bug for blockers of the 4.0.0 release.
Comment 1 Dimitry Andric 2017-01-13 15:48:33 PST
Please merge the fix for bug 31606, e.g. r291955 ("Generalize our tentative DR resolution for inheriting copy/move constructors to better match the pre-P0136R1 behavior")
Comment 2 Hans Wennborg 2017-01-13 16:09:46 PST
(In reply to comment #1)
> Please merge the fix for bug 31606, e.g. r291955 ("Generalize our tentative
> DR resolution for inheriting copy/move constructors to better match the
> pre-P0136R1 behavior")

It's on my list, Richard just wanted it to bake on trunk for a bit before we merge. Thanks.
Comment 3 Jörg Sonnenberger 2017-01-16 08:13:46 PST
May I merge r292119 please? We hit this recently with -Wsystem-header builds, it's a left-over bug from the five-digit-revision age.
Comment 4 Jörg Sonnenberger 2017-01-17 13:37:24 PST
r292244 is a fix for a recent regression reported on NetBSD, i.e. -march=native -mno-sse41 would assert for no good reason. May I merge it?
Comment 5 Hans Wennborg 2017-01-17 16:01:33 PST
(In reply to comment #3)
> May I merge r292119 please? We hit this recently with -Wsystem-header
> builds, it's a left-over bug from the five-digit-revision age.

(In reply to comment #4)
> r292244 is a fix for a recent regression reported on NetBSD, i.e.
> -march=native -mno-sse41 would assert for no good reason. May I merge it?

OK to both.
Comment 6 Dylan McKay 2017-01-18 21:41:25 PST
I've added bug 31685 so that we can merge the fix for bug 30543
Comment 7 Bernhard Rosenkraenzer 2017-01-25 13:50:17 PST
Please merge the fix for bug 27685 - already fixed on master, but not on the branch
Comment 8 Simon Dardis 2017-01-25 13:50:38 PST
I'm not sure if we want to consider bug 31756 a release blocker, as it requires xray to identify setups that inhibit the use of rdtscp. I've asked Dean to comment here about it.
Comment 9 Bernhard Rosenkraenzer 2017-01-25 15:12:26 PST
Bug 31547 is relatively serious for those of us who have to work with libgcc for compatibility reasons. It breaks e.g. building GNU sed 4.3 with clang on a Linux system using libgcc.

IMO this should be fixed for the release.
Comment 10 Michał Górny 2017-01-25 16:43:28 PST
I'd like to request merging two revisions:

- [libunwind] r292723 from bug #30879 -- this fixes exception handling on x86 Linux for me;

- [lld] r293078 from bug #31745 -- this fixes stand-alone build with shared libraries.
Comment 11 Dean Michael Berris 2017-01-27 01:10:52 PST
I think XRay bug 31756 should *not* be considered a release blocker. There's quite a bit of work to do to probe and work-around the lack of rdtscp, and I wouldn't want to be blocking the release contingent on that being addressed.
Comment 12 Hans Wennborg 2017-01-27 10:31:56 PST
(In reply to comment #10)
> I'd like to request merging two revisions:
> 
> - [libunwind] r292723 from bug #30879 -- this fixes exception handling on
> x86 Linux for me;

Merged as r293298

> - [lld] r293078 from bug #31745 -- this fixes stand-alone build with shared
> libraries.

r293289
Comment 13 Bernhard Rosenkraenzer 2017-01-27 16:31:51 PST
Will llgo be released with 4.0 (It does have a release_40 branch and a 4.0.0rc1 tag, but isn't listed anywhere...)?

If so, should add bug 31786 to the blockers -- the fixes attached there are needed to make llgo compile on Linux.
Comment 14 Hans Wennborg 2017-01-27 16:54:48 PST
(In reply to comment #13)
> Will llgo be released with 4.0 (It does have a release_40 branch and a
> 4.0.0rc1 tag, but isn't listed anywhere...)?
> 
> If so, should add bug 31786 to the blockers -- the fixes attached there are
> needed to make llgo compile on Linux.

I didn't tag or branch it, and I don't see any branch here: http://llvm.org/svn/llvm-project/llgo/branches/

I don't think llgo is part of the release process.
Comment 15 Valentin Churavy 2017-01-28 22:29:10 PST
I would like to nominate rL293230 for inclusion into LLVM 4.0 it is a small fix that allows LLVM to build with the Intel 2017 toolchain.
Comment 16 Simon Dardis 2017-01-30 03:59:30 PST
I'd like to request that r292624 - "[mips] Fix debug information for __thread variable"
and r292117 - "[mips] Correct c.cond.fmt instruction definition" be merged for 4.0.
Comment 17 Hans Wennborg 2017-01-31 12:24:03 PST
(In reply to comment #15)
> I would like to nominate rL293230 for inclusion into LLVM 4.0 it is a small
> fix that allows LLVM to build with the Intel 2017 toolchain.

I've replied on the commit email.

(In reply to comment #16)
> I'd like to request that r292624 - "[mips] Fix debug information for
> __thread variable"

r293664

> and r292117 - "[mips] Correct c.cond.fmt instruction definition" be merged
> for 4.0.

r293665.
Comment 18 Ed Schouten 2017-03-05 07:12:12 PST
One of our users discovered a problem with SafeStack. I suspect it may be too late to get it fixed for 4.0, but added it to this tracking bug to ensure it won't get lost.
Comment 19 Hans Wennborg 2017-03-06 11:48:02 PST
(In reply to Ed Schouten from comment #18)
> One of our users discovered a problem with SafeStack. I suspect it may be
> too late to get it fixed for 4.0, but added it to this tracking bug to
> ensure it won't get lost.

It sounds like the revision that regressed that was in 3.9 too, so it's not a regression and too late for 4.0. I'll change it to be a 4.0.1 blocker instead.
Comment 20 Hans Wennborg 2017-03-13 13:48:42 PDT
The release has shipped, so closing this.

For 4.0.1 blockers, use PR32061.