[PATCH] D153725: [clang] Make amdgpu-arch tool work on Windows

Jon Chesterfield via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 10 06:35:15 PDT 2023


JonChesterfield added a comment.

In D153725#4484966 <https://reviews.llvm.org/D153725#4484966>, @arsenm wrote:

> In D153725#4484754 <https://reviews.llvm.org/D153725#4484754>, @JonChesterfield wrote:
>
>> - if you open the driver too many times at once it fails to open, so running a parallel build that uses this tool doesn't work on fast machines
>
> Why would this happen? Seems like a bug to fix?

It definitely annoys me. The argument is you can't usefully run some large N number of programs at the same time anyway and the driver failing to open is a rate limit. The problem is there are things we could usefully do, like this query, without needing to run a kernel as well. The net effect is we don't run tests widely in parallel because they fail if we do, for this and possibly other reasons.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153725/new/

https://reviews.llvm.org/D153725



More information about the cfe-commits mailing list