[PATCH] D91545: [AMDGPU] Consolidate memory-legalizer tests

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 16 10:48:58 PST 2020


rampitec added a comment.

Latest changes in this direction are already pretty invasive. If you run lit tests with a debug compiler build these combined tests take minutes to complete. If you need to debug something there it is very slow, and if you just run make check it is very slow, I assume because otherwise several smaller tests could run in parallel. They also take minutes if not tenth of minutes to load into a browser for review. So I would rather split these tests, not combine them.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91545



More information about the llvm-commits mailing list