NVIDIA / cutlass

CUDA Templates for Linear Algebra Subroutines
Other
5.64k stars 961 forks source link

[BUG] Broken copy.hpp #1508

Open kroburg opened 6 months ago

kroburg commented 6 months ago

Describe the bug I can't compile cute/copy.hpp. Get error: cutlass/include/cute/atom/copy_traits_sm90_tma.hpp(73): error: namespace "cute::detail" has no member "is_prefetch"

Steps/Code to reproduce bug test.cu #include <cute/algorithm/copy.hpp>

/usr/local/cuda/bin/nvcc -forward-unknown-to-host-compiler -I/usr/local/cuda/include -Icontrib/cutlass/include -Icontrib/cutlass/tools/util/include -g --generate-code=arch=compute_89,code=[compute_89,sm_89] -x cu -c test.cu -o test.cu.o

contrib/cutlass/include/cute/atom/copy_traits_sm90_tma.hpp(71): error: namespace "cute::detail" has no member "is_prefetch"
      if constexpr (detail::is_prefetch<CopyOp>) {
                            ^

contrib/cutlass/include/cute/atom/copy_traits_sm90_tma.hpp(71): error: type name is not allowed
      if constexpr (detail::is_prefetch<CopyOp>) {
                                        ^

contrib/cutlass/include/cute/atom/copy_traits_sm90_tma.hpp(71): error: expected an expression
      if constexpr (detail::is_prefetch<CopyOp>) {
                                               ^

3 errors detected in the compilation of "test.cu".
thakkarV commented 6 months ago

Please see https://github.com/NVIDIA/cutlass/issues/1484

kroburg commented 6 months ago

Please see #1484

Thank you!

Please notice that actual broken code is Flash Attention.

thakkarV commented 6 months ago

Yes there's a fix coming for that specific issue imminently.

github-actions[bot] commented 5 months ago

This issue has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. This issue will be labeled inactive-90d if there is no activity in the next 60 days.

github-actions[bot] commented 2 months ago

This issue has been labeled inactive-90d due to no recent activity in the past 90 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed.