[Scummvm-git-logs] scummvm master -> b85fa35036254c2c5a8e277bace570061d0a93f8

[dwatteau]

This automated email contains information about 1 new commit which have been
pushed to the 'scummvm' repo located at https://github.com/scummvm/scummvm .

Summary:
b85fa35036 CONFIGURE: MACOSX: Use -fno-var-tracking-assignments for OSXPPC


Commit: b85fa35036254c2c5a8e277bace570061d0a93f8
    https://github.com/scummvm/scummvm/commit/b85fa35036254c2c5a8e277bace570061d0a93f8
Author: Donovan Watteau (contrib at dwatteau.fr)
Date: 2025-02-28T23:07:28+01:00

Commit Message:
CONFIGURE: MACOSX: Use -fno-var-tracking-assignments for OSXPPC

On OSXPPC, a newer toolchain is required for C++11 support.
This, in turn, requires an updated GDB for the newer symbols.

OSX Tiger can't use anything newer than DWARF v2, it seems, so
that's why we're already using -gdwarf-2 (older 'stabs' formats
weren't reliable with these toolchain and GDB releases).

I've been hitting internal GDB errors when analyzing a backtrace
in the TWP engine, until I dropped the -fvar-tracking-assignments
flag we enable by default.  The GCC manual page says that "Use of
-gdwarf-4 is recommended" along with this option, so that's
probably why I was having issues.

(Debug builds on OSXPPC already have an implicit '-O0' requirement,
anyway, as so the GDB release's so old and clunky that it's unwise
to hope anything about '-O2 -g', so -fvar-tracking-assignments
wasn't that useful there anyway.)

Changed paths:
    configure


diff --git a/configure b/configure
index 12f6ccd9d68..1e353eb2494 100755
--- a/configure
+++ b/configure
@@ -3146,8 +3146,10 @@ EOF
 					# newer GDB, but we need it for the updated toolchain
 					# anyway. This improves the debugging experience quite
 					# a lot, and it still creates meaningful traces for the
-					# Crash Reporter of OSX 10.4.
-					append_var CXXFLAGS "-gdwarf-2"
+					# Crash Reporter of OSX 10.4. -fvar-tracking-assignments
+					# should be disabled then, as it seems to cause issues
+					# and we can't use the later DWARF version it recommends.
+					append_var CXXFLAGS "-gdwarf-2 -fno-var-tracking-assignments"
 				fi
 			fi