On Mon, 8 Apr 2024, J. Dekker wrote:
The preferred way to use LTO is --enable-lto but often times packagers
still end up with -flto in cflags for various reasons. Using grep
on binary object files is brittle and relies on specific object
representation, which in the case of LLVM bitcode, debug-info or other
intermediary formats can fail silently.
This patch changes the check to a more commonly used define for
big-endian systems.
It's not common only for big-endian systems, but for GCC-style compilers
on all endians.
More checks may need to be added in the future to cover legacy machines.
Don't use the word "legacy" here. This define is not standard, so it's
perfectly plausible to have a modern, standards compliant compiler that
just doesn't use this define.
With the commmit message you added here, the change is ok, but please do
reword the last sentence above.
I'd suggest changing the last paragraph into this:
---
This patch changes the check to a more commonly used define for
GCC style compilers. More checks may be needed to cover other potential
compilers that don't use the __BYTE_ORDER__ define.
---
// Martin
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
