I'm pretty sure these two functions aren't virtual.<br><br>On Monday, March 16, 2015, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, Mar 14, 2015 at 02:52:25PM +0100, Björn Steinbrink wrote:<br>
> Hi Tom, Hi David,<br>
><br>
> On 2015.02.16 04: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 release.<br>
<br>
-Tom<br>
<br>
> Cheers<br>
> Björn<br>
</blockquote>