[NFC][Debugify] Rename OptCustomPassManager into DebugifyCustomPassManager

[djtodoro]

Summary

In addition, move the definition of the class into the Debugify.h, so we can use it from different levels.

The motivation for this is D82547.

[vsk]

llvm/include/llvm/IR/LegacyPassManager.h

100 ↗

(On Diff #276395)

The layering for this seems a bit strange, as LegacyPassManager doesn't have much to do with 'debugify'. How does moving the class into Debugify.h, and the class implementation into Debugify.cpp sound? That way, the new-PM version of this can live in the same files.

100 ↗

(On Diff #276395)

To be clear: I'm not asking that you implement a new-PM version of this, it's just some future-proofing.

[djtodoro]

llvm/include/llvm/IR/LegacyPassManager.h

100 ↗

(On Diff #276395)

Nice point, thanks!

I'd rather keep the class def in the Debugify.h in order to avoid linking additional stuff into the libLLVMTransformUtils (for example, isBitcodeWriterPass() won't be available while linking without CMake changes). WDYT?

[djtodoro]

• Move the def of DebugifyCustomPassManager into the Debugify.h

[vsk]

Lgtm.

[djtodoro]

@vsk Thanks!