[compiler-rt] [llvm] Add explicit symbol visibility macros to InstrProfData.inc (PR #110732)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 2 20:29:55 PDT 2024


================
@@ -62,6 +62,15 @@
 #define INSTR_PROF_VISIBILITY
 #endif
 
+/* This is include is needed for symbol visibility macros used on
----------------
MaskRay wrote:

"This is include" => "This include"

That said, we usually add the comment to the use site.

Can you attach a full log of the build failure without this patch? It should be included to the patch description/commit message.

https://github.com/llvm/llvm-project/pull/110732


More information about the llvm-commits mailing list