[LLVMdev] Assert in LiveInterval update

[Sergei Larin]

Hi Lang,

 

  The patch totally makes sense and it seems to fix the failing test.. And
it is way more elegant that my crude attempt at fixing it J 

 I'll run it now on a larger test set.  

Thanks.

 

Sergei

 

---

Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by
The Linux Foundation

 

From: llvmdev-bounces at cs.uiuc.edu [mailto:llvmdev-bounces at cs.uiuc.edu] On
Behalf Of Lang Hames
Sent: Wednesday, September 12, 2012 2:00 AM
To: Jakob Stoklund Olesen
Cc: LLVM Developers Mailing List
Subject: Re: [LLVMdev] Assert in LiveInterval update

 

Hi Jakob, Sergei,

 

I've updated getLastUseBefore in r163685. I'm still looking into a couple of
errors, but I suspect that these are due to my build configuration, rather
than the patch.

 

Sergei - could you let me know if this fixes your issue? Thanks again for
all of your work tracking this down.

 

It is best to avoid the MCRegUnitRootIterator+MCSuperRegIterator combination
because the set of super-registers can be quite large on ARM.

 

Done. I've used TRI.hasRegUnit as suggested. Thanks for the advice!

 

Cheers,

Lang.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120912/910f6c39/attachment.html>