basctl/IwyuFilter_basctl.yaml   |    2 --
 config_host/config_options.h.in |    6 ------
 configure.ac                    |    1 -
 idl/source/objects/types.cxx    |    2 +-
 sc/IwyuFilter_sc.yaml           |    2 --
 sc/source/ui/app/typemap.cxx    |    2 --
 sfx2/IwyuFilter_sfx2.yaml       |    2 --
 sfx2/source/appl/appbas.cxx     |    1 -
 8 files changed, 1 insertion(+), 17 deletions(-)

New commits:
commit cbb42f9dd4cccd859d7dcad865c06b38e3df37b3
Author:     Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Fri Mar 20 16:15:32 2020 +0100
Commit:     Stephan Bergmann <sberg...@redhat.com>
CommitDate: Fri Mar 20 19:31:11 2020 +0100

    The idl __attribute__((__weak__)) hack doesn't seem necessary for 
--enable-lto
    
    dbfc495a9ace0865d764dad7404633a12cda2873 "Add config option used when we 
try to
    link one huge object file" had added it for the --enable-lto case, but 
without
    documenting exactly what it should help with.  And at least my local Linux
    --enable-lto --enable-dbgutil etc. build successfully does `make check &&
    make screenshot` without it.
    
    This removes the only use of STATIC_LINKING, so remove it completely.  (And
    basctl/source/basicide/basidesh.cxx still needs to include config_options.h 
for
    ENABLE_MERGELIBS.)
    
    Change-Id: I3820e1cacccc92f5ac0c9c7fcc539c29a6864694
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90804
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sberg...@redhat.com>

diff --git a/basctl/IwyuFilter_basctl.yaml b/basctl/IwyuFilter_basctl.yaml
index 114e025c2f10..89b634781849 100644
--- a/basctl/IwyuFilter_basctl.yaml
+++ b/basctl/IwyuFilter_basctl.yaml
@@ -2,8 +2,6 @@
 assumeFilename: basctl/source/basicide/baside2b.cxx
 blacklist:
     basctl/source/basicide/basidesh.cxx:
-    # Needed for STATIC_LINKING
-    - config_options.h
     # Needed for TypedWhichId defines is basslots.hxx
     - sfx2/dinfdlg.hxx
     - sfx2/minfitem.hxx
diff --git a/config_host/config_options.h.in b/config_host/config_options.h.in
index 8e6bc70f8d63..e87c119e648a 100644
--- a/config_host/config_options.h.in
+++ b/config_host/config_options.h.in
@@ -5,12 +5,6 @@
 #ifndef CONFIG_OPTIONS_H
 #define CONFIG_OPTIONS_H
 
-/*
- * Whether we want to link as many object files as possible into one big 
object.
- * True in case of --enable-lto.
- */
-#define STATIC_LINKING 0
-
 #define ENABLE_MERGELIBS 0
 
 #define ENABLE_RUNTIME_OPTIMIZATIONS 0
diff --git a/configure.ac b/configure.ac
index eb2befed5276..57caa0d9ae6b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -351,7 +351,6 @@ AC_MSG_CHECKING([whether to use link-time optimization])
 if test -n "$enable_lto" -a "$enable_lto" != "no"; then
     ENABLE_LTO="TRUE"
     AC_MSG_RESULT([yes])
-    AC_DEFINE(STATIC_LINKING)
 else
     ENABLE_LTO=""
     AC_MSG_RESULT([no])
diff --git a/idl/source/objects/types.cxx b/idl/source/objects/types.cxx
index 34b0e3b78847..9cb1684caffc 100644
--- a/idl/source/objects/types.cxx
+++ b/idl/source/objects/types.cxx
@@ -251,7 +251,7 @@ void SvMetaType::WriteSfxItem(
 
     // write the implementation part
     rOutStm.WriteCharPtr( "#ifdef SFX_TYPEMAP" ) << endl;
-    rOutStm.WriteCharPtr( "#if !defined(_WIN32) && 
((defined(DISABLE_DYNLOADING) && (defined(ANDROID) || defined(IOS) || 
defined(LINUX))) || STATIC_LINKING)" ) << endl;
+    rOutStm.WriteCharPtr( "#if !defined(_WIN32) && 
(defined(DISABLE_DYNLOADING) && (defined(ANDROID) || defined(IOS) || 
defined(LINUX)))" ) << endl;
     rOutStm.WriteCharPtr( "__attribute__((__weak__))" ) << endl;
     rOutStm.WriteCharPtr( "#endif" ) << endl;
     rOutStm.WriteOString( aTypeName ).WriteOString( aVarName )
diff --git a/sc/IwyuFilter_sc.yaml b/sc/IwyuFilter_sc.yaml
index 2743efdd96f2..aec7c018fdca 100644
--- a/sc/IwyuFilter_sc.yaml
+++ b/sc/IwyuFilter_sc.yaml
@@ -775,8 +775,6 @@ blacklist:
     # Needed for direct member access
     - ooo/vba/XSinkCaller.hpp
     sc/source/ui/app/typemap.cxx:
-    # Needed for STATIC_LINKING to be there for scslots.hxx
-    - config_options.h
     # Needed for MID* macros
     - mid.h
     - editeng/memberids.h
diff --git a/sc/source/ui/app/typemap.cxx b/sc/source/ui/app/typemap.cxx
index bf40d09962b7..a6a9e689d865 100644
--- a/sc/source/ui/app/typemap.cxx
+++ b/sc/source/ui/app/typemap.cxx
@@ -17,8 +17,6 @@
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  */
 
-#include <config_options.h>
-
 #include <mid.h>
 #include <editeng/memberids.h>
 #include <svx/unomid.hxx>
diff --git a/sfx2/IwyuFilter_sfx2.yaml b/sfx2/IwyuFilter_sfx2.yaml
index 16c922da30a7..e3cab84a6efc 100644
--- a/sfx2/IwyuFilter_sfx2.yaml
+++ b/sfx2/IwyuFilter_sfx2.yaml
@@ -74,8 +74,6 @@ blacklist:
     - sfx2/frame.hxx
     - sfx2/msg.hxx
     - sorgitm.hxx
-    # Needed for STATIC_LINKING to work
-    - config_options.h
     sfx2/source/appl/newhelp.cxx:
     # Actually used
     - com/sun/star/i18n/XBreakIterator.hpp
diff --git a/sfx2/source/appl/appbas.cxx b/sfx2/source/appl/appbas.cxx
index b1fd03e925b1..1dec99095687 100644
--- a/sfx2/source/appl/appbas.cxx
+++ b/sfx2/source/appl/appbas.cxx
@@ -18,7 +18,6 @@
  */
 
 #include <config_features.h>
-#include <config_options.h>
 
 #include <sal/config.h>
 
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to