You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am testing libCEED with a new OKL transpiler (https://github.com/libocca/occa-transpiler) and found a couple of possible bugs that were tolerated by the original OCCA transpilation:
backends/occa/kernels/simplex-basis/cpu-simplex-basis.cpp line 34
Redundant comma at the end of list of the arguments
Function names in OCCA kernels
Original OCCA was performing the macro expansion differently than the standard states, please refer to the issue: OKL incorrect macro expansion libocca/occa#750
That makes the code like TENSOR_FUNCTION(interpElement) or SIMPLEX_FUNCTION(interpElement) behave differently than expected, generating something like: simplex_DIMd_interpElementTranspose_QQ_PP
The text was updated successfully, but these errors were encountered:
@ypanke If you want to try fixing these issues, we'll be happy to review PRs. To my knowledge, nobody is using the OCCA backend, and despite being more lines of code in the implementation than the CUDA or HIP backends, it's substantially slower.
Hello
I am testing libCEED with a new OKL transpiler (https://github.com/libocca/occa-transpiler) and found a couple of possible bugs that were tolerated by the original OCCA transpilation:
backends/occa/kernels/simplex-basis/cpu-simplex-basis.cpp
line 34Redundant comma at the end of list of the arguments
Original OCCA was performing the macro expansion differently than the standard states, please refer to the issue: OKL incorrect macro expansion libocca/occa#750
That makes the code like
TENSOR_FUNCTION(interpElement)
orSIMPLEX_FUNCTION(interpElement)
behave differently than expected, generating something like:simplex_DIMd_interpElementTranspose_QQ_PP
The text was updated successfully, but these errors were encountered: