<div dir="ltr"><div>It's not intentional and I believe it's not actually parsed immediately. Can you elaborate a bit why you think files would be parsed not in parallel?</div><div><br></div><div>As to the name, I totally agree with that. parseFile is no longer a good name for that function. I'll rename loadFile (or readFile, but read could imply that we are going to parse it as part of reading, so I'm inclined to "load").</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 12, 2015 at 5:02 PM, <a href="mailto:kledzik@apple.com">kledzik@apple.com</a> <span dir="ltr"><<a href="mailto:kledzik@apple.com" target="_blank">kledzik@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It still looks like mach-o files are parsed immediately (not in parallel).  If that is intentional, because reworking the mach-o logic is complicated, then leave a FIXME comment there.<br>
<br>
It would be helpful to rename parseFile() if its goal is no longer to parse files.  I seems like its goal is to load the file into memory, and later parse()/doParse() are supposed to do the actual parsing.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D6874" target="_blank">http://reviews.llvm.org/D6874</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div>