[llvm-commits] CVS: llvm/include/llvm/Support/CommandLine.h

[Reid Spencer]

On Fri, 2005-06-17 at 11:24 -0500, Chris Lattner wrote:
> On Fri, 17 Jun 2005, Reid Spencer wrote:
> >> silence a huge number of bogus warnings with GCC 4 on Reid's tester.
> >>
> >> diff -u llvm/include/llvm/Support/CommandLine.h:1.43 llvm/include/llvm/Support/CommandLine.h:1.44
> >> -    typename ParserClass::parser_data_type Val;
> >> +    typename ParserClass::parser_data_type Val =
> >> +       typename ParserClass::parser_data_type();
> >
> > I tried this a while back. I think this runs into some issues when
> > parser_data_type isn't constructible. I've tried various initializations
> > but there doesn't seem to be a consistent one that works with all
> > parser_data_type types.  Did all of LLVM recompile correctly with this
> > change?
> 
> Yes, it does with gcc 3.4

Okay, seems to work with 4.0.0 too. Not sure what I tried to do
previously but a similar patch didn't (previously) work with 4.0.0. Glad
this annoying warning is out of the way. Thanks for taking care of it!

Reid
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20050617/2d83ad77/attachment.sig>