[PATCH] RC->contains is a trap... (PR16106)

Eric Christopher echristo at gmail.com
Wed Jun 12 12:55:44 PDT 2013


On Wed, Jun 12, 2013 at 12:52 PM, Jakob Stoklund Olesen <stoklund at 2pi.dk> wrote:
>
> On Jun 12, 2013, at 12:36 PM, Eric Christopher <echristo at gmail.com> wrote:
>
>> Off by one error. :) PR16105.
>
> Thanks, I see the problem now ;)
>
> I’ll get back to you with a more detailed answer, but note that contains() is used extensively by copyPhysReg, storeRegToStackSlot, and loadRegFromStackSlot in all the targets.
>

Yep. I'm curious about your thoughts here :)

-eric




More information about the llvm-commits mailing list