Re: [Open64-devel] Review request: fix for LNO bug 755

2011-04-07 Thread Jian-Xin Lai
These code should come from original SGI MIPSpro compiler. There is no change log on these lines in open64.net. 2011/4/1 Sun Chan > strange that the original code turned off simplifier first and tries > to do that locally. Does anyone know why? These kind of thing is best > done in the simplifie

Re: [Open64-devel] Review request: fix for LNO bug 755

2011-03-31 Thread Sun Chan
strange that the original code turned off simplifier first and tries to do that locally. Does anyone know why? These kind of thing is best done in the simplifier and you wouldn't need this change. On principle, we don't want to have to deal with all kinds of "canonicalization" everywhere, do we? Su

[Open64-devel] Review request: fix for LNO bug 755

2011-03-31 Thread Gilmore, Doug
I attached the test and patch that were already attached to the bug. The text in the comment associated with patch attachment is: The comment above the code I changed is: // For example, //I4I4LDID 41 <1,4,.preg_I4> T<4,.predef_I4,4> # i //U4INTCONST 8 (0x