diff options
| author | Thiago Macieira <thiago.macieira@intel.com> | 2025-01-23 14:43:10 -0800 |
|---|---|---|
| committer | Thiago Macieira <thiago.macieira@intel.com> | 2025-02-04 11:08:09 -0800 |
| commit | 014e77801fa4b99cdb03330ccce1e212ac13a0af (patch) | |
| tree | 1c5f31de03f778ed23295241d1c15134eda31401 /src/corelib/serialization/qjsonobject.cpp | |
| parent | facf2c17707ac4827e624fb6e9546278aef0c38d (diff) | |
Fix detection of -fPIE code for QT_CONFIG(reduce_relocations)
Amends commit 19b7f854a274812d9c95fc7aaf134a12530c105f, which removed
the __PIE__ check because of the GCC version check that was alongside.
GCC and Clang define both __PIE__ and __PIC__ for -fPIE, so checking for
__PIC__'s presence isn't enough.
Drive-by amend the no-PIC #error message to also suggest
-DFEATURE_no_direct_extern_access=ON, which is more modern and produces
better code for executables too. Because of that, I've also changed the
reduce_relocations option to the CMake one, as there is no configure-
style equivalent for the modern option.
Pick-to: 6.9 6.8 6.5
Fixes: QTBUG-133101
Change-Id: I957b264eaee360fd964afffd60617964b26fe934
Reviewed-by: Marc Mutz <marc.mutz@qt.io>
Diffstat (limited to 'src/corelib/serialization/qjsonobject.cpp')
0 files changed, 0 insertions, 0 deletions