Use getLastArgValue instead of getString
ClosedPublic

Authored by yamaguchi on Jun 14 2017, 12:40 AM.

Details

Summary

I found that getString defined in the LLD's Driver.cpp is exactly the same as Arg::getLastArgValue defined in llvm/Option/ArgLIst.h.
This patch removes that local function and use the function that the Arg class provides.

Diff Detail

Repository
rL LLVM
yamaguchi created this revision.Jun 14 2017, 12:40 AM
yamaguchi edited the summary of this revision. (Show Details)Jun 14 2017, 12:45 AM
ruiu accepted this revision.Jun 14 2017, 12:46 AM

LGTM

Thank you for doing this! I didn't know that Arg class provides that function, that's why I implemented our own getString() function.

This revision is now accepted and ready to land.Jun 14 2017, 12:46 AM
This revision was automatically updated to reflect the committed changes.