https://bugs.winehq.org/show_bug.cgi?id=38713
--- Comment #14 from Sebastian Lackner sebastian@fds-team.de --- (In reply to Ken Thomases from comment #13)
It's true that it's (currently) only used for 64-bit, but I'm guessing it's not the first such architecture-specific dependency. I'm not sure it's worth making the check architecture-specific if there's any chance that it will be used for other architectures in the future (which I don't really know if it will).
That might be true. But at least the configure flag should be added in my opinion, there are a lot of distros (Gentoo for example) which hate auto-magic linking against additional dependencies.
Your patch is better than mine in that I forgot to #define UNW_LOCAL_ONLY for my check.
The loop construct in mine avoids the duplication of the test code in yours.
I agree, feel free to create a merged version with the advantages from both attempts.