This fix is fine Sun On Wed, Apr 6, 2011 at 11:10 AM, Hui Shi <kalin....@gmail.com> wrote: > Could a gatekeeper please review this patch? > APO Ofast assertion, no constraint graph for OpenMP lower generated PU > https://bugs.open64.net/show_bug.cgi?id=749 > Assert in ConstraintGraph::buildCGFromSummary, can't find PU's constraint > graph info in IPA output IR file. > > APO transformation will insert OpenMP pragma in PU. These pragma is lower > after LNO before WOPT, generate a separate PU for the parallel do body. > These PU has no constraint graph info in IPA output IR file. > > Fix is for OpenMP lower generated pu, directly construct constraint graph > for this PU, don’t construct from summary. > > Index: osprey/be/com/alias_analyzer.cxx > =================================================================== > --- osprey/be/com/alias_analyzer.cxx (revision 3534) > +++ osprey/be/com/alias_analyzer.cxx (working copy) > @@ -47,7 +47,7 @@ > > // What alias analyzer are we going to use? > if ( Alias_Nystrom_Analyzer ) { > - if (FILE_INFO_ipa(File_info)) > + if (FILE_INFO_ipa(File_info) && !PU_mp(Get_Current_PU ()) ) > _alias_analyzer = new NystromAliasAnalyzer(ac, tree, true); > else > _alias_analyzer = new NystromAliasAnalyzer(ac, tree); > @@ -97,6 +97,17 @@ > > MEM_POOL_Delete(&_memPool); > IPA_WN_MAP_Delete(Current_Map_Tab, _aliasTagMap); > + // delete the mp generate pu's map > + PU_Info *child = PU_Info_child(Current_PU_Info); > + while (child) { > + if (PU_mp(PU_Info_pu(child))) { > + FmtAssert(PU_Info_maptab(child)->_pool[_aliasTagMap] == > + PU_Info_maptab(Current_PU_Info)->_pool[_aliasTagMap] , > + ("the child mp pu pool is not equal to _memPool")); > + IPA_WN_MAP_Delete(PU_Info_maptab(child), _aliasTagMap); > + } > + child = PU_Info_next(child); > + } > } > > ALIAS_RESULT > > > ------------------------------------------------------------------------------ > Xperia(TM) PLAY > It's a major breakthrough. An authentic gaming > smartphone on the nation's most reliable network. > And it wants your games. > http://p.sf.net/sfu/verizon-sfdev > _______________________________________________ > Open64-devel mailing list > Open64-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/open64-devel > >
------------------------------------------------------------------------------ Xperia(TM) PLAY It's a major breakthrough. An authentic gaming smartphone on the nation's most reliable network. And it wants your games. http://p.sf.net/sfu/verizon-sfdev _______________________________________________ Open64-devel mailing list Open64-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/open64-devel
