<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 16, 2015 at 7:46 AM, Tom Stellard <span dir="ltr"><<a href="mailto:tom@stellard.net" target="_blank">tom@stellard.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Mar 16, 2015 at 07:40:19AM -0700, David Majnemer wrote:<br>
> I'm pretty sure these two functions aren't virtual.<br>
><br>
<br>
</span>OK, does that mean adding them anywhere will break the ABI or does it mean they<br>
are safe to add?<br></blockquote><div><br></div><div>I think it's safe to add.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
-Tom<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> On Monday, March 16, 2015, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br>
><br>
> > On Sat, Mar 14, 2015 at 02:52:25PM +0100, Björn Steinbrink wrote:<br>
> > > Hi Tom, Hi David,<br>
> > ><br>
> > > On <a href="tel:2015.02.16%2004" value="+12015021604">2015.02.16 04</a>:02:10 -0000, David Majnemer wrote:<br>
> > > > Author: majnemer<br>
> > > > Date: Sun Feb 15 22:02:09 2015<br>
> > > > New Revision: 229352<br>
> > > ><br>
> > > > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=229352&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=229352&view=rev</a><br>
> > > > Log:<br>
> > > > IR: Properly return nullptr when getAggregateElement is out-of-bounds<br>
> > > ><br>
> > > > We didn't properly handle the out-of-bounds case for<br>
> > > > ConstantAggregateZero and UndefValue.  This would manifest as a crash<br>
> > > > when the constant folder was asked to fold a load of a constant global<br>
> > > > whose struct type has no operands.<br>
> > > ><br>
> > > > This fixes PR22595.<br>
> > ><br>
> > > Could this be added to 3.5.2?<br>
> > ><br>
> ><br>
> > This adds a new function to two difference classes, which can<br>
> > break the ABI in some cases.  According to abi-compliance-checker:<br>
> ><br>
> ><br>
> > "You can add virtual functions at end of leaf classes with exported<br>
> >  constructors."<br>
> ><br>
> > I have a buildbot setup to check abi compliance, so if you move these<br>
> > functions to the end of the classes, I will approve merging it, but if<br>
> > this is not enough to ensure ABI compliance, I will revert it before the<br>
> > release.<br>
> ><br>
> > -Tom<br>
> ><br>
> > > Cheers<br>
> > > Björn<br>
> ><br>
</div></div></blockquote></div><br></div></div>