[PATCH 2/2] Revert "meson: stop CLang doing inter-procedural analysis"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
---
 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




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux