mirror of
https://github.com/facebook/zstd.git
synced 2025-08-05 19:15:58 +03:00
meson: partial fix for building pzstd on MSVC
It uses non-portable compiler options unconditionally. Elsewhere, we check the compiler ID and only add the right ones, globally. Do the same here. NDEBUG can actually be handled by a core option, so while we are moving things around, do so. Unfortunately, this doesn't fix things entirely. The remaining issue is not Meson's issue though -- MSVC simply does not like this source code and somehow chokes on innocent code with the inscrutable "syntax error" and "illegal token".
This commit is contained in:
committed by
Nick Terrell
parent
a91e7ec175
commit
e0ef09ddba
@@ -18,7 +18,8 @@ pzstd_sources = [join_paths(zstd_rootdir, 'programs/util.c'),
|
|||||||
join_paths(zstd_rootdir, 'contrib/pzstd/SkippableFrame.cpp')]
|
join_paths(zstd_rootdir, 'contrib/pzstd/SkippableFrame.cpp')]
|
||||||
pzstd = executable('pzstd',
|
pzstd = executable('pzstd',
|
||||||
pzstd_sources,
|
pzstd_sources,
|
||||||
cpp_args: [ '-DNDEBUG', '-Wno-shadow', '-Wno-deprecated-declarations' ],
|
cpp_args: pzstd_warning_flags,
|
||||||
include_directories: pzstd_includes,
|
include_directories: pzstd_includes,
|
||||||
dependencies: [ libzstd_dep, thread_dep ],
|
dependencies: [ libzstd_dep, thread_dep ],
|
||||||
|
override_options: ['b_ndebug=true'],
|
||||||
install: true)
|
install: true)
|
||||||
|
@@ -104,8 +104,10 @@ use_lz4 = lz4_dep.found()
|
|||||||
|
|
||||||
add_project_arguments('-DXXH_NAMESPACE=ZSTD_', language: ['c'])
|
add_project_arguments('-DXXH_NAMESPACE=ZSTD_', language: ['c'])
|
||||||
|
|
||||||
|
pzstd_warning_flags = []
|
||||||
if [compiler_gcc, compiler_clang].contains(cc_id)
|
if [compiler_gcc, compiler_clang].contains(cc_id)
|
||||||
common_warning_flags = [ '-Wundef', '-Wshadow', '-Wcast-align', '-Wcast-qual' ]
|
common_warning_flags = [ '-Wundef', '-Wshadow', '-Wcast-align', '-Wcast-qual' ]
|
||||||
|
pzstd_warning_flags = ['-Wno-shadow', '-Wno-deprecated-declarations']
|
||||||
if cc_id == compiler_clang
|
if cc_id == compiler_clang
|
||||||
common_warning_flags += ['-Wconversion', '-Wno-sign-conversion', '-Wdocumentation']
|
common_warning_flags += ['-Wconversion', '-Wno-sign-conversion', '-Wdocumentation']
|
||||||
endif
|
endif
|
||||||
|
Reference in New Issue
Block a user