[PATCH] D22308: [pdb] Introduce an MsfBuilder class for laying out PDB files

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 13 15:51:39 PDT 2016


ruiu added a comment.

You can return an Error on second and subsequent calls of `build()`, no? I don't see a reason to allow users to call `build()` multiple times. If they want to create multiple MSF headers, they can simply create as many MsfBuilder instances they want, and call `build()` only once for each instance.


http://reviews.llvm.org/D22308





More information about the llvm-commits mailing list