On Wed, Jun 20, 2018 at 5:42 PM Stella Stamenova <sti...@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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D46485: A... Bruno Cardoso Lopes via Phabricator via cfe-commits
    • [PATCH] D464... Bruno Cardoso Lopes via Phabricator via cfe-commits
    • [PATCH] D464... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D464... Bruno Cardoso Lopes via Phabricator via cfe-commits
    • [PATCH] D464... Bruno Cardoso Lopes via Phabricator via cfe-commits
    • [PATCH] D464... Stella Stamenova via Phabricator via cfe-commits
      • Re: [PAT... Bruno Cardoso Lopes via cfe-commits
        • Re: ... Bruno Cardoso Lopes via cfe-commits
          • ... Stella Stamenova via cfe-commits
            • ... Bruno Cardoso Lopes via cfe-commits
              • ... Stella Stamenova via cfe-commits
                • ... Galina Kistanova via cfe-commits

Reply via email to