diff options
author | Oleg Oshmyan <chortos@inbox.lv> | 2021-09-18 23:36:17 +0300 |
---|---|---|
committer | Oleg Oshmyan <chortos@inbox.lv> | 2022-02-12 19:40:24 +0200 |
commit | fe296cb5930d12b323237089bd2da42de1790ae0 (patch) | |
tree | 44e2dc6582a75c33637b85a55c8d84884004391b /libass/Makefile.am | |
parent | c809820d6d24192e61f53702c9e13ed6b7f13a48 (diff) | |
download | libass-fe296cb5930d12b323237089bd2da42de1790ae0.tar.bz2 libass-fe296cb5930d12b323237089bd2da42de1790ae0.tar.xz |
build: use `X86` as the generic name for whole x86 family
Currently, we use `INTEL` to denote any supported x86 variant
and `X64` to denote specifically x86-64 (and previously used
`X86` to denote 32-bit x86).
`INTEL` is a subpar name for two reasons:
* Intel has produced notable non-x86 architectures (Itanium).
* x86-64 was designed and patented not by Intel but by AMD.
This is currently only used in configure.ac and Makefile.am,
so it does not matter much. However, the next commit will expose
a config.h macro flagging the whole x86 family, which non-Autotools
users will have to set manually, so this will cease being a purely
internal choice and will start confusing users.
Instead, the most generic name for the whole family that is in common
use and does not suffer from such problems is "x86", so use `X86`.
The switch is slightly complicated by the fact our configure.ac and
Makefile.am used to use `X86` as in Microsoft's naming scheme, where "x86"
means "32-bit x86". Ideally, we would use another equally good name for
the family that is free from this ambiguity; however, I know no such name.
(Indeed, `X86` as a name for 32-bit x86 is problematic even if one
disregards x86-64, because 16-bit x86 exists as well. This commit
technically leaves configure.ac to lie about 16-bit x86 by setting
`X86=false`, but we have no 16-bit assembler code, so the new use of
`X86` should be viewed as a name for "any *supported* variant of x86".)
To somewhat highlight that we're no longer using Microsoft's scheme,
also switch the name for the x86-64 architecture from `X64` to `X86_64`.
This should also make it more intuitively obvious that one (X86_64) is
a subset of the other (X86). This is also a more standard name with a
clearer etymology, and we already use this name in our `ASFLAGS`.
Diffstat (limited to 'libass/Makefile.am')
-rw-r--r-- | libass/Makefile.am | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/libass/Makefile.am b/libass/Makefile.am index 207fe27..a088130 100644 --- a/libass/Makefile.am +++ b/libass/Makefile.am @@ -14,8 +14,8 @@ nasm_verbose_0 = @echo " NASM " $@; .asm.lo: $(nasm_verbose)$(LIBTOOL) $(AM_V_lt) --tag=CC --mode=compile $(top_srcdir)/ltnasm.sh $(AS) $(ASFLAGS) -I$(srcdir)/ -Dprivate_prefix=ass -o $@ $< -SRC_INTEL = x86/rasterizer.asm x86/blend_bitmaps.asm x86/be_blur.asm x86/blur.asm x86/cpuid.asm \ - x86/cpuid.h +SRC_X86 = x86/rasterizer.asm x86/blend_bitmaps.asm x86/be_blur.asm x86/blur.asm x86/cpuid.asm \ + x86/cpuid.h SRC_FONTCONFIG = ass_fontconfig.c ass_fontconfig.h SRC_DIRECTWRITE = ass_directwrite.c ass_directwrite.h ass_directwrite_info_template.h dwrite_c.h @@ -48,8 +48,8 @@ libass_la_SOURCES += $(SRC_CORETEXT) endif if ASM -if INTEL -libass_la_SOURCES += $(SRC_INTEL) +if X86 +libass_la_SOURCES += $(SRC_X86) endif endif |