diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2021-05-08 00:07:51 +0200 |
|---|---|---|
| committer | Arnd Bergmann <arnd@arndb.de> | 2021-05-10 17:50:47 +0200 |
| commit | 0652035a57945e14e611dafae2ec5b46a05bc1d1 (patch) | |
| tree | 0c34544a7755bc1ee0a822e86458d3ed9215533f /arch/arm/include | |
| parent | f12d3ff3f41cc92f67cfaf29697685e8834fe4a4 (diff) | |
| download | linux-stable-0652035a57945e14e611dafae2ec5b46a05bc1d1.tar.gz linux-stable-0652035a57945e14e611dafae2ec5b46a05bc1d1.tar.bz2 linux-stable-0652035a57945e14e611dafae2ec5b46a05bc1d1.zip | |
asm-generic: unaligned: remove byteshift helpers
In theory, compilers should be able to work this out themselves so we
can use a simpler version based on the swab() helpers.
I have verified that this works on all supported compiler versions
(gcc-4.9 and up, clang-10 and up). Looking at the object code produced by
gcc-11, I found that the impact is mostly a change in inlining decisions
that lead to slightly larger code.
In other cases, this version produces explicit byte swaps in place of
separate byte access, or comparing against pre-swapped constants.
While the source code is clearly simpler, I have not seen an indication
of the new version actually producing better code on Arm, so maybe
we want to skip this after all. From what I can tell, gcc recognizes
the byteswap pattern in the byteshift.h header and can turn it into
explicit instructions, but it does not turn a __builtin_bswap32() back
into individual bytes when that would result in better output, e.g.
when storing a byte-reversed constant.
Suggested-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Diffstat (limited to 'arch/arm/include')
| -rw-r--r-- | arch/arm/include/asm/unaligned.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/arch/arm/include/asm/unaligned.h b/arch/arm/include/asm/unaligned.h index ab905ffcf193..3c5248fb4cdc 100644 --- a/arch/arm/include/asm/unaligned.h +++ b/arch/arm/include/asm/unaligned.h @@ -10,13 +10,11 @@ #if defined(__LITTLE_ENDIAN) # include <linux/unaligned/le_struct.h> -# include <linux/unaligned/be_byteshift.h> # include <linux/unaligned/generic.h> # define get_unaligned __get_unaligned_le # define put_unaligned __put_unaligned_le #elif defined(__BIG_ENDIAN) # include <linux/unaligned/be_struct.h> -# include <linux/unaligned/le_byteshift.h> # include <linux/unaligned/generic.h> # define get_unaligned __get_unaligned_be # define put_unaligned __put_unaligned_be |