<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Issue</th>
<td>
<a href=https://github.com/llvm/llvm-project/issues/58400>58400</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>
Incorrect code generation when we use __builtin_constant_p built-in function
</td>
</tr>
<tr>
<th>Labels</th>
<td>
new issue
</td>
</tr>
<tr>
<th>Assignees</th>
<td>
</td>
</tr>
<tr>
<th>Reporter</th>
<td>
alex-curiou
</td>
</tr>
</table>
<pre>
Incorrect codegeneration from version 8.0.0:
- with -O2 flag the return NOT CORRECT value 13 (https://godbolt.org/z/xs9EzP4Mo);
- with -Og flag the return CORRECT value 60 (https://godbolt.org/z/v8E3xGvT6);
>From version 13.0.0 and till 15.0.0 INCORRECT code generation for both -Og and -O2,
the return value is 13, but the bar func code is generated to calculate 60:
- with -O2 flag: https://godbolt.org/z/nzn5zT65e
- with -Og flag: https://godbolt.org/z/4osxW8fnz
The minimized example of code:
~~~
#define test(val1) \
__builtin_constant_p(val1) ? (val1 + 10) : (val1 * 20);
unsigned int bar(unsigned int value1) {
return test(value1);
}
int main(void) {
unsigned int a = bar(3);
return a;
}
~~~
The __builtin_constant_p considers the val1 value to be NOT known to be constant at compile time. So the generated code is
val1 * 20
But optimizer considers that the value is WELL known and can WELL be calculated.
So, the return value of func bar is calculated at compile time and the call to the bar func is dropped away.
And the final return value is WRONG:
val1 + 10
As a result we have different return values with -O2 and -Og optimization flags, 13 and 60 accordingly.
For example, here is the compiler output for version 8.0.0:
-O2 will return 13
https://godbolt.org/z/xs9EzP4Mo
~~~
bar: # @bar
shl edi, 2
lea eax, [rdi + 4*rdi]
ret
main: # @main
mov eax, 13
ret
~~~
-Og will return 60
https://godbolt.org/z/v8E3xGvT6
~~~
bar: # @bar
shl edi, 2
lea eax, [rdi + 4*rdi]
ret
main: # @main
mov edi, 3
jmp bar
~~~
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJzlVltvozoQ_jXkZdSIcMnlgYe0TVcr7WmPupH6WBkwwXscG9kmyebXn7FNEnKpNu-LEsA2nvnmm_kGcln-zr6LQipFCwOFLOmKCqqIYVJApeQaNlRpO5gOw2EYxPMgfA7COTzAlpkaHt4iqDhZgakpKGpaJeD1bQlPb-_vi6clbAhvKYxiCKJpbUyjrYXoBX8rWeaSm6FUKxzt8b_Ts8X-3-QfGUSzIH68dLS6cnTuZBze42QzXcS7b5vluO_En1_64Y5iGy8QUYJhnMModePvrwenlivokyUV5LJDarchNUH05E33QHuwTKMHXIa8NS6knCioWlF4u7jcmaboX0JBeNFyHGGUX-YAF-BP4Yu9SPfLcUpvs3uPiUTq3ce0Evs-d_68xEDWTLA12yNuuiPrhlOQlQvqiLs7Txbdzw-juKQVExQM1QYTiTyNMEcQpB2HAJ-fecu4YeKzkEIbIsxn038yfoFuiNdHGIV-dt6bnUMUXmfen1uh2UogcCaMzQfuOptymfOeJofqxKPL6wm2f-jkYvLc92ItrQkT9lnJyitzZy4Jwn_usMTXuHvuyQ1_FwwfU3SLR7C3rMTqd-Xo6PKliuWXUyfq_4Tcim582AnEto11wzDRhq3pEH5KZ-FUv11Fe__9PPRgPaIMZGNc5agzLMQcAHnZfCx-_OiQWJUVRPgpi-mgknLorf6UVmJX4sOCdFKzmkOLp22XwXj5184yt4GfKRW3lko2jd23Jb-H_Xjm3UasaMKvpP_x_vb67aiHfsH2TWhMv6K65Qa2FGqyoVCyqqKKIu19k_rUB3zjWR247FoTKltbJrAP2wewU5ICe37JxIqf437BLtbp1m6o0ZkF7DjwxCiQrWkwXbbh3X45-LOFs7Wts4OK_c7N3_8a-LpbWEXErvz_dGBfgSAJ7YajZOyha-6utGQ20Oh8lVPiV8nOrgbpI5LlcpRg7eJ9kD6f78Ag_YTT9l3YOmhuw5mttdz0vR-Iu3J1U-A2-33ax-F9tJ9ejH897R7bBeu_1o27HkM6cjOg2Wg8TmezKBxFgzKLy1k8IwPDDKcXH1f9D4ZtTYUVdqu_6Mhu6oEJ123slkGreHaRRxR-mw9RmzjgfHO4PDRK_kKvOGRaY4vAm3SahOGgzug0rSZJnlYEYUZRleRxMcGjSgtSzapowElOuc6Q_iCKBN2CM4H3SP-AZVEYRaNwNAkncRxHQ0qLUTijeZJPJxWhCXJLkVw-tDhsgQ1U5iDlLXahJORMG31aJNq976hzh_ZJa2qpMsLp7qFoFZPtwLnPHPz_AdR_G4U">