[PATCH] D122939: [NVPTX][tests] Do not run the test CodeGen/Generic/2010-11-04-BigByval.ll

Igor Chebykin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 4 13:43:24 PDT 2022


i-chebykin added a comment.

Great, thanks for pointing me.

In D122939#3427311 <https://reviews.llvm.org/D122939#3427311>, @tra wrote:

> Not sure if you're the author of this post: https://discourse.llvm.org/t/nvptx-codegen-surprisingly-slow-on-some-functions/61307, but the issue is somewhat similar -- there's a quadratic behavior in LoadStoreVectorizer.

The spent time on my machine is about `const * (arg_size)^1.6`

> So, what we have here is that the test is nearly a show-stopper for the affected users, we do want to keep it running everywhere else as it has a value for most of the users, and we have no easy way to satisfy both.
>
> We have incoming LIT changes for conditional test execution (https://reviews.llvm.org/D122569). If we can expose 'default target' as a LIT feature, then we could run the test only if the default target is not NVPTX.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122939



More information about the llvm-commits mailing list