<div dir="ltr"><div>I can give it a try as well.</div><div><br></div><div>Thanks</div><div><br></div><div>Galina<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 20, 2018 at 6:30 PM, Stella Stamenova via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div>
<div>


<div style="direction:ltr">
<div style="direction:ltr">Yes, I can try it out. Feel free to add me to the review as well.</div>
</div>
<div> </div>
<hr style="display:inline-block;width:98%">
<div id="m_3903728985732245820divRplyFwdMsg" dir="dir="ltr""><font style="font-size:11pt" face="Calibri, sans-serif" color="#000000"><span class=""><b>From:</b> Bruno Cardoso Lopes <<a href="mailto:bruno.cardoso@gmail.com" target="_blank">bruno.cardoso@gmail.com</a>><br>
</span><b>Sent:</b> Wednesday, June 20, 2018 6:25 PM<br>
<b>To:</b> Stella Stamenova<br>
<b>Cc:</b> Richard Smith; Duncan Exon Smith; <a href="mailto:jkorous@apple.com" target="_blank">jkorous@apple.com</a>; Michał Górny; cfe-commits<span class=""><br>
<b>Subject:</b> Re: [PATCH] D46485: Add python tool to dump and construct header maps
<div> </div>
</span></font></div><span class="">
On Wed, Jun 20, 2018 at 5:42 PM Stella Stamenova <<a href="mailto:stilis@microsoft.com" target="_blank">stilis@microsoft.com</a>> wrote: <br>
> <br>
> Thanks Bruno, <br>
> <br>
> I ran a build as well and I can see that hmaptool is now in the correct bin directory. The tests still failed though because on Windows, at least, you need to explicitly call python to run a script e.g. "python hmaptool".
<br>
> <br>
> There are a few tests in LLVM that do that, for example: <br>
> <br>
> ; RUN: %python -c "print(' ' * 65536)" > %t.cache/llvmcache-foo <br>
> <br>
> I am not sure whether you can simply call python on hmaptool or if you would have to include the fullpath to it though.
<br>
<br>
Oh, I see. I'll revert the commits while I find a solution for this. <br>
Any chance you can give it a try before I re-commit if I send you an <br>
updated patch? <br>
<br>
Thanks, <br>
<br>
-- <br>
Bruno Cardoso Lopes <br>
</span><a href="https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.brunocardoso.cc&data=02%7C01%7Cstilis%40microsoft.com%7C9a3d5b51b26c4ff1a8db08d5d715df88%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636651411307060645&sdata=2EamqaQFSd35ZIqp%2Ft9TxAUZYoeDg8NkD47VaERkoG8%3D&reserved=0" target="_blank">https://na01.safelinks.<wbr>protection.outlook.com/?url=<wbr>http%3A%2F%2Fwww.brunocardoso.<wbr>cc&data=02%7C01%7Cstilis%<wbr>40microsoft.com%<wbr>7C9a3d5b51b26c4ff1a8db08d5d715<wbr>df88%<wbr>7C72f988bf86f141af91ab2d7cd011<wbr>db47%7C1%7C0%<wbr>7C636651411307060645&sdata=<wbr>2EamqaQFSd35ZIqp%<wbr>2Ft9TxAUZYoeDg8NkD47VaERkoG8%<wbr>3D&reserved=0</a>
<br>
</div>
</div>

<br>______________________________<wbr>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>