[LLVMdev] commit r152019 broke architectures with more than 255 registers

Craig Topper craig.topper at gmail.com
Tue Mar 6 00:33:20 PST 2012


This has been changed to uint16_t in r152100.

On Mon, Mar 5, 2012 at 10:40 AM, Jakob Stoklund Olesen <stoklund at 2pi.dk>wrote:

>
> On Mar 5, 2012, at 5:39 AM, Heikki Kultala wrote:
>
> > Our architecture(TCE) can have LOTS of registers.
> >
> > It seems r152019 changed some register bookkeeping data structures to
> > 8-bit. This broke support for architectures with >255 registers.
> >
> > Please revert this change or make those register-related values at least
> > 16 bits wide.
>
> I agree. We can limit the number of physregs to 64k, but no more.
>
> /jakob
>
>


-- 
~Craig
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120306/3ab33aae/attachment.html>


More information about the llvm-dev mailing list