Is it worth it to introduce a new class just to get rid of some llvm_unreachables? The mutability was already enforced via const. That is, only a non-const File could have its addAtoms() method called. -Nick http://llvm-reviews.chandlerc.com/D274