Another thought; the same changed are probably needed for the vertprog code. I think there are also a lot of bugs there.
On 3/18/07, Oliver McFadden <[EMAIL PROTECTED]> wrote: > This patch seems to break one of my longer fragment programs. I believe this > is > because it's running out of registers, but I haven't looked into it in > detail > yet. > > I think this patch should be committed, but directly followed by a patch to > reduce the number of registers used. > > > On 3/18/07, Nicolai Haehnle <[EMAIL PROTECTED]> wrote: > > There were a number of bugs related to the pairing of vector and > > scalar operations where swizzles ended up using the wrong source > > register, or an instruction was moved forward and ended up overwriting > > an aliased register. > > > > The new algorithm for register allocation is slightly conservative and > > may run out of registers before it's strictly necessary. On the plus > > side, it Just Works. > > > > Pairing of instructions is done whenever possible, and in more cases > > than before, so in practice this change should be a net win. > > > > The patch mostly fixes glean/texCombine. One remaining problem is that > > the code duplicates constants and parameters all over the place and > > therefore quickly runs out of resources and falls back to software. > > I'm going to look into that as well. > > > > Please test and commit this patch. If you notice any regressions, > > please tell me (but the tests are looking good). > > > > ~Nicolai > > > ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
