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

[Benjamin Kramer via llvm-commits]

On Fri, Feb 12, 2016 at 7:12 PM, Rafael Espíndola
<rafael.espindola at gmail.com> wrote:
> On 11 February 2016 at 10:41, Benjamin Kramer via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>> 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.
>
> Can you open a bug with the case that gcc produces an error but we don't?

No error, just warnings. We already have http://llvm.org/PR13009 for that :)