diff options
| author | Hans Wennborg <hans@hanshq.net> | 2013-07-22 16:18:13 +0000 |
|---|---|---|
| committer | Hans Wennborg <hans@hanshq.net> | 2013-07-22 16:18:13 +0000 |
| commit | 6c7e78746e059916892f1df279b681d962a70954 (patch) | |
| tree | a75b151dd9ef8e34a4b176d442e7f9b378a38294 /lib/Option | |
| parent | 02265382929b0275d7b7b334eab5e2fd34e1b9fe (diff) | |
| download | external_llvm-6c7e78746e059916892f1df279b681d962a70954.zip external_llvm-6c7e78746e059916892f1df279b681d962a70954.tar.gz external_llvm-6c7e78746e059916892f1df279b681d962a70954.tar.bz2 | |
Option parsing: allow aliases in groups
Option aliases in option groups were previously disallowed by an assert.
As far as I can tell, there was no technical reason for this, and I would
like to be able to put cl.exe compatible options in their own group for Clang,
so let's change the assert.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@186838 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Option')
| -rw-r--r-- | lib/Option/Option.cpp | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/lib/Option/Option.cpp b/lib/Option/Option.cpp index f1b7941..bed779a 100644 --- a/lib/Option/Option.cpp +++ b/lib/Option/Option.cpp @@ -22,12 +22,10 @@ using namespace llvm::opt; Option::Option(const OptTable::Info *info, const OptTable *owner) : Info(info), Owner(owner) { - // Multi-level aliases are not supported, and alias options cannot - // have groups. This just simplifies option tracking, it is not an - // inherent limitation. - assert((!Info || !getAlias().isValid() || (!getAlias().getAlias().isValid() && - !getGroup().isValid())) && - "Multi-level aliases and aliases with groups are unsupported."); + // Multi-level aliases are not supported. This just simplifies option + // tracking, it is not an inherent limitation. + assert(!Info || !getAlias().isValid() || !getAlias().getAlias().isValid() && + "Multi-level aliases are not supported."); } Option::~Option() { |