tarantool / luajit / 6812138826

93%

tarantool/master: 93%

push

github

Fix C file generation in jit.bcsave.

Thanks to codicodi.

(cherry picked from commit 62903bacf)

LuaJIT allows exporting bytecode to a C file using the option `-b`, see
[1]. For building a generated C file in C++ projects, C file uses a
macro `__cplusplus` [2], but this macro was broken by commit
a9dd47b7f ("Extend -b to generate
c/h/obj/o files with embedded bytecode."). With this breakage, C/C++
compiler removes the definition of an array with bytecode, and the
resulted object file has missed a symbol with bytecode.

The patch fixes the broken macro.

Note, test test/lua-Harness-tests/411-luajit.t checks the precense of
the macro `__cplusplus` in the generated C file, however, it doesn't
catch the bug.

Sergey Bronnikov:
* added the description and the test for the problem

Part of tarantool/tarantool#9145

1. https://luajit.org/running.html
2. https://en.cppreference.com/w/cpp/preprocessor/replace

5341 of 5972 branches covered (0.0%)

Branch coverage included in aggregate %.

20488 of 23293 relevant lines covered (87.96%)

2762723.9 hits per line