Revert "Issue 1486: cgo: fix C++ dynamic initialization of static variables when using alwayslink = True (#4438)" (#4551)

This reverts commit 20f7e2b as it
causes a regression (see #4548)

Author: fmeum

CONTRIBUTORS

@@ -9,7 +9,6 @@
 # Names should be added to this file as:
 #     Name <email address>
 
-Andrei Reshetkov <enscogitans.v@gmail.com>
 Benjamin Staffin <benley@gmail.com>
 Brian Silverman <bsilver16384@gmail.com>
 Damien Martin-Guillerez <dmarting@google.com>

go/private/rules/cgo.bzl

@@ -93,7 +93,6 @@ def cgo_configure(go, srcs, cdeps, cppopts, copts, cxxopts, clinkopts):
     deps_direct = []
     lib_opts = []
     runfiles = go._ctx.runfiles(collect_data = True)
-    seen_alwayslink_libs = {}
 
     # Always include the sandbox as part of the build. Bazel does this, but it
     # doesn't appear in the CompilationContext.
@@ -146,13 +145,7 @@ def cgo_configure(go, srcs, cdeps, cppopts, copts, cxxopts, clinkopts):
                         # libclntsh.dylib.12.1, users have to create a unversioned symbolic link,
                         # so it can be treated as a simple shared library too.
                         continue
-
-                if lib.basename.endswith(".lo") and lib.path not in seen_alwayslink_libs:
-                    seen_alwayslink_libs[lib.path] = True
-                    lib_opts.extend(_alwayslink_lib_opts(go, lib.path))
-                else:
-                    lib_opts.append(lib.path)
-
+                lib_opts.append(lib.path)
             clinkopts.extend(cc_link_flags)
 
         elif hasattr(d, "objc"):
@@ -209,11 +202,6 @@ def _cc_libs_and_flags(target):
                 libs.append(library_to_link.dynamic_library)
     return libs, flags
 
-def _alwayslink_lib_opts(go, lib_path):
-    if go.mode.goos == "darwin":
-        return ["-Wl,-force_load,{}".format(lib_path)]
-    return ["-Wl,-whole-archive", lib_path, "-Wl,-no-whole-archive"]
-
 def _include_unique(opts, flag, include, seen):
     if include in seen:
         return