[llvm] r260529 - Drop the hidden visibility from DebugHandlerBase for now.

Benjamin Kramer via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 11 07:41:56 PST 2016


Author: d0k
Date: Thu Feb 11 09:41:56 2016
New Revision: 260529

URL: http://llvm.org/viewvc/llvm-project?rev=260529&view=rev
Log:
Drop the hidden visibility from DebugHandlerBase for now.

If a class has hidden visibility all derived classes and all classes
that have it as a member must have hidden visibility too. That may
be fixable here but requires changes to quite a lot of debug info
classes.

This is also one of the things that GCC enforces aggressively while
clang ignores it, making testing more annoying than necessary.

Modified:
    llvm/trunk/lib/CodeGen/AsmPrinter/DebugHandlerBase.h

Modified: llvm/trunk/lib/CodeGen/AsmPrinter/DebugHandlerBase.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DebugHandlerBase.h?rev=260529&r1=260528&r2=260529&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/AsmPrinter/DebugHandlerBase.h (original)
+++ llvm/trunk/lib/CodeGen/AsmPrinter/DebugHandlerBase.h Thu Feb 11 09:41:56 2016
@@ -26,7 +26,7 @@ class MachineModuleInfo;
 
 /// Base class for debug information backends. Common functionality related to
 /// tracking which variables and scopes are alive at a given PC live here.
-class LLVM_LIBRARY_VISIBILITY DebugHandlerBase : public AsmPrinterHandler {
+class DebugHandlerBase : public AsmPrinterHandler {
 protected:
   DebugHandlerBase(AsmPrinter *A);
 




More information about the llvm-commits mailing list