libgav1_flags.cmake: add a bug link
for check_cxx_compiler_flag() behavior mentioned in cl/332506801 +
update the affected range, this should be addressed in 3.18.3.
PiperOrigin-RevId: 333097345
Change-Id: Ib3a442300ea65947fe0e0dade064e956e48d5e3c
diff --git a/cmake/libgav1_flags.cmake b/cmake/libgav1_flags.cmake
index 9e29413..2d8d9a6 100644
--- a/cmake/libgav1_flags.cmake
+++ b/cmake/libgav1_flags.cmake
@@ -118,10 +118,11 @@
if(cxx_flags)
message("--- Testing flags from $cxx_flags: " "${cxx_flags}")
foreach(cxx_flag ${cxx_flags})
- # Since 3.17.0 check_cxx_compiler_flag() sets a normal variable at
- # parent scope while check_cxx_source_compiles() continues to set an
- # internal cache variable, so we unset both to avoid the failure /
- # success state persisting between checks.
+ # Between 3.17.0 and 3.18.2 check_cxx_compiler_flag() sets a normal
+ # variable at parent scope while check_cxx_source_compiles() continues
+ # to set an internal cache variable, so we unset both to avoid the
+ # failure / success state persisting between checks. See
+ # https://gitlab.kitware.com/cmake/cmake/-/issues/21207.
unset(cxx_flag_test_passed)
unset(cxx_flag_test_passed CACHE)
message("--- Testing flag: ${cxx_flag}")
