[PATCH] Move calls to ResolveExceptionSpec into DefineImplicit*

Ben Langmuir blangmuir at apple.com
Tue Sep 23 17:03:08 PDT 2014


Hi Doug,

As suggested, this patch moves calls to ResolveExceptionSpec out of SetDeclDefaulted and into DefineImplicit*.  This fixes an assertion failure in CodeGen where we were not resolving an exception specification.  The one bit I’m not sure of is the position of the calls relative to the note that we issue inside the DefineImplicit* functions.  It seems to work where it is, but if I call ResolveExceptionSpec before that diagnostic I change the locations of several other diagnostics in the regression tests and drop several of these notes.  I don’t see any obvious reason for that, and haven’t had a chance to try debugging what’s going on.

Ben

-------------- next part --------------
A non-text attachment was scrubbed...
Name: rdar18309639.patch
Type: application/octet-stream
Size: 11733 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140923/1286eff0/attachment.obj>


More information about the cfe-commits mailing list