<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
p.msonormal0, li.msonormal0, div.msonormal0
{mso-style-name:msonormal;
mso-margin-top-alt:auto;
margin-right:0in;
mso-margin-bottom-alt:auto;
margin-left:0in;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
span.EmailStyle18
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri",sans-serif;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal">I think with all the tests running in parallel we should err on the side of having unique names for everything automatically generated.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><b>From:</b> Aleksandr Urakov <aleksandr.urakov@jetbrains.com>
<br>
<b>Sent:</b> Wednesday, December 5, 2018 12:48 PM<br>
<b>To:</b> Zachary Turner <zturner@google.com><br>
<b>Cc:</b> reviews+D54942+public+2603ca548f36d222@reviews.llvm.org; Stella Stamenova <stilis@microsoft.com>; llvm-commits@lists.llvm.org; abidh.haq@gmail.com; teemperor@gmail.com; lldb-commits@lists.llvm.org; sanimir@subpath.org; chirag@raincode.com; b.gianfo@gmail.com<br>
<b>Subject:</b> Re: [PATCH] D54942: [PDB] Make PDB lit tests use the new builder<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal" style="margin-bottom:12.0pt">With such solution there would be even no need to change the current commit. But I'm not sure that it's trivial to do - the output file name may contain path with directories. May be we can replace slashes with
underscores in the output file path and concatenate it with the object file name? Or even replace slashes in the source file path and concatenate it with the output file path - so object files will be located in the same place as the output file.<o:p></o:p></p>
<div>
<div>
<p class="MsoNormal">Am Mi., 5. Dez. 2018, 23:30 hat Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> geschrieben:<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal">It is not possible to specify object file name in compile and link mode. But perhaps we can just change the default object file name to include something from the output file as well
<o:p></o:p></p>
<div>
<div>
<p class="MsoNormal">On Wed, Dec 5, 2018 at 12:26 PM Aleksandr Urakov via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal" style="margin-bottom:12.0pt">aleksandr.urakov added a subscriber: zturner.<br>
aleksandr.urakov added a comment.<br>
<br>
The similar problem with `typedefs.test` is here: <a href="https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Flab.llvm.org%3A8014%2Fbuilders%2Flldb-x64-windows-ninja%2Fbuilds%2F1940%2Fsteps%2Ftest%2Flogs%2Fstdio&data=02%7C01%7Cstilis%40microsoft.com%7C91af7eec00544628293a08d65af2edaa%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636796396749066675&sdata=qk20%2B1v4xKGNNWic0O8p5ybgNJF5jzurS%2BW3YXxuvKQ%3D&reserved=0" target="_blank">
http://lab.llvm.org:8014/builders/lldb-x64-windows-ninja/builds/1940/steps/test/logs/stdio</a><br>
<br>
I have an assumption about the cause of the problem. Are the tests running in parallel? In this case `typedefs.test` and `enums-layout.test` are writing to the same object file together, because they both are compiled from the same source.<br>
<br>
@zturner Is it possible to specify object file's name in `compile-and-link` mode? Then we can specify different names in different tests for both object files and executables. But I think that splitting the source or combining the tests would be a better idea.<br>
<br>
I will fix it only tomorrow, because I'm already at home today. Feel free to revert it if needed.<br>
<br>
<br>
Repository:<br>
rL LLVM<br>
<br>
CHANGES SINCE LAST ACTION<br>
<a href="https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Freviews.llvm.org%2FD54942%2Fnew%2F&data=02%7C01%7Cstilis%40microsoft.com%7C91af7eec00544628293a08d65af2edaa%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636796396749076680&sdata=IZDR90FOCIsmfggh9InT9n%2FDb1jESMfX1RXiwJdgHMg%3D&reserved=0" target="_blank">
https://reviews.llvm.org/D54942/new/</a><br>
<br>
<a href="https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Freviews.llvm.org%2FD54942&data=02%7C01%7Cstilis%40microsoft.com%7C91af7eec00544628293a08d65af2edaa%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636796396749076680&sdata=OqLhRdcrQtVav7%2BlfRNCbGZA2xijPr2WYWYQ9v5rFdI%3D&reserved=0" target="_blank">https://reviews.llvm.org/D54942</a><br>
<br>
<br>
<o:p></o:p></p>
</blockquote>
</div>
</blockquote>
</div>
</div>
</div>
</div>
</body>
</html>