<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:SimSun;
        panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:"\@SimSun";
        panose-1:2 1 6 0 3 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
span.EmailStyle18
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:#1F497D;
        font-weight:normal;
        font-style:normal;
        text-decoration:none none;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-family:"Calibri",sans-serif;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="color:#1F497D">Hi Raghuveer,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D">Do you mean the result is incorrect or just fp overflow flag? I think Clang doesn’t promise the fp status always be correct by default. But you can use -ffp-model=strict or #pragma float_control(except, on) to
 force it to do so.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D">Thanks<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D">Pengfei<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><b>From:</b> llvm-dev <llvm-dev-bounces@lists.llvm.org> <b>On Behalf Of
</b>Raghuveer Devulapalli via llvm-dev<br>
<b>Sent:</b> Sunday, December 20, 2020 9:56 AM<br>
<b>To:</b> llvm-dev@lists.llvm.org<br>
<b>Subject:</b> [llvm-dev] bug in clang?<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">Hi all, <o:p></o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Here is a snippet of code taken from NumPy and the corresponding output when using clang 10.0 (or even clang 12.0). Can also view the code here: <a href="https://godbolt.org/z/Peznjx">https://godbolt.org/z/Peznjx</a><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">The compiler optimizes and removes the first blendv_ps instruction (i.e eliminates the temp1 variable all together). The goal of this instruction was to prevent an overflow flag generated in the following multiplication instruction. By
 eliminating this instruction, the resulting code causes an unintended fp overflow flag which I think is incorrect behavior.  <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">C code: <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">__m256 fma_get_mantissa(__m256 x)<br>
{<br>
    __m256 two_power_100 = _mm256_castsi256_ps(_mm256_set1_epi32(0x71800000));<br>
    __m256 denormal_mask = _mm256_cmp_ps(x, _mm256_set1_ps(FLT_MIN), _CMP_LT_OQ);<br>
    __m256 normal_mask = _mm256_cmp_ps(x, _mm256_set1_ps(FLT_MIN), _CMP_GE_OQ);<br>
<br>
    __m256 temp1 = _mm256_blendv_ps(x, _mm256_set1_ps(0.0f), normal_mask);<br>
    __m256 temp = _mm256_mul_ps(temp1, two_power_100);<br>
    x = _mm256_blendv_ps(x, temp, denormal_mask);<br>
<br>
    __m256i mantissa_bits = _mm256_set1_epi32(0x7fffff);<br>
    __m256i exp_126_bits  = _mm256_set1_epi32(126 << 23);<br>
    return _mm256_castsi256_ps(<br>
                _mm256_or_si256(<br>
                    _mm256_and_si256(<br>
                        _mm256_castps_si256(x), mantissa_bits), exp_126_bits));<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">} <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">object code output from clang: <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">.LCPI0_0:<br>
        .long   0x00800000                      # float 1.17549435E-38<br>
.LCPI0_1:<br>
        .long   0x71800000                      # float 1.2676506E+30<br>
.LCPI0_2:<br>
        .long   4294967170                      # 0xffffff82<br>
.LCPI0_3:<br>
        .long   0xc2c80000                      # float -100<br>
fma_get_exponent:                       # @fma_get_exponent<br>
        vbroadcastss    ymm1, dword ptr [rip + .LCPI0_0] # ymm1 = [1.17549435E-38,1.17549435E-38,1.17549435E-38,1.17549435E-38,1.17549435E-38,1.17549435E-38,1.17549435E-38,1.17549435E-38]<br>
        vcmpltps        ymm2, ymm0, ymm1<br>
        vbroadcastss    ymm3, dword ptr [rip + .LCPI0_1] # ymm3 = [1.2676506E+30,1.2676506E+30,1.2676506E+30,1.2676506E+30,1.2676506E+30,1.2676506E+30,1.2676506E+30,1.2676506E+30]<br>
        <b>vmulps  ymm3, ymm0, ymm3</b><br>
        vcmpnleps       ymm1, ymm1, ymm0<br>
        vandps  ymm1, ymm1, ymm3<br>
        vblendvps       ymm0, ymm0, ymm1, ymm2<br>
        vpsrld  ymm0, ymm0, 23<br>
        vpbroadcastd    ymm1, dword ptr [rip + .LCPI0_2] # ymm1 = [4294967170,4294967170,4294967170,4294967170,4294967170,4294967170,4294967170,4294967170]<br>
        vpaddd  ymm0, ymm0, ymm1<br>
        vcvtdq2ps       ymm0, ymm0<br>
        vbroadcastss    ymm1, dword ptr [rip + .LCPI0_3] # ymm1 = [-1.0E+2,-1.0E+2,-1.0E+2,-1.0E+2,-1.0E+2,-1.0E+2,-1.0E+2,-1.0E+2]<br>
        vaddps  ymm1, ymm0, ymm1<br>
        vblendvps       ymm0, ymm0, ymm1, ymm2<br>
        ret<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Raghuveer<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</div>
</body>
</html>