<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, May 25, 2017 at 5:20 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div>I guess it's possible I'm just missing something, but what I don't understand is why `Thin` can't just be the only determining factor of whether we write full paths.  Why the second variable?  AFAICT nowhere in this patch is `false` ever explicitly passed, it only takes on a `false` value implicitly when `Thin` is also false.  So just delete the variable, and only use the value of `Thin`?</div></div></div>
</blockquote></div><br></div><div class="gmail_extra">Microsoft tools won't understand thin archives. Thin archives leave the object files on disk instead of copying them into the archive. The whole point of this patch is to make llvm-lib.exe behave more like lib.exe, so it would defeat the purpose if we went our own way here.</div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote">On Thu, May 25, 2017 at 5:55 PM, Rafael Avila de Espindola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">> The names we invent for import files don't have path separators in them, so<br>> there should be no behavior change either way here. By passing true, we<br>> don't needlessly run sys::path::filename on them. Are you suggesting that<br>> maybe it will cause us to not use the short strtab format? I guess I'll<br>> change it back. Most DLL names are < 16 characters.<br><br></span>How about passing the final name to useStringTable? That way it can just<br>check if there is '/' in it or if it is too large.<br></blockquote><div><br></div><div>Sure. So, if the relative path from a thin archive to the object file is < 16 chars and has no slashes, is it OK to store it in the archive member header?</div></div></div></div>