Charles Davis cdavis5x@gmail.com writes:
Try 2: Clarify that this is a Mac OS-only change. Don't use $CC as the assembler. Use the right flags to select 32/64-bit assembly with Clang.
The configure script invokes the system compiler to figure out if .cfi pseudo-ops are supported, on the assumption that it will invoke the system assembler. Winebuild, on the other hand, invokes the system assembler directly. This caused a problem on Mac OS hosts when compiling with Clang: configure would detect that .cfi ops were supported, because Clang's integrated assembler supports them; however, when winebuild then emitted assembly with .cfi ops in them, the system assembler would then choke. This patch makes winebuild invoke Clang's assembler if a) we are on Mac OS, b) $CC is detected to be Clang, and c) it is using (whether by request or by default) its built-in assembler.
You can't use ifdefs in winebuild. This would have to be a runtime check.