From 6c7e78746e059916892f1df279b681d962a70954 Mon Sep 17 00:00:00 2001 From: Hans Wennborg Date: Mon, 22 Jul 2013 16:18:13 +0000 Subject: 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 --- lib/Option/Option.cpp | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) (limited to 'lib/Option') 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() { -- cgit v1.1