ExtractMappedBlocksFromModule discards the "blocks to not extract" list

[llvmbot]

Bugzilla Link	1775
Resolution	FIXED
Resolved on	Nov 14, 2007 00:49
Version	trunk
OS	All
Reporter	LLVM Bugzilla Contributor
CC	@nlewycky

Extended Description

In tools/bugpoint/ExtractFunction.cpp:ExtractMappedBlocksFromModule, bugpoint is discarding the "blocks to not extract" list since it's doing a fork+exec to run the block extractor pass. bugpoint used to just fork, so it used to work.

[llvmbot]

assigned to @nlewycky

[nlewycky]

adds a command-line option to block extractor and makes bugpoint use it
This proposed patch should improve the situation.

This modifies bugpoint to write out the (Function, BasicBlock) names to a file, then passing the name of that file over to the extract basic blocks pass. It isn't perfect, as function and BB names might have spaces, newlines, etc. in them. It'd be cool if that could be fixed by leveraging the .ll parser/printer bits.

[lattner]

Patch looks great to me, please apply!

[nlewycky]

Applied. The bug for the possible things it won't handle is llvm/llvm-bugzilla-archive#1797 .

[nlewycky]

mentioned in issue llvm/llvm-bugzilla-archive#1797