Page MenuHomePhabricator

[Frontend] Remove unused FileMgr in pp arg parse
ClosedPublic

Authored by modocache on Jan 9 2018, 8:09 AM.

Details

Summary

A FIXME added 8 years ago (2010) in https://reviews.llvm.org/rL118203
mentioned that a FileManager should not need to be used when parsing
preprocessor arguments. In fact, its only use was removed 6 years ago
(2012), in https://reviews.llvm.org/rL166452. Remove the unused
variable and the obsolete FIXME.

Test Plan: check-clang

Diff Detail

Repository
rC Clang

Event Timeline

modocache created this revision.Jan 9 2018, 8:09 AM
This revision is now accepted and ready to land.Jan 9 2018, 12:21 PM
This revision was automatically updated to reflect the committed changes.

Great, thanks for the review, @v.g.vassilev!