search

immunant / c2rust

Migrate C code to Rust
https://c2rust.com/
Other
4k stars 240 forks source link

Bump CXX_STANDARD to 17 for LLVM 16 #892

Closed thedataking closed 1 year ago

thedataking commented 1 year ago

Summary of changes:

  1. Request C++17 in the makefiles for the ASTExtractor. This should gracefully fall back to C++14 if the compiler is too old and nothing seems to break in CI. Closes #891 which independently suggested that C++17 was needed on Darwin.

  2. Add the compiler resource dir such that the transpiler test suite passes on Darwin.

chrysn commented 1 year ago

Given this affects a wide range of systems (MacOS, and I've seen reports on Alpine Linux as well), would you consider pushing a 0.17.1 that includes this?