[PATCH] D120129: [NVPTX] Enhance vectorization of ld.param & st.param

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 25 08:48:49 PDT 2022


jdoerfert added a comment.

In D120129#3408116 <https://reviews.llvm.org/D120129#3408116>, @kovdan01 wrote:

> In D120129#3408088 <https://reviews.llvm.org/D120129#3408088>, @jdoerfert wrote:
>
>> Godbold seems to not have included this:
>>
>>   $ cat test.cpp
>>   int main(){return 0;}
>>   $ clang++ -fopenmp -fopenmp-targets=nvptx64 -Xopenmp-target -march=sm_80 test.cpp
>>   ... static bool llvm::isa_impl_cl<llvm::ConstantAsMetadata, const llvm::Metadata *>::doit(const From *) [To = llvm::ConstantAsMetadata, From = const llvm::Metadata *]: Assertion `Val && "isa<> used on a null pointer"' failed.
>
> Thanks for the reproducer! Is the problem specific for sm_80 or not? I am unable to reproduce the issue with sm_75 and lower.

You don't need to run it. If you use this command line it doesn't crash?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120129



More information about the cfe-commits mailing list