whytofly
/
qt6-bb10
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove QT_HAS_FOO() wrapper macros for __has_foo() 

Using wrappers for these macros is problematic when for example passing
the -frewrite-includes flag to preprocess sources before shipping off to
distcc or Icecream. It will also start producing warnings when compilers
implement http://eel.is/c++draft/cpp.cond#7.sentence-2. See for example
https://reviews.llvm.org/D49091

Now that all uses of the macros are gone, we can follow up
c3bd5ffdc8 and remove the wrappers.

Change-Id: I764aea17dcdabd420097a7f4bc0b987a53a345eb
Reviewed-by: Marc Mutz <marc.mutz@qt.io>
bb10
Tor Arne Vestbø 2022-11-21 15:30:03 +01:00 committed by Marc Mutz
parent 23be657dda
commit 71f35d6057
1 changed files with 0 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/corelib/global/qcompilerdetection.h
View File

@ -445,14 +445,6 @@
# define __has_include_next(x) 0
#endif
// Kept around until all submodules have transitioned
#define QT_HAS_BUILTIN(x) __has_builtin(x)
#define QT_HAS_FEATURE(x) __has_feature(x)
#define QT_HAS_ATTRIBUTE(x) __has_attribute(x)
#define QT_HAS_CPP_ATTRIBUTE(x) __has_cpp_attribute(x)
#define QT_HAS_INCLUDE(x) __has_include(x)
#define QT_HAS_INCLUDE_NEXT(x) __has_include_next(x)
#ifdef __cplusplus
# if __has_include(<version>) /* remove this check once Integrity, QNX have caught up */
# include <version>