Skip to content

Commit

Permalink
Define L_ENDIAN for linux64-loongarch64
Browse files Browse the repository at this point in the history
In commit d7c0fc5 we removed L_ENDIAN
definition for guessed linux64-loongarch64 as it had caused an
inconsistency between configurations with and without explicit
specifying linux64-loongarch64.  Now add it back to the proper location.

Unlike MIPS or RISC-V, LoongArch is always little-endian [1].

By the way, change "LOONGARCH" to "LoongArch" in a comment as LOONGARCH
should only appear in the identifiers of macros, constants, etc.

[1]:https://loongson.github.io/LoongArch-Documentation/LoongArch-Vol1-EN.html#endian

Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #23064)
  • Loading branch information
xry111 authored and t8m committed Dec 18, 2023
1 parent c4496b8 commit e1002c8
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion Configurations/10-main.conf
Original file line number Diff line number Diff line change
Expand Up @@ -821,12 +821,13 @@ my %targets = (
asm_arch => 'riscv32',
},

# loongarch64 below refers to contemporary LOONGARCH Architecture
# loongarch64 below refers to contemporary LoongArch Architecture
# specifications,
"linux64-loongarch64" => {
inherit_from => [ "linux-generic64"],
perlasm_scheme => "linux64",
asm_arch => 'loongarch64',
lib_cppflags => add("-DL_ENDIAN"),
},

#### IA-32 targets...
Expand Down

0 comments on commit e1002c8

Please sign in to comment.