diff options
| author | Sara Golemon <sara.golemon@mongodb.com> | 2017-10-06 12:58:06 -0400 |
|---|---|---|
| committer | Sara Golemon <sara.golemon@mongodb.com> | 2017-11-22 13:50:30 -0500 |
| commit | cd3e3011158cab7dbf5d2fed86641c8f0b8b88fb (patch) | |
| tree | 4ef92efeaa2c239750d1f746a89696b759f37316 /SConstruct | |
| parent | b38a035145e98512f9cd3e2d69e7bef3893cb02b (diff) | |
| download | mongo-cd3e3011158cab7dbf5d2fed86641c8f0b8b88fb.tar.gz | |
SERVER-31498 Make build more clang5 friendly
* Remove unused lambda capture variables
* Resolve signed/unsigned comparisons
* Apply upstream MozJS bugfix 1390214 [1]
1 - https://github.com/mozilla/gecko-dev/commit/a6da168bd0cd41d4b5dd2a0560490ec2ae33d5f7#diff-56d2d8dc33027ea7d747c116640db8ac
Diffstat (limited to 'SConstruct')
| -rw-r--r-- | SConstruct | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/SConstruct b/SConstruct index f949f654756..ebc9ea96faf 100644 --- a/SConstruct +++ b/SConstruct @@ -2025,13 +2025,6 @@ def doConfigure(myenv): # it isn't required AddToCXXFLAGSIfSupported(myenv, "-Wno-instantiation-after-specialization") - # This warning was added in clang-5 and flags many of our lambdas. Since it isn't actively - # harmful to capture unused variables we are suppressing for now with a plan to fix later. - # Additionally, this has some false-positives where removing the capture makes the code - # incorrect and fail to compile on other compilers. - # See https://bugs.llvm.org/show_bug.cgi?id=34865. - AddToCCFLAGSIfSupported(myenv, "-Wno-unused-lambda-capture") - # This warning was added in clang-5 and incorrectly flags our implementation of # exceptionToStatus(). See https://bugs.llvm.org/show_bug.cgi?id=34804 AddToCCFLAGSIfSupported(myenv, "-Wno-exceptions") |