[PATCH] D42004: [Driver] Suggest valid integrated tools

2018-01-15 Thread Brian Gesiak via Phabricator via cfe-commits
modocache added a comment. Great, thanks for the review @bkramer! Repository: rC Clang https://reviews.llvm.org/D42004 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D42004: [Driver] Suggest valid integrated tools

2018-01-15 Thread Brian Gesiak via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC322517: [Driver] Suggest valid integrated tools (authored by modocache, committed by ). Changed prior to commit: https://reviews.llvm.org/D42004?vs=129668=129902#toc Repository: rC Clang

[PATCH] D42004: [Driver] Suggest valid integrated tools

2018-01-15 Thread Benjamin Kramer via Phabricator via cfe-commits
bkramer accepted this revision. bkramer added a comment. This revision is now accepted and ready to land. The -cc1 stuff isn't exactly user-facing, so we didn't put so much emphasis on providing great error messages there. This change LGTM. Repository: rC Clang

[PATCH] D42004: [Driver] Suggest valid integrated tools

2018-01-12 Thread Brian Gesiak via Phabricator via cfe-commits
modocache created this revision. modocache added reviewers: sepavloff, bkramer, phosek. There are only two valid integrated Clang driver tools: `-cc1` and `-cc1as`. If a user asks for an unknown tool, such as `-cc1asphalt`, an error message is displayed to indicate that there is no such tool, but