[PATCH] D10677: Allow deque to handle incomplete types

[Eric Fiselier via cfe-commits]

> I'm pretty sure it only affects template evaluation order, and does not
change the mangling of any name.

I agree it doesn't change the mangling of any name. I just want to make
sure that moving the initializer won't result in incompatible code gen.
I'm sure I'm insane for being concerned about this, but I just want
somebody to tell me I'm actually insane. It would make me feel better :)

Sorry
/Eric

On Mon, Oct 19, 2015 at 11:28 AM, Evgeniy Stepanov <eugenis at google.com>
wrote:

> eugenis added a comment.
>
> In http://reviews.llvm.org/D10677#266595, @EricWF wrote:
>
> > For the most part this looks good. I'm a touch concerned though about
> the changes to the static initialization. The initializer is moved from
> within the function body to outside it. Could you have somebody confirm
> this won't affect the existing ABI?
>
>
> I'm pretty sure it only affects template evaluation order, and does not
> change the mangling of any name.
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D10677
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20151021/d0252c5b/attachment-0001.html>