This is an archive of the discontinued LLVM Phabricator instance.

[lnttool] Refactor the Makefile logic out behind an interface
AbandonedPublic

Authored by jmolloy on Nov 10 2015, 7:51 AM.

Details

Reviewers
beanz
cmatthews
Summary

This should set the scene for logic to be plugged in to drive the new CMake based build system. It encapsulates the Makefile-specific behaviors and code and hides it all away.

At some point we're going to have to do a spring clean of the supported LNT options; putting in the effort to make some work until CMake and binning others. Unfortunately I have no idea what LNT features people are using though :(

The interface is deliberately not documented because it's not expected to be stable, and it does whatever the Makefile code expects at the moment.

Diff Detail

Repository
rL LLVM

Event Timeline

jmolloy updated this revision to Diff 39812.Nov 10 2015, 7:51 AM
jmolloy retitled this revision from to [lnttool] Refactor the Makefile logic out behind an interface.
jmolloy updated this object.
jmolloy added reviewers: cmatthews, beanz.
jmolloy set the repository for this revision to rL LLVM.
jmolloy added a subscriber: llvm-commits.
cmatthews edited edge metadata.Nov 10 2015, 9:54 AM

Question: would it be simpler to just to make a new runtest module? nt.py IS just really a makefile wrapper. No point in supporting the same options if that adds complexity.

jmolloy abandoned this revision.Nov 11 2015, 3:03 AM