<div dir="auto">You can sign up for an account by emailing the admin account. </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 26, 2019, 5:04 PM Anton Yudintsev via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">AntonYudintsev added a comment.<br>
<br>
I have found bug in clang-cl (win32 clang), related to recent inroduction of ffp-exception-behavior.<br>
Unfortunately, I don't have a working patch yet, and since LLVM bugtracker registration is closed, I can not even submit a bug.<br>
<br>
So, if it is not a trouble for you, I will email the bug description here.<br>
<br>
Please let me know if it isn't appropriate. Bug description:<br>
============================================================<br>
<br>
Windows: clang-cl is generating call to non-existing lib function for win32 with /fp:except option.<br>
With recent ffp-exception-behavior=maytrap/strict, fp:except in clang-cl became generate FPE aware code.<br>
<br>
But in case of floorf and ceilf it generates call to non-existing library function.<br>
<br>
clang-cl.exe -m32 /Ox /fp:except testFloor.cpp /FA<br>
testFloor.cpp:<br>
<br>
  #include <math.h><br>
  float ret(float v) {  return floorf(v); }<br>
<br>
resulting assember:<br>
<br>
  push    eax<br>
  movss    xmm0, dword ptr [esp + 8]<br>
  movss    dword ptr [esp], xmm0<br>
  call    _floorf #no such function!!!<br>
  pop    eax<br>
  ret<br>
<br>
Expected behaviour:<br>
<br>
there is no floorf lib function. Like with cosf and other math functions, floorf in MSVC is implemented as inline function.<br>
<br>
So, it should be call to _floor (with apropriate conversion first).<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D62731/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D62731/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D62731" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D62731</a><br>
<br>
<br>
<br>
</blockquote></div>