Re: [PATCH v2 7/8] minmax: Use __auto_type

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

To: David Laight <David.Laight@xxxxxxxxxx>
Subject: Re: [PATCH v2 7/8] minmax: Use __auto_type
From: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxx>
Date: Sun, 28 Jul 2024 09:59:12 -0700
Cc: "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, Jens Axboe <axboe@xxxxxxxxx>, "Matthew Wilcox (Oracle)" <willy@xxxxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>, Dan Carpenter <dan.carpenter@xxxxxxxxxx>, Arnd Bergmann <arnd@xxxxxxxxxx>, "Jason@xxxxxxxxx" <Jason@xxxxxxxxx>, "pedro.falcato@xxxxxxxxx" <pedro.falcato@xxxxxxxxx>, Mateusz Guzik <mjguzik@xxxxxxxxx>, "linux-mm@xxxxxxxxx" <linux-mm@xxxxxxxxx>, Lorenzo Stoakes <lorenzo.stoakes@xxxxxxxxxx>
In-reply-to: <431f7c45e7294d4da4f8abcd57ce7b5e@AcuMS.aculab.com>
References: <402c3c617c29465c898b1af55e3c6095@AcuMS.aculab.com> <431f7c45e7294d4da4f8abcd57ce7b5e@AcuMS.aculab.com>

On Sun, 28 Jul 2024 at 07:24, David Laight <David.Laight@xxxxxxxxxx> wrote:
>
> Replacing 'typeof(x) _x = (x)' with '__auto_type _x = (x)' removes
> one expansion of 'x'.

Ack. We should do this more widely, but the whole "typeof()" predates
__auto_type by many years, and we (and by that I mean "I" - the royal
we) have just years of historical mental baggage.

              Linus

References:
- [PATCH v2 0/8] minmax: reduce compilation time
  - From: David Laight
- [PATCH v2 7/8] minmax: Use __auto_type
  - From: David Laight

Prev by Date: Re: [PATCH v2 4/8] minmax: Simplify signedness check
Next by Date: Re: [PATCH v2 1/8] minmax: Put all the clamp() definitions together
Previous by thread: [PATCH v2 7/8] minmax: Use __auto_type
Next by thread: [PATCH v2 8/8] minmax: minmax: Add __types_ok3() and optimise defines with 3 arguments
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]