[patch] Delete unnecessary generality in loadFile
ruiu at google.com
Thu Apr 23 13:12:07 PDT 2015
This is a change I wanted to create, so LGTM. Thank you for doing this.
One thing this could be improved is probably to return
ErrorOr<unique_ptr<File>> instead of error_code. Returning the actual
return value using 3rd argument which is passed-by-reference looks a bit
On Thu, Apr 23, 2015 at 1:06 PM, Rafael Espíndola <
rafael.espindola at gmail.com> wrote:
> loadFile can load mulitple files just because yaml has a feature for
> putting multiple documents in one file.
> Designing a linker around what yaml can do seems like a bad idea to
> me. The attached wip patch changes it to read a single file. It also
> updates one test to show the difference.
> OK with the remaining tests updated?
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits