[llvm-commits] [PATCH] [lld] Factor File::addAtom out to MutableFile::addAtom.

Nick Kledzik kledzik at apple.com
Fri Jan 11 13:50:08 PST 2013


I don't see anything "wrong" with it.  I've seen mutability as part of the type system used in other languages, but not so much in C++ because careful use of const can do the same thing.  But if you guys have seen that this as better in practice, then go ahead.

-Nick

On Jan 11, 2013, at 1:30 PM, Michael Spencer wrote:
>  Ping. I'd like to get a consensus here.
> 
> http://llvm-reviews.chandlerc.com/D274




More information about the llvm-commits mailing list