[cfe-commits] [PATCH] Simplify isPowerOfTwo signature
Duncan Sands
baldrick at free.fr
Tue Dec 11 07:12:32 PST 2012
On 11/12/12 16:04, Rafael EspĂndola wrote:
> On 11 December 2012 02:06, Duncan Sands <baldrick at free.fr> wrote:
>> Hi Saleem, why do you want this? While isPowerOfTwo doesn't use data layout
>> today, it's not unreasonable to imagine that it may use it one day.
>
> We can always add it back that day, no?
Sure, but if there's no real need to remove it why not leave it alone?
It seems a rather pointless clean up to me. That said, I won't object if
Saleem goes ahead and removes it.
Ciao, Duncan.
>
>> As
>> such,
>> requiring target data can be considered part of the isPowerOfTwo API.
>
> This is the c++ api, so it is not required to be stable.
>
>> Ciao, Duncan.
>>
>
> Cheers,
> Rafael
>
More information about the cfe-commits
mailing list