corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1935812 - Compile libjxl as C++17. r=tnikkel,saschanaz?

Browse Source 
We've been compiling libjxl as C++11 to work around a C++17 bug in clang 5: https://gitlab.com/wg1/jpeg-xl/-/issues/227

Now that our base toolchain's minimum clang version is 8, there is no need to force C++11.

Differential Revision: https://phabricator.services.mozilla.com/D231470
This commit is contained in:
Chris Peterson
2024-12-07 23:44:31 +00:00
parent 79ea57a98d
commit 776d17a149
1 changed files with 0 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
media/libjxl/moz.build
View File

@@ -118,9 +118,3 @@ FINAL_LIBRARY = "xul"
# We allow warnings for third-party code that can be updated from upstream.
AllowCompilerWarnings()
# Clang 5.0 has a compiler bug that prevents build in c++17
# See https://gitlab.com/wg1/jpeg-xl/-/issues/227
# This should be okay since we are using the C API.
if CONFIG["CC_TYPE"] == "clang":
CXXFLAGS += ["-std=c++11"]