Merge r340125 into 7.0.1

[llvmbot]

Bugzilla Link	39905
Resolution	FIXED
Resolved on	Dec 06, 2018 16:24
Version	7.0
OS	All
Blocks	#38454
Reporter	LLVM Bugzilla Contributor
CC	@tstellar
Fixed by commit(s)	r340125 r348555

Extended Description

Hope this is not too late. It took our team a while to isolate the problem.

r340125 fixes allocation for common symbols in loading object code. The bug is doubling the allocation size per symbol and it will request way too much memory with a few dozens of common symbols.

We are seeing it trying to allocating >64GB of memory.

[llvmbot]

assigned to @tstellar

[tstellar]

We are already past the merge request deadline, so I'm not going to delay the release for this. However, if we can get it approved before I tag the next release candidate (likely Friday), then I'll merge it.

[tstellar]

Lang, does this look OK to merge: https://reviews.llvm.org/rL340125

[lhames]

Yep: that should be fine to merge.

[tstellar]

Merged: r348555