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

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

 



On Mon. 5 Feb. 2024 at 18:48, Finn Thain <fthain@xxxxxxxxxxxxxx> wrote:
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.

I am deeply sorry, but with all your respect, this request is unfair.
I will not fix the compiler.

Let me repeat my question for the third time: are you (or any other
m68k maintainer) ready to acknowledge that we can degrade the
performance for clang m68k users? With that acknowledgement, I will
remove the asm and replace it with the builtins.


Yours sincerely,
Vincent Mailhol




[Index of Archives]     [Video for Linux]     [Yosemite News]     [Linux S/390]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux