[llvm] r183985 - Remove a use of PathV1.h.

Rafael EspĂ­ndola rafael.espindola at gmail.com
Tue Jun 18 05:31:41 PDT 2013


On 17 June 2013 18:09, Lang Hames <lhames at gmail.com> wrote:
> Hi Rafael,
>
> No worries. Thanks for fixing it so quickly.
>
> Just so I understand - the plan is ultimately to add these checks to
> sys::fs::remove? (Or add a "safe" remove variant that does the checks)?

I added the check to sys::fs::remove, since according to the comment
in unlinkFromDisk we don't want any unsafe uses in llvm.

> Once that's done we should be able to re-apply your patch, right?

I have done it :-)

Cheers,
Rafael



More information about the llvm-commits mailing list