Re: [PATCH v5 1/4] tree-object-size: Support dynamic sizes in conditions

[Siddhesh Poyarekar]

On 10/01/2022 16:07, Jakub Jelinek wrote:
You test the above with both possibilities.
+  if (test_builtin_calloc_condphi (128, 1, 0) == 128)
+    FAIL ();

But not this one, why?  Also, it would be better to have
a != ... test rather than ==, if it is the VLA, then 128 * sizeof (struct { int 
a; char b; })
?

I think I'll move the test_builtin_calloc_condphi test into the 
GIMPLE_CALL patch since that's where it becomes fully functional.

Thanks,
Siddhesh