<div class="gmail_quote">On Wed, Mar 2, 2011 at 12:14 PM, Dan Gohman <span dir="ltr"><<a href="mailto:gohman@apple.com">gohman@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div style="word-wrap:break-word"><div><div></div><div class="h5"><br>On Mar 2, 2011, at 7:54 AM, Liam Staskawicz wrote:<br><br><blockquote type="cite">ping.<br><br>Is anybody available to review and commit if OK?  Thanks!<br>
<br>Liam<br>Forwarded message:<br><br><blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px">From: Liam Staskawicz <<a href="mailto:lstask@gmail.com" target="_blank">lstask@gmail.com</a>><br>
To: <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>Date: Monday, February 28, 2011 10:39:43 AM<br>Subject: [PATCH] sys::fs::can_read() and sys::fs::can_write() for Unix<br><br>Hi - the attached patch provides implementations and tests for sys::fs::can_read() and sys::fs::can_write() for Unix.  These were adapted from the previous implementations in lib/Support/Unix/Path.inc.  can_execute() could be implemented quite similarly, but I wasn't sure where the best place to add a test for it would be, so left it out for now.</blockquote>
</blockquote><br></div></div><div>I'm surprised that anything in LLVM and clang is actually using canWrite().</div><div>It appears that there is one use, in clang, and that one looks pretty</div><div>suspicious. Perhaps it can be removed altogether instead of ported to PathV2?</div>
<div><br></div><font color="#888888"><div>Dan</div><div><br></div></font></div></blockquote><div><br></div><div>Removing it altogether would actually be fine from my perspective, though it would be good to know what the plan is, so I can plan accordingly.  Thanks.  </div>
<div><br></div><div>Liam</div></div><br>