This is an archive of the discontinued LLVM Phabricator instance.

clang-cl: Add support for the /o option for object files, executables, and preprocessor output
ClosedPublic

Authored by ehsan on Sep 10 2014, 6:51 PM.

Details

Summary

cl.exe recognizes /o as a deprecated and undocumented option similar to
/Fe. This patch adds support for this option to clang-cl for /Fe, /Fo
and /Fi. It also ensures that the last option among /o and /F* wins,
if both specified.

This is required at least for building autoconf based software, since
autoconf uses -o to specify the executable output.

This fixes http://llvm.org/PR20894.

Diff Detail

Event Timeline

ehsan updated this revision to Diff 13573.Sep 10 2014, 6:51 PM
ehsan retitled this revision from to clang-cl: Add support for the /o option for object files, executables, and preprocessor output.
ehsan updated this object.
ehsan edited the test plan for this revision. (Show Details)
ehsan added a reviewer: rnk.
ehsan added a subscriber: Unknown Object (MLST).
hans accepted this revision.Sep 11 2014, 10:27 AM
hans added a reviewer: hans.
hans added a subscriber: hans.

This looks good to me.

This revision is now accepted and ready to land.Sep 11 2014, 10:27 AM
ehsan closed this revision.Sep 11 2014, 11:26 AM
ehsan updated this revision to Diff 13592.

Closed by commit rL217615 (authored by @ehsan).