[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 06/25] util: Add CRC16 (CCITT) calculation routines

From: Philippe Mathieu-Daudé
Subject: Re: [PATCH v2 06/25] util: Add CRC16 (CCITT) calculation routines
Date: Tue, 26 Jan 2021 08:44:55 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.6.0

On 1/24/21 10:41 PM, Richard Henderson wrote:
> On 1/24/21 10:24 AM, Philippe Mathieu-Daudé wrote:
>> On 1/24/21 9:07 PM, Richard Henderson wrote:
>>> Doesn't this get put in libutil, where it is only pulled from the archive 
>>> when
>>> needed?  Also, libutil is built once, not per-target.
>> Hmm I just sent a pull request with this change squashed in.
>> Should I cancel it?
> I guess not.
>> My view is we don't install libqemuutil.a anywhere, so why overload
>> building unused objects when some configuration don't need it?
> My view is that util/meson.build should not be overly complicated with
> conditionals.

Well I do see an impact when building on slow hosts.

> If we were building objects per-target, that would be one thing.  If we were
> doing --whole-archive, and including unused code in the executables, that 
> would
> be another thing.  But otherwise...

I understand your position :)



reply via email to

[Prev in Thread] Current Thread [Next in Thread]