[cmake/multilib] Teach llgo to respect the LLVM_LIBDIR_SUFFIX variable for multilib build and installs.

[chandlerc]

Summary

With this, I get the go subdirectory under the suffixed library which matches
the system's go subdirectory. However, when I run check-llgo everything fails
due to the linker being unable to find libgobegin-llgo or libgo-llgo. It
doesn't seem to be searching the lib64 directory where the libraries are.

Also, there are a *lot* of hard-coded 'lib' references in the gllgo code which
I feel are probably in need of multilib support as well.

So, sending this patch mostly as a starting point for folks to figure out how
to make llgo happy in a multilib world.

[chandlerc]

(added pcc as an explicit reviewer)

[pcc]

I think the right thing to do is to teach the getVariantDir function in gllgo.go to return an appropriate string (something like "../lib${LLVM_LIBDIR_SUFFIX}[/sanitizer]"). That string should eventually make its way to many of the appropriate places including the libgo build system. That and the initCompiler function looks like everything that needs to be taught about this.

Did you end up exposing LLVM_LIBDIR_SUFFIX somewhere like a #define in a header file? If so, we should be able to pick it up from Go (see e.g. bindings/go/llvm/version.go for an example of how to do that).

Let me know if you have any trouble making changes to the Go code and I'll see if I can take a look at it.

CMakeLists.txt

113

We shouldn't need this change after making my suggested changes.

[chandlerc]

Update to reflect a different direction for this suggested by Peter on IRC
where we use a CGO-accessed header file.

Still some debate as to what the right place to put this header file really
is...

[pcc]

LGTM

cmd/gllgo/gllgo.go

64 ↗

(On Diff #17677)

The formatting here looks inconsistent, did you run gofmt?

CMakeLists.txt

function(add_libgo_variant suffix cflags gocflags deps exclude_from_all)
 
   add_clobber_steps(libffi${suffix} clang ${deps})
 
   externalproject_add(libgo${suffix}
     DEPENDS clang llgo cc-wrapper libbacktrace${suffix} libffi${suffix} ${deps}
     SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR}/third_party/gofrontend/libgo
     BINARY_DIR ${CMAKE_CURRENT_BINARY_DIR}/${suffix}/libgo
     INSTALL_DIR ${CMAKE_BINARY_DIR}
-    CONFIGURE_COMMAND <SOURCE_DIR>/configure --disable-multilib --without-libatomic --prefix=<INSTALL_DIR> "CC=env REAL_CC=${CMAKE_BINARY_DIR}/bin/clang@SPACE@${cflags} ${CMAKE_BINARY_DIR}/bin/cc-wrapper" "GOC=${CMAKE_BINARY_DIR}/bin/llgo -no-prefix -fcompilerrt-prefix=${CMAKE_BINARY_DIR} ${gocflags}"
+    CONFIGURE_COMMAND <SOURCE_DIR>/configure --disable-multilib --without-libatomic --prefix=<INSTALL_DIR> --libdir=<INSTALL_DIR>/lib${LLVM_LIBDIR_SUFFIX} "CC=env REAL_CC=${CMAKE_BINARY_DIR}/bin/clang@SPACE@${cflags} ${CMAKE_BINARY_DIR}/bin/cc-wrapper" "GOC=${CMAKE_BINARY_DIR}/bin/llgo -no-prefix -fcompilerrt-prefix=${CMAKE_BINARY_DIR} ${gocflags}"
     BUILD_COMMAND make -j${PROCESSOR_COUNT}
     LOG_CONFIGURE 1
     LOG_BUILD 1
     LOG_INSTALL 1
   )
   set_property(TARGET libgo${suffix}
                PROPERTY EXCLUDE_FROM_ALL ${exclude_from_all})
 

 if(TARGET msan)
   add_libgo_variant("_msan" "-fsanitize=memory" "-fsanitize=memory" msan TRUE)
 endif()
 
 if(TARGET dfsan)
   add_libgo_variant("_dfsan" "-fsanitize=dataflow" "-fsanitize=dataflow" dfsan TRUE)
 endif()
 
-install(FILES ${CMAKE_BINARY_DIR}/lib/libgo-llgo.a
-              ${CMAKE_BINARY_DIR}/lib/libgo-llgo.so
-              ${CMAKE_BINARY_DIR}/lib/libgo-llgo.so.6
-              ${CMAKE_BINARY_DIR}/lib/libgo-llgo.so.6.0.0
-              ${CMAKE_BINARY_DIR}/lib/libgobegin-llgo.a
-        DESTINATION lib)
+set(LLGO_LIBRARY_DIR ${CMAKE_BINARY_DIR}/lib${LLVM_LIBDIR_SUFFIX})
 
-install(DIRECTORY ${CMAKE_BINARY_DIR}/lib/go
-        DESTINATION lib)
+install(FILES ${LLGO_LIBRARY_DIR}/libgo-llgo.a
+              ${LLGO_LIBRARY_DIR}/libgo-llgo.so
+              ${LLGO_LIBRARY_DIR}/libgo-llgo.so.6
+              ${LLGO_LIBRARY_DIR}/libgo-llgo.so.6.0.0
+              ${LLGO_LIBRARY_DIR}/libgobegin-llgo.a
+        DESTINATION lib${LLVM_LIBDIR_SUFFIX})
+
+install(DIRECTORY ${LLGO_LIBRARY_DIR}/go
+        DESTINATION lib${LLVM_LIBDIR_SUFFIX})
 
 add_custom_target(check-libgo
   COMMAND make -C ${CMAKE_CURRENT_BINARY_DIR}/libgo -j${PROCESSOR_COUNT} check
   DEPENDS libgo
   COMMENT "Running libgo tests")
 
 add_custom_target(check-llgo-bootstrap
   COMMAND strip -R .note.gnu.build-id -o ${CMAKE_CURRENT_BINARY_DIR}/llgo-stage2.stripped