This is an archive of the discontinued LLVM Phabricator instance.

Remove Support/Options.h, it is unused
ClosedPublic

Authored by rnk on Nov 14 2019, 3:21 PM.

Details

Summary

It was added in 2014 in 732e0aa9fb84f1 with one use in Scalarizer.cpp.
That one use was then removed when porting to the new pass manager in
2018 in b6f76002d9158628e78.

While the RFC and the desire to get off of static initializers for
cl::opt all still stand, this code is now dead, and I think we should
delete this code until someone is ready to do the migration.

This change depends on a large mechanical cleanup of including
CommandLine.h, StringMap.h, and raw_ostream.h which I'll push shortly.

Event Timeline

rnk created this revision.Nov 14 2019, 3:21 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald Transcript
beanz accepted this revision.Nov 14 2019, 9:30 PM

Yea. Unfortunately the migration stalled out because there was no way to make the registration method work with the new pass manager.

This revision is now accepted and ready to land.Nov 14 2019, 9:30 PM
This revision was automatically updated to reflect the committed changes.
llvm/lib/Support/CMakeLists.txt