[PATCH 2/2] Revert "meson: stop CLang doing inter-procedural analysis"
This reverts commit f9f5ab57189b9c378d1893cf302865ac9b88bbbc.
This should not be needed any more thanks to previous commit (util:
Separate numactl and stubs into different source files) and future issue
should be fixed the same way as in that commit.
Signed-off-by: Martin Kletzander <mkletzan(a)redhat.com>
---
meson.build | 20 --------------------
1 file changed, 20 deletions(-)
diff --git a/meson.build b/meson.build
index c35823a79aa8..319ed790f941 100644
--- a/meson.build
+++ b/meson.build
@@ -404,26 +404,6 @@ cc_flags += [
'-Wwrite-strings',
]
-if cc.get_id() == 'clang'
- # Stop CLang from doing inter-procedural analysis of calls
- # between functions in the same compilation unit. Such an
- # optimization has been know to break the test suite by
- # making assumptions that a return value is a constant.
- # This makes it impossible to mock certain functions with
- # replacement definitions via LD_PRELOAD that have different
- # semantics.
- #
- # This is a bit of a big hammer, but alternatives don't work:
- #
- # - 'weak' attribute - weak symbols get dropped from
- # when the .a libs are combined into the .so
- # see commit 407a281a8e2b6c5078ba1148535663ea64fd9314
- #
- # - 'noipa' attribute - only available with GCC currently
- # https://reviews.llvm.org/D101011
- cc_flags += [ '-fsemantic-interposition' ]
-endif
-
supported_cc_flags = []
if get_option('warning_level') == '2'
supported_cc_flags = cc.get_supported_arguments(cc_flags)
--
2.40.0