jserv / rv32jit

JIT-accelerated RISC-V instruction set simulator
MIT License
30 stars 7 forks source link

Use "clang++-15" only or "clang++" 15 and higher? #3

Closed 25077667 closed 11 months ago

25077667 commented 11 months ago

The README said "You will need clang version 15+ for the build". However the Makefile is hard-coded clang++-15.

Would you like to use some checks like:

CXX := clang++

# checking CXX version is 15 or higher
CXX_VERSION := $(shell $(CXX) --version | grep -oP 'clang version \K\d+')

ifeq ($(shell echo "$(CXX_VERSION) >= 15" | bc), 1)
    $(info clang++ version is 15 or higher)
else
    $(error clang++ version is below 15)
endif