Seeing double

Is there a reason that BlockExtractorPass is duplicated? See tools/Bugpoint/ExtractFunction.cpp and lib/Transforms/IPO/LoopExtractor.cpp.

— Gordon

Huh, how about that. It looks like bugpoint should use the other one and drop its copy.

-Chris

Is there a reason that BlockExtractorPass is duplicated? See tools/Bugpoint/ExtractFunction.cpp and lib/Transforms/IPO/LoopExtractor.cpp.

Huh, how about that.

This existed to communicate the "blocks to not extract" list via a static variable to the forked bugpoint subprocess. Since bugpoint subprocesses now exec, this form of IPC is no longer possible. I opened http://llvm.org/PR1775 for the matter.

It looks like bugpoint should use the other one and drop its copy.

Done.

— Gordon