Enable bug workaround also for MSVC 2017.4

Summary:
Workaround a bug in template instantiation in MSVC 2017 U3/4 with /permissive-

https://developercommunity.visualstudio.com/content/problem/81223/incorrect-error-c5037-with-permissive.html

/cc Orvid
Closes https://github.com/facebook/folly/pull/691

Reviewed By: yfeldblum

Differential Revision: D6085652

Pulled By: Orvid

fbshipit-source-id: f068dc3f8e474163815fc497b05410fe76834a52

diff --git a/folly/stats/Histogram.h b/folly/stats/Histogram.h
index 06db3d049d92a7af17c80a8985a80ab9ddfec46d..1f34d0e9a024bc5a61096e7612ee2966d5581393 100644 (file)
--- a/folly/stats/Histogram.h
+++ b/folly/stats/Histogram.h
@@ -480,12 +480,13 @@ class Histogram {
 
 } // namespace folly
 
-// MSVC 2017 Update 3 has an issue with explicitly instantiating templated
+// MSVC 2017 Update 3/4 has an issue with explicitly instantiating templated
 // functions with default arguments inside templated classes when compiled
 // with /permissive- (the default for the CMake build), so we directly include
 // the -defs as if it were -inl, and don't provide the explicit instantiations.
 // https://developercommunity.visualstudio.com/content/problem/81223/incorrect-error-c5037-with-permissive.html
-#if defined(_MSC_VER) && _MSC_FULL_VER >= 191125506 && _MSC_FULL_VER < 191125542
+#if defined(_MSC_VER) && _MSC_FULL_VER >= 191125506 && \
+    _MSC_FULL_VER <= 191125547
 #define FOLLY_MSVC_USE_WORKAROUND_FOR_C5037 1
 #else
 #define FOLLY_MSVC_USE_WORKAROUND_FOR_C5037 0