<div dir="ltr">On 20 August 2013 13:08, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Commit 6fcbe52 "cmake: Refactor compiler flags into new module" exposed a bug<br>
in the test for thread-local storage, causing the test to always fail. If the<br>
regex "warning|error" appears in the compiler log, then the test fails. This<br>
causes false failures when the test invokes the compiler with<br>
'-Werror=missing-prototypes'.<br>
<br>
Fix the regex to more closely match gcc and clang error messages.<br>
<br>
Reported-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br>
Signed-off-by: Chad Versace <<a href="mailto:chad.versace@linux.intel.com">chad.versace@linux.intel.com</a>><br></blockquote><div><br></div><div>Confirmed that with this patch, my build uses TLS again.<br><br></div>
<div>Tested-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

---<br>
 cmake/Modules/WaffleCheckThreadLocalStorage.cmake | 4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/cmake/Modules/WaffleCheckThreadLocalStorage.cmake b/cmake/Modules/WaffleCheckThreadLocalStorage.cmake<br>
index c8bc3fe..f988f6f 100644<br>
--- a/cmake/Modules/WaffleCheckThreadLocalStorage.cmake<br>
+++ b/cmake/Modules/WaffleCheckThreadLocalStorage.cmake<br>
@@ -61,7 +61,7 @@ macro(waffle_check_thread_local_storage)<br>
<br>
         waffle_has_tls<br>
<br>
-        FAIL_REGEX "warning|error"<br>
+        FAIL_REGEX "warning: |error: "<br>
         )<br>
<br>
     check_c_source_compiles(<br>
@@ -77,6 +77,6 @@ macro(waffle_check_thread_local_storage)<br>
<br>
         waffle_has_tls_model_initial_exec<br>
<br>
-        FAIL_REGEX "warning|error"<br>
+        FAIL_REGEX "warning: |error: "<br>
         )<br>
 endmacro()<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.8.3.1<br>
<br>
</font></span></blockquote></div><br></div></div>