[PATCH][SH] Check for 0 length with inlined strnlen builtin
Oleg Endo
oleg.endo@t-online.de
Tue Jan 6 09:41:00 GMT 2015
On Tue, 2015-01-06 at 10:28 +0100, Christian Bruel wrote:
> Hello,
>
> We should not enter the first iteration when length is 0. Testcase
> attached. Difficult to reduce because register allocation generated
> accidentally the correct return value.
>
> testsuite OK
>
> OK for 4.9 and trunk ?
In your patch:
+ emit_insn (gen_tstsi_t (len, len));
Please use 'gen_cmpeqsi_t (len, const0_rtx)' for comparing a value
against zero instead of the bit test insn. Otherwise, I think it's OK
for trunk and 4.9.
Cheers,
Oleg
More information about the Gcc-patches
mailing list