Skip to content

Commit

Permalink
Update minimum required compiler versions and C++ version
Browse files Browse the repository at this point in the history
Basically we need more C++17...
  • Loading branch information
keryell committed Mar 13, 2018
1 parent 360e083 commit 625e018
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions doc/testing.rst
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,11 @@ Testing
Installation
------------

Only Clang_ 3.9+ or GCC_ 5.4+, `Boost.MultiArray`_ (which adds to `C++`_
Only Clang_ 5+ or GCC_ 7+, `Boost.MultiArray`_ (which adds to `C++`_
the nice Fortran array semantics and syntax), Boost.Operators and a few
other Boost libraries are needed.

To have all tests compiling, it is better to use Clang_ 5.0+ or GCC_
To have all the tests compiling, it is better to use Clang_ 5.0+ or GCC_
7+.

To install them on latest Linux Debian/unstable (this should work on
Expand All @@ -24,7 +24,7 @@ latest Ubuntu too, just adapt the compiler versions):
There is nothing else to do for now to use the include files from triSYCL_
``include`` directory when compiling a program. Just add a
``-I.../include`` option and ``-std=c++14`` or ``-std=c++17`` when compiling.
``-I.../include`` option and ``-std=c++1z`` or ``-std=c++17`` when compiling.

triSYCL_ is configurable through preprocessor macros described in
`macros <macros.rst>`_.
Expand Down
2 changes: 1 addition & 1 deletion tests/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@ CLEANING_TARGETS += $(KERNELS_BC) $(KERNELS_BIN) $(KERNELS_INTERNALIZED_CXX) \
# The implementation uses C++14.
# Use -Wno-ignored-attributes to avoid a lot of warning with
# Boost.Compute and g++ version >= 6
CXXFLAGS += -Wall -Wno-ignored-attributes -std=c++1y -I$(triSYCL_DIR)/include \
CXXFLAGS += -Wall -Wno-ignored-attributes -std=c++1z -I$(triSYCL_DIR)/include \
-I$(triSYCL_DIR)/tests/common

# Vivado HLS path
Expand Down

0 comments on commit 625e018

Please sign in to comment.