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

Lang Hames lhames at gmail.com
Wed Jun 19 19:20:17 PDT 2013


Excellent. Thanks Rafael!


On Tue, Jun 18, 2013 at 5:31 AM, Rafael EspĂ­ndola <
rafael.espindola at gmail.com> wrote:

> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130619/da2c6471/attachment.html>


More information about the llvm-commits mailing list