[PATCH] D140898: [llvm-exegesis] Remove functional test from unit tests

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 4 02:44:50 PST 2023


gchatelet added a comment.

In D140898#4023600 <https://reviews.llvm.org/D140898#4023600>, @rupprecht wrote:

> If you want to keep this test but just not run it by default, can you put it in a separate target that the build bot doesn't run? I guess it's up to you.
>
> btw, we should be able to remove `nobuildkite` from llvm/unittests/BUILD.bazel after this patch. (Or if we keep the test but put it in a separate target, move the tag to the new one). I can land that separately if you prefer.

Thx! I've submitted it separately as 8ec0a369675b8406460fac1f94a6f2d84b7c0bb4 <https://reviews.llvm.org/rG8ec0a369675b8406460fac1f94a6f2d84b7c0bb4>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140898



More information about the llvm-commits mailing list