[PATCH] D142225: [memprof] Change test output redirection to enable it to run on Windows

Andrew Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 27 01:55:29 PST 2023


andrewng added a comment.

> While the change itself looks good to me, this test is for linux (`REQUIRES: x86_64-linux`). Is there some motivation for this change?

If you build a Windows LLVM toolchain that targets `x86_64-linux` by default (i.e. cross compilation for Linux on Windows), then this test will be run.

Thanks for the review.


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

https://reviews.llvm.org/D142225



More information about the llvm-commits mailing list