This is an archive of the discontinued LLVM Phabricator instance.

Add opt-viewer testing

Authored by anemet on Nov 17 2017, 5:22 PM.



Detects whether we have the Python modules (pygments, yaml) required by
opt-viewer and hooks this up to REQUIRES.

This fixes (the lack of opt-viewer

It's also related to and the idea is
to expose LLVM_HAVE_OPT_VIEWER_MODULES to the Swift cmake.

Event Timeline

anemet created this revision.Nov 17 2017, 5:22 PM
beanz added inline comments.Nov 27 2017, 9:44 AM
1668 ↗(On Diff #123453)

This code all looks good, but it is in the wrong place. Can you move this into config-ix.cmake?

That is where we generally execute configuration checks.

anemet updated this revision to Diff 124414.Nov 27 2017, 10:03 AM

Thanks, Chris! This moves the cmake bits to config-ix.cmake.

beanz accepted this revision.Nov 27 2017, 10:37 AM


This revision is now accepted and ready to land.Nov 27 2017, 10:37 AM
This revision was automatically updated to reflect the committed changes.