Thanks for the quick fix!
2017-05-18 21:48 GMT+02:00 Richard Smith :
> I see, the problem is that template parameters are temporarily put in the
> wrong DeclContext while the rest of the declaration is being parsed (and
> thus end up temporarily owned by the wrong module). Fixed in r303373.
>
> On
I see, the problem is that template parameters are temporarily put in the
wrong DeclContext while the rest of the declaration is being parsed (and
thus end up temporarily owned by the wrong module). Fixed in r303373.
On 18 May 2017 at 11:52, Richard Smith wrote:
> Thanks for the awesome testcase
Thanks for the awesome testcase, taking a look now.
On 18 May 2017 at 07:15, Raphael Isemann wrote:
> Hi,
>
> this is breaking our STL module builds. Can we revert this?
>
> We also have a minimal reproducer for our crash here
> http://teemperor.de/pub/stl_merging_issue.zip
>
> - Raphael
>
> 201
Hi,
this is breaking our STL module builds. Can we revert this?
We also have a minimal reproducer for our crash here
http://teemperor.de/pub/stl_merging_issue.zip
- Raphael
2017-05-17 2:24 GMT+02:00 Richard Smith via cfe-commits
:
> Author: rsmith
> Date: Tue May 16 19:24:14 2017
> New Revision
Author: rsmith
Date: Tue May 16 19:24:14 2017
New Revision: 303224
URL: http://llvm.org/viewvc/llvm-project?rev=303224&view=rev
Log:
[modules] When creating a declaration, cache its owning module immediately
rather than waiting until it's queried.
Currently this is only applied to local submodule
