[PATCH] D46499: [Support] Add docs for 'openFileFor{Write,Read}'

Brian Gesiak via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 9 08:55:50 PDT 2018


modocache added a comment.

Thanks for the comments!

> Name is really the *path* of the file to open, right?

Correct, it's the path. It might be aelative or absolute. I'll update the docs (but not the variable name).

> What happens to ResultFD if the function fails?

Good point. On Unix it's set to the return value of `open(2)`. I'll double check what happens on Windows and update the docs.


Repository:
  rL LLVM

https://reviews.llvm.org/D46499





More information about the llvm-commits mailing list