This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Bug in expand_builtin_sprintf
Kaveh R. GHAZI wrote:
> On Mon, 8 Oct 2007, Heikki Linnakangas wrote:
>
>> I think there's a small bug in expand_builtin_sprintf in trunk,
>> introduced by commit 122018. Fix below:
>>
>> Index: builtins.c
>> ===================================================================
>> --- builtins.c (revision 129124)
>> +++ builtins.c (working copy)
>> @@ -5449,7 +5449,7 @@
>> dest = CALL_EXPR_ARG (exp, 0);
>> if (! POINTER_TYPE_P (TREE_TYPE (dest)))
>> return NULL_RTX;
>> - fmt = CALL_EXPR_ARG (exp, 0);
>> + fmt = CALL_EXPR_ARG (exp, 1);
>> if (! POINTER_TYPE_P (TREE_TYPE (fmt)))
>> return NULL_RTX;
>>
>
> Testcase please?
I don't have one, unfortunately. I spotted this by looking at the code.
I believe the only consequence is that the simplifications are never
done in expand_builtin_sprintf.
--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com