[PATCH] D122370: Split up large test files under clang/test/CodeGen/RISCV

2022-03-28 Thread Brandon Wu via Phabricator via cfe-commits
4vtomat added a comment. In D122370#3412522 , @luismarques wrote: > What's the timeout value that is being exceeded? I guess it’s 60 seconds? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122370/new/

[PATCH] D122370: Split up large test files under clang/test/CodeGen/RISCV

2022-03-28 Thread Luís Marques via Phabricator via cfe-commits
luismarques added a comment. What's the timeout value that is being exceeded? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122370/new/ https://reviews.llvm.org/D122370 ___ cfe-commits mailing list

[PATCH] D122370: Split up large test files under clang/test/CodeGen/RISCV

2022-03-23 Thread Brandon Wu via Phabricator via cfe-commits
4vtomat added a comment. In D122370#3404590 , @MaskRay wrote: > I assume this is to improve test parallelism. Do you have runtime comparison? actually, this is to prevent single file from timeout due to many test cases, such as

[PATCH] D122370: Split up large test files under clang/test/CodeGen/RISCV

2022-03-23 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. I assume this is to improve test parallelism. Do you have runtime comparison? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122370/new/ https://reviews.llvm.org/D122370 ___

[PATCH] D122370: Split up large test files under clang/test/CodeGen/RISCV

2022-03-23 Thread Brandon Wu via Phabricator via cfe-commits
4vtomat created this revision. Herald added subscribers: s, VincentWu, luke957, vkmr, frasercrmck, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, kito-cheng, niosHD, sabuasal,