[LLVMdev] fix warning with newer g++ compilers

Chris Lattner sabre at nondot.org
Sat Dec 15 09:33:47 PST 2007


On Dec 14, 2007, at 11:10 PM, Mike Stump wrote:

> Newer g++ compilers can emit:
>
> /Volumes/mrs5/net/llvm/llvm/llvm/lib/AsmParser/LLLexer.cpp:789:
> warning: suggest a space before ';' or explicit braces around empty
> body in 'for' statement
>
> The usual style is to put a space before the ;, or put the ; on a new
> line to visually separate the ; from the for statement, so that the
> follow like doesn't look like it goes with the for.

Hey Mike,

Thanks for the patch.  It doesn't apply because your mailer included  
it inline.  If you're using the apple mailer, putting a .patch suffix  
on it (instead of .txt) might help.

w.r.t the debate, I'd prefer:

  for (++CurPtr; isdigit(CurPtr[0]); ++CurPtr)
    /*empty*/;

but I really don't care that much :)

-Chris




More information about the llvm-dev mailing list