[llvm] r181864 - ELFRelocationEntry::operator<(): Try to stabilize the order. r_offset was insufficient to sort Relocs.

Bill Wendling isanbard at gmail.com
Fri May 17 12:55:42 PDT 2013


On May 17, 2013, at 12:27 PM, Rafael Espíndola <rafael.espindola at gmail.com> wrote:

> On 17 May 2013 14:52, Bill Wendling <isanbard at gmail.com> wrote:
>> I applied 181864, but 181910 had conflicts, so I didn't apply it.
> 
> Not a big argument one way or the other, but the conflict is just from
> adjustFixupOffset being removed on trunk. I have attached the trunk
> and 3.3 versions of the patch.
> 
Thanks for the patches. However, if this is just a cleanup and not fixing a bug, then I’d prefer to just leave it in there. That is as long as it won’t interfere with further patches in that area. We’re not yet at the “bug-fixes-only” stage, but I want to limit the amount of churn so that rc2 will go smoothly.

-bw





More information about the llvm-commits mailing list