[PATCH] D147835: [X86][mem-fold] Speed up test by not re-generating the .inc file

[Nico Weber via Phabricator via llvm-commits]

thakis added a comment.

> @thakis Yes, it's an option to me for most cases, thanks. Even 14 seconds is acceptable for me. I created this patch just for a better speed not for a concern. I'm fine with your revert. BTW, should we add back the `REQUIRES: x86-registered-target` in the test? I don't think users care about the test when x86 is not the build targets.

Great, thanks!

If the requires isn't needed, I wouldn't add it. I think it's nice if as many tests as possible run independent of build settings. That makes it easier to debug test failures locally.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147835