Re: r310382 - [Availability] Don't make an availability attribute imply default visibility on macOS

2017-08-15 Thread Erik Pilkington via cfe-commits
Probably not, this might be a risky change. - Erik On 8/15/17 9:15 AM, Nico Weber wrote: Do we want to merge this to 5.0? On Tue, Aug 8, 2017 at 1:09 PM, Erik Pilkington via cfe-commits mailto:cfe-commits@lists.llvm.org>> wrote: Author: epilk Date: Tue Aug 8 10:09:09 2017 New Re

Re: r310382 - [Availability] Don't make an availability attribute imply default visibility on macOS

2017-08-15 Thread Nico Weber via cfe-commits
Do we want to merge this to 5.0? On Tue, Aug 8, 2017 at 1:09 PM, Erik Pilkington via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: epilk > Date: Tue Aug 8 10:09:09 2017 > New Revision: 310382 > > URL: http://llvm.org/viewvc/llvm-project?rev=310382&view=rev > Log: > [Availability] Do

r310382 - [Availability] Don't make an availability attribute imply default visibility on macOS

2017-08-08 Thread Erik Pilkington via cfe-commits
Author: epilk Date: Tue Aug 8 10:09:09 2017 New Revision: 310382 URL: http://llvm.org/viewvc/llvm-project?rev=310382&view=rev Log: [Availability] Don't make an availability attribute imply default visibility on macOS Fixes PR33796, rdar://33655115 Differential revision: https://reviews.llvm.or