[PATCH] Add option to operate on symbolic links for sys::fs::remove() and sys::fs::status()

Rafael Espíndola rafael.espindola at gmail.com
Thu Mar 20 10:25:54 PDT 2014


> Yes, this seems better.

Cool. It LGTM :-)

> But isn’t the sys::fs::status change to allow stat’ing symlinks good in general ? Otherwise how is sys::fs::is_symlink() supposed to work, it doesn’t work currently.

sys::fs is a bit odd. It is very llvm specific (like stating to make
sure we cannot remove /dev/null), but is somewhat over general for
that. I would like to make it include only the small bits that llvm
actually uses, even if one day sys::fs is implemented on top of a
generic, standard c++ api, that api would necessarily be much more
complicated than sys::fs.

So the answer is that we should remove is_symlink, since it is dead. I
will do that in a sec.

Cheers,
Rafael




More information about the llvm-commits mailing list