This is an archive of the discontinued LLVM Phabricator instance.

Driver: Accept multiple --config options if filenames are the same
ClosedPublic

Authored by tstellar on Jun 8 2020, 1:15 PM.

Details

Summary

We're trying to use the --config options to pass distro specific
options for Fedora via the CFLAGS variable. However, some projects
end up using the CFLAGS variable multiple times in their command line,
which leads to an error when --config is used.

This patch resolves this issue by allowing more than one --config option
on the command line as long as the file names are the same.

Diff Detail

Event Timeline

tstellar created this revision.Jun 8 2020, 1:15 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 8 2020, 1:15 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
sepavloff accepted this revision.Jun 9 2020, 4:19 AM

LGTM.

This revision is now accepted and ready to land.Jun 9 2020, 4:19 AM
This revision was automatically updated to reflect the committed changes.