Re: [PATCH v4 2/5] m68k/bitops: use __builtin_{clz,ctzl,ffs} to evaluate constant expressions

From: Finn Thain
Date: Mon Feb 05 2024 - 04:48:57 EST



On Mon, 5 Feb 2024, Vincent MAILHOL wrote:

>
> This is why I am asking whether or not clang support is important or not
> for m68k. If you tell me it is not, then fine, I will remove all the asm
> (by the way, the patch is already ready). But if there are even a few
> users who care about clang for m68k, then I do not think we should
> penalize them and I would not sign-off a change which negatively impacts
> some users.
>

If clang support is important then clang's builtins are important. So why
not improve those instead? That would resolve the issue in a win-win.