[Mesa-dev] [PATCH] mesa: Move register_allocate.c to util.

Ian Romanick idr at freedesktop.org
Wed Sep 24 15:51:08 PDT 2014


On 09/24/2014 12:52 PM, Eric Anholt wrote:
> Roland Scheidegger <sroland at vmware.com> writes:
> 
>> Oh yes and missing ALIGN + MAX2 too. I guess we could easily move these
>> to util code. That plus the things I already mentioned should be all
>> needed I think. But I strongly believe either this needs to be done or
>> we should revert it.
> 
> What's the actual problem being caused, though, other than needing to
> add an include dir?  I mean, the previous state was that drivers were
> compiling a second copy of the file themselves inside of gallium, and
> being careful to make all our link targets link it exactly once, which
> seems definitely worse than "oh, yeah, something outside of mesa/
> includes some mesa/ static inlines and #define helpers".
> 
> If someone was really offended by this, you could move bitset to util/,
> and evict ffs code to the single caller in nouveau (which is probably a
> good idea, because using ffs in your loop is probably a terrible idea
> anyway).  But I think we all have more important things to be doing, and
> I don't think there's any actual harm being done.

Yeah... these sound like great janitor projects for new community
developers... not so much for people with deadlines and deliverables.

> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140924/b946faab/attachment.sig>


More information about the mesa-dev mailing list