[PATCH] autoconf: add missing main() for __builtin_alloca check

[Niels Möller]

Marc Glisse <marc.glisse at inria.fr> writes:

> 1) the test is not super relevant nowadays (written for HP's itanium
> compiler) and never worked properly (?), so we could remove it.

I'm all for deleting long-obsolete cruft.

> 2) we can keep it as a sanity check (we have a few of those), but then
> I think the proper fix is adding int main(void){return 0;} to the test
> explicitly.

In general, I think configure tests are more useful when we think it is
also worth the effort to work around detected problems. "Sanity check"
kind of tests make sense to me when they detect problems that are
reasonably likely to happen to some users, and where we can provide an
error message at configure time that is more helpful than a later
compile error or failed make check.

Regards,
/Niels

-- 
Niels Möller. PGP key CB4962D070D77D7FCB8BA36271D8F1FF368C6677.
Internet email is subject to wholesale government surveillance.