This is an archive of the discontinued LLVM Phabricator instance.

[MC] Move MCAssembler::dump into the correct cpp file. NFC
ClosedPublic

Authored by sbc100 on May 7 2018, 3:53 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

sbc100 created this revision.May 7 2018, 3:53 PM
sbc100 retitled this revision from [MC] Move MCFragment into the correct cpp file to [MC] Move MCAssembler into the correct cpp file.May 7 2018, 3:53 PM
sbc100 retitled this revision from [MC] Move MCAssembler into the correct cpp file to [MC] Move MCAssembler into the correct cpp file. NFC.May 7 2018, 3:57 PM
sbc100 added a reviewer: dschuff.
sbc100 retitled this revision from [MC] Move MCAssembler into the correct cpp file. NFC to [MC] Move MCAssembler::dump into the correct cpp file. NFC.

Have you tested this on a system that has ordered linking semantics (Linux/unix-y things, not Mac or Windows)? It might be in the 'wrong' place to workaround some dependency cycle during linking or the like.

Yes. I work on linux.

dblaikie accepted this revision.May 15 2018, 2:51 PM

Fair enough, well, *fingers crossed* :)

This revision is now accepted and ready to land.May 15 2018, 2:51 PM
dschuff accepted this revision.May 15 2018, 2:59 PM

Also if you use the LINK_LLVM_DYLIB CMake mode (as our bots do) you should test the regular non-dylib static link build.

This revision was automatically updated to reflect the committed changes.