Page MenuHomePhabricator

eina: move builtin detection to eina_config.h
Needs ReviewPublic

Authored by bu5hm4n on Apr 4 2019, 1:24 AM.

Details

Reviewers
vtorri
zmike
Summary

this means we don't need to handle this in the buildtool at all.

Diff Detail

Repository
rEFL core/efl
Branch
master
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 10749
bu5hm4n created this revision.Apr 4 2019, 1:24 AM
bu5hm4n requested review of this revision.Apr 4 2019, 1:24 AM
vtorri added a comment.Apr 4 2019, 2:41 AM

S you consider that, if byteswap.h exists, then the functions bswap_** exist ?

That is what we use in eina_cpu_include.x yes.

zmike added inline comments.Apr 17 2019, 5:22 AM
src/lib/eina/eina_config.h.in
31

Are there platforms which would reach this point?

There was a complain on the ML from the last two days that this is only supported by gcc > 5 so it looks like there are systems that are going there.

zmike requested changes to this revision.Apr 19 2019, 6:00 AM

I'd guess that means LTS distros or something. I don't think we can make this switch in full now. Leave the meson.build parts as the #else block for now and it can be removed later.

This revision now requires changes to proceed.Apr 19 2019, 6:00 AM

Mhm, that would not work, it appears that I did a mistake when I wrote the meson part. I checked for the bswap function but I should have checked for the builtin. However if I do check it with the builtin check, then meson will use the has_builtin() so then both values would depend on that function. I even think that meson could not handle anything without available has_attribute ...

bu5hm4n requested review of this revision.Apr 20 2019, 6:05 AM