alloc_size attribute removed from clang

[kikofernandez]

After installing the dependencies (doxygen, clang, ghc and premake4), running make throws an error.

The reason is that clang-600.0.51 (based on LLVM 3.5) does not support alloc_size anymore.
More info:
http://reviews.llvm.org/rL197866
neovim/neovim#429

Should we fix this issue and allow developers to go with the latest clang version? Otherwise, they are restricted to clang-503.0.40

[TobiasWrigstad]

I recently saw this as well. I have locally commented out
these attributes, but I don't know whether it makes sense
to solve it in any other way (like go with an earlier
clang).

Does anyone have a strong suggestion?

[EliasC]

Could we ask Sylwan if he considers it important? Reading @kikofernandez links seems to suggest that it does not do anything anyway.

[EliasC]

...ah, but of course it might actually have an effect if you use gcc.

[TobiasWrigstad]

@kikofernandez Can you fix up a macro solution for this so that clang compilers don't get to see any warnings, but that gcc compilers do get to use the feature and close this issue?

[kikofernandez]

no problem