yiguolei commented on code in PR #67701:
URL: https://github.com/apache/doris/pull/67701#discussion_r3968724636


##########
be/CMakeLists.txt:
##########
@@ -855,7 +855,17 @@ if (GLIBC_COMPATIBILITY)
     # Keep lance_c here instead of COMMON_THIRDPARTY: placing its required 
libm there
     # would resolve -lm symbol before Doris compatibility is scanned, 
preventing
     # the linker from selecting Doris' optimized implementations.
+    # lance_c can also introduce single-precision math references after the
+    # compatibility archive has already been scanned. As ClickHouse does for 
its
+    # compatibility archive, force the GLIBC_2.27 math symbols to be undefined 
up
+    # front so the linker extracts Doris' musl implementations instead of 
binding
+    # late references to the toolchain's libm.
     set(DORIS_LINK_LIBS ${DORIS_LINK_LIBS}
+        -Wl,-u,logf

Review Comment:
   musl does not make sure that, so we will not deal with it in doris.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to