[llvm] r196607 - Integrated assembler incorrectly lexes ARM-style comments

David Peixotto dpeixott at codeaurora.org
Fri Dec 6 14:52:07 PST 2013


> Just for consistency with I'd put the initialization in the body of the
> constructor like the initializer for isAtStartOfLine. Good to commit with
> that change. Thanks!

I had to put it in the initializer list because I made the class variable
const. I wasn't sure about that decision. Is it better to keep it const and
use the initializer list, or make it non-const (then I can initialize it in
the body)?





More information about the llvm-commits mailing list