Re: r322984 - Allow BlockDecl in CXXRecord scope to have no access specifier.

2018-01-22 Thread Hans Wennborg via cfe-commits
Thanks! r323108. On Sat, Jan 20, 2018 at 12:24 AM, Richard Trieu wrote: > Hans, > > I recommend merging this revision into the release. It fixes an assertion > error when mixing modules and blocks. > > Richard > > > On Fri, Jan 19, 2018 at 12:46 PM, Richard Trieu via cfe-commits > wrote: >> >>

Re: r322984 - Allow BlockDecl in CXXRecord scope to have no access specifier.

2018-01-19 Thread Richard Trieu via cfe-commits
Hans, I recommend merging this revision into the release. It fixes an assertion error when mixing modules and blocks. Richard On Fri, Jan 19, 2018 at 12:46 PM, Richard Trieu via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: rtrieu > Date: Fri Jan 19 12:46:19 2018 > New Revision:

r322984 - Allow BlockDecl in CXXRecord scope to have no access specifier.

2018-01-19 Thread Richard Trieu via cfe-commits
Author: rtrieu Date: Fri Jan 19 12:46:19 2018 New Revision: 322984 URL: http://llvm.org/viewvc/llvm-project?rev=322984&view=rev Log: Allow BlockDecl in CXXRecord scope to have no access specifier. Using a BlockDecl in a default member initializer causes it to be attached to CXXMethodDecl without