ok. please checkin Sun 2010/11/1 Chen, Rui (Roger, TSG-GDCC-SH) <rui.c...@hp.com>: > I think I've modified all "WN_set_operator" invocations to > "WN_change_operator" in wopt. Only one call in opt_symb.cxx causes problem, > so I have to manually reset its map_id to -1 before call WN_change_operator. > > Regards, > Roger > > -----Original Message----- > From: Sun Chan [mailto:sun.c...@gmail.com] > Sent: 2010年11月1日 14:43 > To: Chen, Rui (Roger, TSG-GDCC-SH) > Cc: open64-devel@lists.sourceforge.net > Subject: Re: [Open64-devel] Review request for map_id problem > > looks good. OTOH, is this the only place that uses set_operator? If so, > should they need to be changed to your new scheme? > Sun > > 2010/10/31 Chen, Rui (Roger, TSG-GDCC-SH) <rui.c...@hp.com>: >> I modified the patch a little bit, creating a new "WN_change_operator" >> method to avoid many duplicate assertions. >> >> >> >> Pls help to review it again. Thanks. >> >> >> >> Regards, >> >> Roger >> >> >> >> From: Chen, Rui (Roger, TSG-GDCC-SH) >> Sent: 2010年10月29日 13:51 >> To: open64-devel@lists.sourceforge.net >> Subject: [Open64-devel] Review request for map_id problem >> >> >> >> Hi all, >> >> >> >> I found that some code changes OPERATOR of a whirl node without reset >> its map_id. This causes problems if the new OPERATOR belongs to a >> different mapcat than the old OPERATOR. >> >> >> >> I attached a test case, compile the test case using "opencc -O3 -c min.ii" >> causes assertion fault: >> >> >> >> ### Assertion failure at line 875 of >> /home/chenr/workspace/open64_backup_build/osprey/../../open64_backup/osprey/be/opt/opt_verify.cxx: >> >> ### Compiler Error in file min.ii during Global Optimization -- Verify >> Live-Range phase: >> >> ### Verify_version_CR: Found no-def, but current statement is not >> entry chi >> >> >> >> I attached a patch to fix this problem. Pls help to review the changes. >> >> >> >> Regards, >> >> Roger >> >> ---------------------------------------------------------------------- >> -------- Nokia and AT&T present the 2010 Calling All Innovators-North >> America contest Create new apps & games for the Nokia N8 for consumers >> in U.S. and Canada >> $10 million total in prizes - $4M cash, 500 devices, nearly $6M in >> marketing Develop with Nokia Qt SDK, Web Runtime, or Java and Publish >> to Ovi Store http://p.sf.net/sfu/nokia-dev2dev >> _______________________________________________ >> Open64-devel mailing list >> Open64-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/open64-devel >> >> >
------------------------------------------------------------------------------ Nokia and AT&T present the 2010 Calling All Innovators-North America contest Create new apps & games for the Nokia N8 for consumers in U.S. and Canada $10 million total in prizes - $4M cash, 500 devices, nearly $6M in marketing Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store http://p.sf.net/sfu/nokia-dev2dev _______________________________________________ Open64-devel mailing list Open64-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/open64-devel
