<div dir="ltr">I will check your patch this week on a mac / windows and linux , It's still a hack so the proper way to solve it is like described in this stackoverflow solution <div><br><div><a href="http://stackoverflow.com/questions/11228855/header-files-for-x86-simd-intrinsics" target="_blank">http://stackoverflow.com/quest<wbr>ions/11228855/header-files-<wbr>for-x86-simd-intrinsics</a></div><div><br></div><div>I used tips from </div><div><a href="https://bidetly.io/2017/02/08/crc-part-1/" target="_blank">https://bidetly.io/2017/02/08/<wbr>crc-part-1/</a></div><div><br></div><div>and eliminated msse4 flag from compiler command line </div><div><br></div><div><a href="https://gist.github.com/mariuz/753f6fce7ebe0ac9bcf2cb26905c1cc1" target="_blank">https://gist.github.com/<wbr>mariuz/<wbr>753f6fce7ebe0ac9bcf2cb26905c1c<wbr>c1</a><br><div><br></div><div><br></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, May 13, 2017 at 9:31 AM, Julien Nabet <span dir="ltr"><<a href="mailto:serval2412@yahoo.fr" target="_blank">serval2412@yahoo.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <div class="m_208124130050527153moz-cite-prefix">Hi,<br>
      <br>
      As I put on the gerrit patch, I give up with this patch because:<br>
      - it's too complicated to modify the patch : the patching part was
      working on my Mac and it fails to apply on Jenkins Mac<br>
      - sse4 pb.
      Even if it builds, LO may crash on pc which aren't compatible with
      sse4<br>
      <br>
      Julien<br>
      <br>
      On 12/05/2017 14:33, marius adrian popa wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">Also
        in firebird/extern/cloop/src/<wbr>tests/test1/CTest.c
        <div><br>
          <div>related to this patch </div>
          <div><a href="https://gerrit.libreoffice.org/#/c/37488/6/external/firebird/firebird-macosx.patch.1" target="_blank">https://gerrit.libreoffice.<wbr>org/#/c/37488/6/external/<wbr>firebird/firebird-macosx.<wbr>patch.1</a><br>
            <div><br>
            </div>
            <div>
              <div>these lines and changes are still needed for osx <span class="m_208124130050527153gmail-Apple-tab-span" style="white-space:pre-wrap">       </span></div>
              <div> </div>
              <div> #include "CalcCApi.h"</div>
              <div>-#include <malloc.h></div>
              <div>+#include <stdlib.h></div>
              <div> #include <stdio.h></div>
            </div>
            <div><br>
            </div>
            <div>now build fails <a href="https://ci.libreoffice.org/job/lo_gerrit/11317/Config=macosx_clang_dbgutil/" target="_blank">https://ci.libreoffice.org/<wbr>job/lo_gerrit/11317/Config=<wbr>macosx_clang_dbgutil/</a></div>
          </div>
        </div>
      </div>
      <div class="gmail_extra"><br>
        <div class="gmail_quote">On Fri, May 12, 2017 at 1:10 PM, Tomaž
          Vajngerl <span dir="ltr"><<a href="mailto:quikee@gmail.com" target="_blank">quikee@gmail.com</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
            <span><br>
              On Fri, May 12, 2017 at 11:05 AM, marius adrian popa <<a href="mailto:mapopa@gmail.com" target="_blank">mapopa@gmail.com</a>>
              wrote:<br>
              > Known issue in Firebird 3.0.x<br>
              ><br>
              > <a href="http://firebird.1100200.n4.nabble.com/std-c-11-added-to-CXXFLAGS-in-3-0-on-linux-td4645224.html" rel="noreferrer" target="_blank">http://firebird.1100200.n4.nab<wbr>ble.com/std-c-11-added-to-CXXF<wbr>LAGS-in-3-0-on-linux-td4645224<wbr>.html</a><br>
              ><br>
              > introduced by this commit<br>
              ><br>
              > <a href="https://github.com/FirebirdSQL/firebird/commit/52d9a05a0f3d" rel="noreferrer" target="_blank">https://github.com/FirebirdSQL<wbr>/firebird/commit/52d9a05a0f3d</a><br>
              ><br>
              <br>
            </span>They would need to add something like "%/CRC32C.o:
            COMMON_FLAGS +=<br>
            -msse4" to prefix.darwin too (clang should have compatible
            switches)<br>
            <br>
            For Windows It is complicated as it depends if the
            particular VS<br>
            version supports the instructions or not, but you need to
            add some<br>
            compile switch too...<br>
            <br>
            That's only for compiler - in addition you also need
            run-time<br>
            detection, but I see this is already covered by
            SSE4_2Supported()<br>
            method.<br>
            <br>
            Regards, Tomaž<br>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <p><br>
    </p>
  </div>

</blockquote></div><br></div>