Charles Davis <cdavi...@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. -- Alexandre Julliard julli...@winehq.org
