[llvm-commits] [llvm] r63384 - in /llvm/trunk: include/llvm/Support/CommandLine.h lib/Support/CommandLine.cpp

[Mike Stump]

On Jan 30, 2009, at 10:54 AM, Chris Lattner wrote:
>> On Jan 30, 2009, at 10:00 AM, Chris Lattner wrote:
>>> Please revert this patch in the meantime, thanks!
>>
>> Any objection to just fixing it?
>
> What do you mean?

Tweaking the interface to be as you requested....  The typical usage  
would look like:

static llvm::cl::opt<bool>
EnableBlocks("fblocks", llvm::cl::desc("enable the 'blocks' language  
feature"),
              llvm::cl::ValueDisallowed, llvm::cl::AllowInverse);


How about the below?  I think the usage side is nicer.

I can do up the doc changes for consideration, if the patches are  
reasonable enough.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: opt-1.patch
Type: application/octet-stream
Size: 5112 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090130/88342c26/attachment.obj>
-------------- next part --------------