[PATCH] D46485: Add python tool to dump and construct header maps

Bruno Cardoso Lopes via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 20 18:25:15 PDT 2018


On Wed, Jun 20, 2018 at 5:42 PM Stella Stamenova <stilis at microsoft.com> wrote:
>
> Thanks Bruno,
>
> 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".
>
> There are a few tests in LLVM that do that, for example:
>
> ; RUN: %python -c "print(' ' * 65536)" > %t.cache/llvmcache-foo
>
> I am not sure whether you can simply call python on hmaptool or if you would have to include the fullpath to it though.

Oh, I see. I'll revert the commits while I find a solution for this.
Any chance you can give it a try before I re-commit if I send you an
updated patch?

Thanks,

-- 
Bruno Cardoso Lopes
http://www.brunocardoso.cc


More information about the cfe-commits mailing list