spcl / spcl/dace

std::complex / double Allocation Dimensions

Open
#1,994 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
593
Forks
163
Avg merge
2d 23h
Merged PRs (30d)
60

Description

**Describe the bug**

NPBench `stocckham_fft` benchmark fails with the following error:

```
***** Testing DaCe CPU with stockham_fft on the S dataset *****
NumPy - default - validation: 8ms
Failed to compile DaCe cpu fusion implementation.
Compiler failure:
...
/home/primrose/Work/npbench/.dacecache/fusion/src/cpu/fusion.cpp:53:52: error: expression in new-declarator must have integral or enumeration type
53 | tmp_perm = new dace::complex128 DACE_ALIGN(64)[(dace::math::pow(R, K))];
| ^
/home/primrose/Work/npbench/.dacecache/fusion/src/cpu/fusion.cpp:53:75: warning: ignoring attributes applied to class type ‘dace::complex128’ {aka ‘std::complex’} outside of definition [-Wattributes]
53 | tmp_perm = new dace::complex128 DACE_ALIGN(64)[(dace::math::pow(R, K))];
| ^
/home/primrose/Work/npbench/.dacecache/fusion/src/cpu/fusion.cpp:55:45: error: expression in new-declarator must have integral or enumeration type
55 | D = new dace::complex128 DACE_ALIGN(64)[(dace::math::pow(R, K))];
| ^
/home/primrose/Work/npbench/.dacecache/fusion/src/cpu/fusion.cpp:55:68: warning: ignoring attributes applied to class type ‘dace::complex128’ {aka ‘std::complex’} outside of definition [-Wattributes]
55 | D = new dace::complex128 DACE_ALIGN(64)[(dace::math::pow(R, K))];
```

Need to extend validation for dtypes. (TODO I will add details to this)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.