
<!DOCTYPE html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div>Hi Jordan,<br></div><div><br></div><div>I'm looking into the fallout of a proposal I'm making to deprecate `sys::path::system_temp_directory`. That lead me to D59033, which then caused me to wonder what problem it was trying to solve.<br></div><div><br></div><div>Dave</div><div><br></div><div>On Wed, Jun 24, 2020, at 11:35 AM, Jordan Rupprecht via llvm-dev wrote:<br></div><blockquote type="cite" id="qt" style=""><div dir="ltr">I was able to dig up that this was reviewed in D59033, which was accidentally omitted from the commit message. But I can't recall/find any more context that you're looking for. (Not the author, but I reviewed it). Are you noticing race condition issues here, or just curious about the history?<br></div><div><br></div><div class="qt-gmail_quote"><div dir="ltr" class="qt-gmail_attr">On Tue, Jun 23, 2020 at 5:26 AM David Zarzycki via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="qt-gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204, 204, 204);padding-left:1ex;"><div>Hi Jake,<br></div><div> <br></div><div> About a year ago in commit 5049c3422d26b2b68877307c41b35d7e6aae3235, you attempted to solve a race in llvm-objcopy. What was the race? I ask because unless "last change wins" is the result you want, then the race isn't solved. The problem is that `sys::fs::rename` is just a thin wrapper around POSIX semantics, and replacing an existing file is not an error.<br></div><div> <br></div><div> Dave<br></div><div> _______________________________________________<br></div><div> LLVM Developers mailing list<br></div><div> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br></div><div> <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br></div></blockquote></div><div>_______________________________________________<br></div><div>LLVM Developers mailing list<br></div><div><a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br></div><div><a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br></div><div><br></div><div><br></div><div><b>Attachments:</b><br></div><ul><li>smime.p7s<br></li></ul></blockquote><div><br></div></body></html>