ctopcsutahedu / bricklib Goto Github PK

1. This branch is hit at compile time:

   bricklib/stencils/cpuvfold.h

   Line 27 in d817250

   #error "This is for SVE512"

   which is rather annoying. Can you test whether 512 vector width is supported in CMake, and disable it if unsupported? Otherwise the build simply fails.
2. Can you please make this here

   bricklib/CMakeLists.txt

   Lines 105 to 118 in d817250

   	set(BRICK_BUILD_ARCH "native" CACHE STRING "Specify build architecture target")
   	check_cxx_compiler_flag("-march=${BRICK_BUILD_ARCH}" HAS_MARCH)
   	if (HAS_MARCH)
   	message(STATUS "Using march=${BRICK_BUILD_ARCH}")
   	add_compile_options(-march=${BRICK_BUILD_ARCH})
   	else ()
   	# Work around Summit only have mcpu flag
   	check_cxx_compiler_flag("-mcpu=${BRICK_BUILD_ARCH}" HAS_MCPU)
   	if (HAS_MCPU)
   	message(STATUS "Using mcpu=${BRICK_BUILD_ARCH}")
   	add_compile_options(-mcpu=${BRICK_BUILD_ARCH})
   	endif ()
   	endif ()

   opt in? Compiler wrappers may set the relevant -march flag, so in your CMake logic you can just test what vectorization features are supported without any additional flags and work with that.

In commit 4849707 (by @Sam3077), two images were added that should be stored in LFS, but they are not available on the Github repository. This causes failures to build in Spack when using Git. FWIW, it was merged via commit 4849707.

Wherever these files live, git lfs push --all $remote_for_github (guessing because the commit messages reference issue/PR ids that don't exist here, so I assume there is some other development platform in use) should be able to resolve it.

When I try to generate cuda code using vecscatter, I am facing the following error.
I run the command # vecscatter 3axis.cu out.cu in bricklib/stencils/

In file included from /usr/include/c++/5/type_traits:35:0,
                 from /home/bricklib/include/brick.h:10,
                 from /home/bricklib/include/brick-cuda.h:9,
                 from stencils_cu.h:8,
                 from 3axis.cu:5:
/usr/include/c++/5/bits/c++0x_warning.h:32:2: error: #error This file requires compiler and library support for the ISO C++ 2011 standard. This support must be enabled with the -std=c++11 or -std=gnu++11 compiler options.
 #error This file requires compiler and library support \
  ^

3axis.cu 226
Traceback (most recent call last):
  File "/root/miniconda3/bin/vecscatter", line 4, in <module>
    __import__('pkg_resources').run_script('st==0.1', 'vecscatter')
  File "/root/miniconda3/lib/python3.8/site-packages/pkg_resources/__init__.py", line 650, in run_script
    self.require(requires)[0].run_script(script_name, ns)
  File "/root/miniconda3/lib/python3.8/site-packages/pkg_resources/__init__.py", line 1446, in run_script
    exec(code, namespace, namespace)
  File "/root/miniconda3/lib/python3.8/site-packages/st-0.1-py3.8.egg/EGG-INFO/scripts/vecscatter", line 184, in <module>
    eval(call)
  File "<string>", line 1, in <module>
  File "/root/miniconda3/lib/python3.8/site-packages/st-0.1-py3.8.egg/EGG-INFO/scripts/vecscatter", line 150, in Brick
    backend = get_backend(vec)
  File "/root/miniconda3/lib/python3.8/site-packages/st-0.1-py3.8.egg/EGG-INFO/scripts/vecscatter", line 83, in get_backend
    from st.codegen.backend import BackendAVX512, BackendAVX2, BackendScalar, BackendCUDA, BackendFlex, BackendSSE, \
ModuleNotFoundError: No module named 'st.codegen'

By the way, the Codegen Integeration guide page (https://github.com/CtopCsUtahEdu/bricklib/blob/r0.1) seems to be lost. I'll appreciate it a lot if someone can fix it. Thank you for attention.

bricklib is built in https://github.com/spack/spack CI, but right now it looks like it's better to drop this cause:

1. #9 compilation issues
2. there are no releases, so there's no way to get a deterministic build other than fixing a particular commit.

Is it possible to fix both issues?