Re: AtomicCounter::is_always_lock_free on armel

[Stephan Bergmann]

On 06/11/2019 18:39, Rene Engelhard wrote:
Given the introduced AtomicCounter is used later, too I tried the simplified
diff --git a/vcl/inc/opengl/zone.hxx b/vcl/inc/opengl/zone.hxx
index 3210186c3096..13ac3bf6793e 100644
--- a/vcl/inc/opengl/zone.hxx
+++ b/vcl/inc/opengl/zone.hxx
@@ -36,7 +36,9 @@ class VCL_DLLPUBLIC OpenGLZone {
      // increasing, so will eventually overflow, so the underlying type better 
be unsigned, which
      // sig_atomic_t is not guaranteed to be:
      using AtomicCounter = 
std::atomic<std::make_unsigned_t<std::sig_atomic_t>>;
+#if !defined ARM32 && !defined __ARM_PCS_VFP
      static_assert(AtomicCounter::is_always_lock_free);
+#endif
  
      /// how many times have we entered a GL zone
      static AtomicCounter gnEnterCount;

(atking that define from bridges...)

and that builds...

-> https://gerrit.libreoffice.org/#/c/82165/

I don't understand your "Given the introduced AtomicCounter is used 
later..." reasoning above, but commented at 
<https://gerrit.libreoffice.org/#/c/82165/> "disable static_assert on 
AtomicCounter::is_always_lock_free on armel ..." now.