This is an archive of the discontinued LLVM Phabricator instance.

Fix the LLVM_ENABLE_MODULES build after adding TargetOpcodes.def in r259726
ClosedPublic

Authored by DavidKreitzer on Feb 8 2016, 3:12 PM.

Details

Summary

Thanks, Richard, for the advice on how to fix to LLVM_ENABLE_MODULES build. I trust this is what you were suggesting?

I have been unable to test this, because I have not yet been able to get the stage 2 build working in my environment. (That is, the build fails before it hits the error reported by the buildbot.) The change is simple enough, though, that my plan is to just watch the buildbot and verify that it starts working again.

Diff Detail

Event Timeline

DavidKreitzer retitled this revision from to Fix the LLVM_ENABLE_MODULES build after adding TargetOpcodes.def in r259726.
DavidKreitzer updated this object.
DavidKreitzer added a reviewer: rsmith.
DavidKreitzer added a subscriber: llvm-commits.
rsmith accepted this revision.Feb 8 2016, 3:54 PM
rsmith edited edge metadata.

Yes, thanks.

This revision is now accepted and ready to land.Feb 8 2016, 3:54 PM
This revision was automatically updated to reflect the committed changes.