Hi Matt,

I can patch up something that would spit out an error message and exit
rather than Segfaulting.

Right now there is no bounds check. 

Other thing I could do is re-allocate the Macro Array size when it gets
full. So it would not fail until system starts swapping :-)

I would prefer the second option. Because it might hinder your and other
language development. This might become a bigger issue as you start
writing bigger programs or libraries.

Let me know your thoughts.

-Vishal

On Mon, 2006-07-10 at 22:11 -0700, Matt Diephouse wrote:
> Vishal Soni <[EMAIL PROTECTED]> wrote:
> > Hi Matt,
> >
> > This patch is because the number of .constant decls in IMCC is limited
> > to 4096. This is a todo to make this dynamic. The evil code seems to
> > have about 4200 .constant decls being generated.
> >
> > Here is the patch to fix it. For now I bumped up the limit to 8192 and
> > it works. But this is a TODO for sure.
> 
> Thanks for investigating this, Vishal. Coke mentioned on IRC that he
> had bumped this number up once before. I'll change the ticket in RT to
> document the real problem. In the meantime, would it be possible to
> die with an error saying "too many .constants" instead of just
> segfaulting?
> 
> Thanks again,
> 

Reply via email to