hi, sun.
No one reviewed. Just do a format work.
2010/8/4 Sun Chan <sun.c...@gmail.com>
> who reviewed?
> Sun
>
> On Tue, Aug 3, 2010 at 8:11 PM, <s...@open64.net> wrote:
> > Author: shenruifen
> > Date: 2010-08-03 23:11:36 -0400 (Tue, 03 Aug 2010)
> > New Revision: 3301
> >
> > Modified:
> > trunk/osprey/be/cg/gra_mon/gra_color.cxx
> > trunk/osprey/be/cg/gra_mon/gra_create.cxx
> > trunk/osprey/be/cg/gra_mon/gra_grant.cxx
> > trunk/osprey/be/cg/gra_mon/gra_split.cxx
> > Log:
> > Format GRA code enclosed by TARG_SL
> >
> > Modified: trunk/osprey/be/cg/gra_mon/gra_color.cxx
> > ===================================================================
> > --- trunk/osprey/be/cg/gra_mon/gra_color.cxx 2010-08-03 05:03:53 UTC
> (rev 3300)
> > +++ trunk/osprey/be/cg/gra_mon/gra_color.cxx 2010-08-04 03:11:36 UTC
> (rev 3301)
> > @@ -248,38 +248,37 @@
> > void
> > Get_Rid_For_Lrange(LRANGE* lrange, vector <RID*> * rid_vec )
> > {
> > -
> > - LRANGE_LIVE_GBB_ITER live_gbb_iter;
> > - BB* bb;
> > - if(lrange->Type()== LRANGE_TYPE_REGION) { //region for major
> thread;
> > - return;
> > - }
> > - if(lrange->Type() ==LRANGE_TYPE_LOCAL) {
> > - bb = lrange->Gbb()->Bb();
> > - RID* rid = BB_rid(bb);
> > - if(rid ) { // don't split register for major fork
> > - if(RID_TYPE_major(rid))
> > - return;
> > - else { // for minor fork
> > - rid_vec->push_back(rid);
> > - return;
> > - }
> > - }
> > - }
> > - else { //complement region
> > - Is_True((lrange->Type()==LRANGE_TYPE_COMPLEMENT), ("unknown
> region type"));
> > - LRANGE_LUNIT_ITER lunit_iter;
> > - for (lunit_iter.Init(lrange); ! lunit_iter.Done();
> lunit_iter.Step())
> > - {
> > - LUNIT* lunit = lunit_iter.Current();
> > - GRA_BB* gbb = lunit->Gbb();
> > - bb = gbb->Bb();
> > - if(BB_rid(bb) && (find(rid_vec->begin(), rid_vec->end(),
> BB_rid(bb)) == rid_vec->end())) {
> > - rid_vec->push_back(BB_rid(bb));
> > - }
> > - }
> > - }
> > - return;
> > + LRANGE_LIVE_GBB_ITER live_gbb_iter;
> > + BB* bb;
> > + if(lrange->Type()== LRANGE_TYPE_REGION) { //region for major thread;
> > + return;
> > + }
> > + if(lrange->Type() ==LRANGE_TYPE_LOCAL) {
> > + bb = lrange->Gbb()->Bb();
> > + RID* rid = BB_rid(bb);
> > + if(rid ) { // don't split register for major fork
> > + if(RID_TYPE_major(rid))
> > + return;
> > + else { // for minor fork
> > + rid_vec->push_back(rid);
> > + return;
> > + }
> > + }
> > + }
> > + else { //complement region
> > + Is_True((lrange->Type()==LRANGE_TYPE_COMPLEMENT), ("unknown region
> type"));
> > + LRANGE_LUNIT_ITER lunit_iter;
> > + for (lunit_iter.Init(lrange); ! lunit_iter.Done();
> lunit_iter.Step())
> > + {
> > + LUNIT* lunit = lunit_iter.Current();
> > + GRA_BB* gbb = lunit->Gbb();
> > + bb = gbb->Bb();
> > + if(BB_rid(bb) && (find(rid_vec->begin(), rid_vec->end(),
> BB_rid(bb)) == rid_vec->end())) {
> > + rid_vec->push_back(BB_rid(bb));
> > + }
> > + }
> > + }
> > + return;
> > }
> > #endif
> >
> > @@ -335,20 +334,20 @@
> > }
> >
> > #ifdef TARG_SL //minor_reg_alloc
> > -// if( !lrange->Spans_Multiregions()) {
> > - vector< RID* > rid_vec;
> > - vector<RID* >::iterator iter;
> > - Get_Rid_For_Lrange(lrange, &rid_vec);
> > - for(iter = rid_vec.begin(); iter != rid_vec.end(); iter++) {
> > - RID* rid = *iter;
> > - if(rid && RID_TYPE_minor(rid)) {
> > - RID * pair_rid = gra_para_region_mgr.Get_Pair_Rid(rid);
> > - Is_True((pair_rid), ("pair_rid is NULL"));
> > - GRA_PARA_REGION* pair_region =
> gra_para_region_mgr.Get(pair_rid);
> > - pair_region->Add_One_Exclude_Register(rc, reg);
> > - }
> > - }
> > -// }
> > +//if( !lrange->Spans_Multiregions()) {
> > + vector< RID* > rid_vec;
> > + vector<RID* >::iterator iter;
> > + Get_Rid_For_Lrange(lrange, &rid_vec);
> > + for(iter = rid_vec.begin(); iter != rid_vec.end(); iter++) {
> > + RID* rid = *iter;
> > + if(rid && RID_TYPE_minor(rid)) {
> > + RID * pair_rid = gra_para_region_mgr.Get_Pair_Rid(rid);
> > + Is_True((pair_rid), ("pair_rid is NULL"));
> > + GRA_PARA_REGION* pair_region = gra_para_region_mgr.Get(pair_rid);
> > + pair_region->Add_One_Exclude_Register(rc, reg);
> > + }
> > + }
> > +//}
> > #endif
> >
> > }
> > @@ -689,20 +688,19 @@
> > current_lrange = 0;
> > #endif // TARG_IA64
> > #ifdef TARG_SL //minor_reg_alloc
> > -// if(!lrange->Spans_Multiregions())
> > -// {
> > - vector< RID*> rid_vec;
> > - vector< RID* >::iterator iter;
> > - Get_Rid_For_Lrange(lrange, &rid_vec);
> > +//if(!lrange->Spans_Multiregions()) {
> > + vector< RID*> rid_vec;
> > + vector< RID* >::iterator iter;
> > + Get_Rid_For_Lrange(lrange, &rid_vec);
> > for(iter=rid_vec.begin(); iter !=rid_vec.end(); iter++ ) {
> > - RID* rid = *iter;
> > - if(rid && RID_TYPE_minor(rid)) {
> > - GRA_PARA_REGION* region = gra_para_region_mgr.Get(rid);
> > - REGISTER_SET exclude_set =
> region->Registers_Exclude(lrange->Rc());
> > - allowed = REGISTER_SET_Difference(allowed, exclude_set);
> > - }
> > + RID* rid = *iter;
> > + if(rid && RID_TYPE_minor(rid)) {
> > + GRA_PARA_REGION* region = gra_para_region_mgr.Get(rid);
> > + REGISTER_SET exclude_set =
> region->Registers_Exclude(lrange->Rc());
> > + allowed = REGISTER_SET_Difference(allowed, exclude_set);
> > + }
> > }
> > -// }
> > +//}
> > #endif
> >
> > if ( lrange->Has_Wired_Register() ) {
> > @@ -918,11 +916,11 @@
> >
> > #ifdef TARG_SL2 //minor_reg_alloc
> > BB* bb = gbb->Bb();
> > - if(BB_rid(bb) && RID_TYPE_minor(BB_rid(bb))) {
> > - GRA_PARA_REGION* region = gra_para_region_mgr.Get(BB_rid(bb));
> > - REGISTER_SET exclude_set = region->Registers_Exclude(rc);
> > + if(BB_rid(bb) && RID_TYPE_minor(BB_rid(bb))) {
> > + GRA_PARA_REGION* region = gra_para_region_mgr.Get(BB_rid(bb));
> > + REGISTER_SET exclude_set = region->Registers_Exclude(rc);
> > allowed = REGISTER_SET_Difference(allowed, exclude_set);
> > - }
> > + }
> > #endif
> >
> > #ifdef HAS_STACKED_REGISTERS
> > @@ -967,12 +965,12 @@
> > GRA_GRANT_Local_Register(gbb,rc,reg);
> > #ifdef TARG_SL //minor_reg_alloc
> > if(BB_rid(gbb->Bb()) && RID_TYPE_minor(BB_rid(gbb->Bb()))) {
> > - RID* rid = BB_rid(gbb->Bb());
> > - RID* pair_rid = gra_para_region_mgr.Get_Pair_Rid(rid);
> > - GRA_PARA_REGION * pair_region =
> > - gra_para_region_mgr.Get(pair_rid);
> > - pair_region->Add_One_Exclude_Register(rc, reg);
> > - }
> > + RID* rid = BB_rid(gbb->Bb());
> > + RID* pair_rid = gra_para_region_mgr.Get_Pair_Rid(rid);
> > + GRA_PARA_REGION * pair_region =
> > + gra_para_region_mgr.Get(pair_rid);
> > + pair_region->Add_One_Exclude_Register(rc, reg);
> > + }
> > #endif
> > }
> > else {
> > @@ -1454,18 +1452,18 @@
> > void
> > Set_Children_GRA_Colored (RID * rid)
> > {
> > - RID* kid;
> > + RID* kid;
> >
> > - if(rid == NULL) return; //func_entry rid
> > -
> > - RID_was_gra_Set(rid);
> > - RID_has_reg_alloc_Set(rid);
> > + if(rid == NULL) return; //func_entry rid
> >
> > - for(kid = RID_first_kid(rid); kid != NULL; kid = RID_next(kid))
> > - {
> > - Set_Children_GRA_Colored(kid);
> > - }
> > - return;
> > + RID_was_gra_Set(rid);
> > + RID_has_reg_alloc_Set(rid);
> > +
> > + for(kid = RID_first_kid(rid); kid != NULL; kid = RID_next(kid))
> > + {
> > + Set_Children_GRA_Colored(kid);
> > + }
> > + return;
> > }
> > #endif // TARG_SL
> >
> >
> > Modified: trunk/osprey/be/cg/gra_mon/gra_create.cxx
> > ===================================================================
> > --- trunk/osprey/be/cg/gra_mon/gra_create.cxx 2010-08-03 05:03:53 UTC
> (rev 3300)
> > +++ trunk/osprey/be/cg/gra_mon/gra_create.cxx 2010-08-04 03:11:36 UTC
> (rev 3301)
> > @@ -481,10 +481,10 @@
> > RID * rid = BB_rid(bb);
> > GRA_PARA_REGION* para_region = gra_para_region_mgr.Get(rid);
> > if(rid && RID_TYPE_minor(rid)) {
> > - gra_para_region_mgr.Add_Rid_Into_Minor_Vector(rid);
> > + gra_para_region_mgr.Add_Rid_Into_Minor_Vector(rid);
> > }
> > if(para_region)
> > - para_region->Add_BB(bb);
> > + para_region->Add_BB(bb);
> > #endif
> >
> >
> > @@ -2245,40 +2245,39 @@
> > void
> > Mark_Lrange_For_Minor_Thread()
> > {
> > - ISA_REGISTER_CLASS rc;
> > - GRA_REGION_RC_NL_LRANGE_ITER iter0;
> > - GRA_REGION_GBB_ITER gbb_iter;
> > - GRA_REGION *region = gra_region_mgr.Complement_Region();
> > + ISA_REGISTER_CLASS rc;
> > + GRA_REGION_RC_NL_LRANGE_ITER iter0;
> > + GRA_REGION_GBB_ITER gbb_iter;
> > + GRA_REGION *region = gra_region_mgr.Complement_Region();
> >
> > -// FOR_ALL_ISA_REGISTER_CLASS( rc ) {
> > - rc = ISA_REGISTER_CLASS_integer;
> > - for (iter0.Init(region,rc); ! iter0.Done(); iter0.Step()) {
> > - LRANGE* lrange0 = iter0.Current();
> > - LRANGE_LIVE_GBB_ITER live_gbb_iter;
> > - LRANGE_LUNIT_ITER lunit_iter;
> > -/* for now we think the rid_count greater than 2 means the lrange spans
> multi-region,
> > - * there only two parallel regions active at same time in minor mode
> > - */
> > - vector<INT> met_rid;
> > - for (lunit_iter.Init(lrange0); !lunit_iter.Done();
> lunit_iter.Step())
> > - {
> > - LUNIT* lunit = lunit_iter.Current();
> > - GRA_BB *live_gbb = lunit->Gbb();
> > - BB* bb = live_gbb->Bb();
> > - if(BB_rid(bb) && RID_TYPE_minor(BB_rid(bb))) { // NULL for
> func_entry region
> > - if(find(met_rid.begin(), met_rid.end(),
> RID_id(BB_rid(bb))) == met_rid.end()) {
> > - met_rid.push_back(RID_id(BB_rid(bb)));
> > - }
> > - }
> > - }
> > - if(met_rid.size() > 1 && met_rid.size() < 3)
> > - {
> > - lrange0->Spans_Multiregions_Set();
> > - }
> > - met_rid.clear();
> > + // FOR_ALL_ISA_REGISTER_CLASS( rc ) {
> > + rc = ISA_REGISTER_CLASS_integer;
> > + for (iter0.Init(region,rc); ! iter0.Done(); iter0.Step()) {
> > + LRANGE* lrange0 = iter0.Current();
> > + LRANGE_LIVE_GBB_ITER live_gbb_iter;
> > + LRANGE_LUNIT_ITER lunit_iter;
> > + /* for now we think the rid_count greater than 2 means the lrange
> spans multi-region,
> > + * there only two parallel regions active at same time in minor
> mode
> > + */
> > + vector<INT> met_rid;
> > + for (lunit_iter.Init(lrange0); !lunit_iter.Done();
> lunit_iter.Step())
> > + {
> > + LUNIT* lunit = lunit_iter.Current();
> > + GRA_BB *live_gbb = lunit->Gbb();
> > + BB* bb = live_gbb->Bb();
> > + if(BB_rid(bb) && RID_TYPE_minor(BB_rid(bb))) { // NULL for
> func_entry region
> > + if(find(met_rid.begin(), met_rid.end(), RID_id(BB_rid(bb))) ==
> met_rid.end()) {
> > + met_rid.push_back(RID_id(BB_rid(bb)));
> > + }
> > }
> > -//}
> > -
> > + }
> > + if(met_rid.size() > 1 && met_rid.size() < 3)
> > + {
> > + lrange0->Spans_Multiregions_Set();
> > + }
> > + met_rid.clear();
> > + }
> > + //}
> > }
> > #endif
> >
> >
> > Modified: trunk/osprey/be/cg/gra_mon/gra_grant.cxx
> > ===================================================================
> > --- trunk/osprey/be/cg/gra_mon/gra_grant.cxx 2010-08-03 05:03:53 UTC
> (rev 3300)
> > +++ trunk/osprey/be/cg/gra_mon/gra_grant.cxx 2010-08-04 03:11:36 UTC
> (rev 3301)
> > @@ -226,10 +226,10 @@
> > #if defined (TARG_SL)
> > if ((REGISTER_SET_MemberP(REGISTER_CLASS_function_value(rc), reg))) {
> > if (BB_call(gbb->Bb()))
> > - if (GTN_SET_MemberP(BB_live_out(gbb->Bb()),Build_Dedicated_TN(rc,
> reg, 8)))
> > - return;
> > - else if (GTN_SET_MemberP(BB_live_in
> (gbb->Bb()),Build_Dedicated_TN(rc, reg, 8)))
> > - return;
> > + if
> (GTN_SET_MemberP(BB_live_out(gbb->Bb()),Build_Dedicated_TN(rc, reg, 8)))
> > + return;
> > + else if (GTN_SET_MemberP(BB_live_in
> (gbb->Bb()),Build_Dedicated_TN(rc, reg, 8)))
> > + return;
> > }
> > else
> > #endif
> > @@ -293,15 +293,15 @@
> > gbb->Registers_Used(rc));
> > FOR_ALL_REGISTER_SET_members(free_regs, reg) {
> > #ifdef TARG_SL2 //minor_reg_alloc
> > - if(BB_rid(gbb->Bb()) && RID_TYPE_minor(BB_rid(gbb->Bb())))
> > - {
> > - GRA_PARA_REGION* region =
> gra_para_region_mgr.Get(BB_rid(gbb->Bb()));
> > - if( !REGISTER_SET_Intersection1(region->Registers_Exclude(rc),
> reg))
> > - GRA_GRANT_Local_Register(gbb, rc, reg);
> > - }
> > - else
> > + if(BB_rid(gbb->Bb()) && RID_TYPE_minor(BB_rid(gbb->Bb())))
> > + {
> > + GRA_PARA_REGION* region =
> gra_para_region_mgr.Get(BB_rid(gbb->Bb()));
> > + if( !REGISTER_SET_Intersection1(region->Registers_Exclude(rc),
> reg))
> > + GRA_GRANT_Local_Register(gbb, rc, reg);
> > + }
> > + else
> > #endif
> > - GRA_GRANT_Local_Register(gbb, rc, reg);
> > + GRA_GRANT_Local_Register(gbb, rc, reg);
> > }
> > }
> > }
> >
> > Modified: trunk/osprey/be/cg/gra_mon/gra_split.cxx
> > ===================================================================
> > --- trunk/osprey/be/cg/gra_mon/gra_split.cxx 2010-08-03 05:03:53 UTC
> (rev 3300)
> > +++ trunk/osprey/be/cg/gra_mon/gra_split.cxx 2010-08-04 03:11:36 UTC
> (rev 3301)
> > @@ -339,13 +339,13 @@
> >
> > #ifdef TARG_SL //minor_reg_alloc
> > /* exclude registers used in parallel region */
> > - if(BB_rid(gbb->Bb()) && RID_TYPE_minor(BB_rid(gbb->Bb()))) {
> > - RID* rid = BB_rid(gbb->Bb());
> > - GRA_PARA_REGION* region = gra_para_region_mgr.Get(rid);
> > - Is_True((region), ("para region is NULL"));
> > - REGISTER_SET exclude_set = region->Registers_Exclude(rc);
> > - used = REGISTER_SET_Union(used, exclude_set);
> > - }
> > + if(BB_rid(gbb->Bb()) && RID_TYPE_minor(BB_rid(gbb->Bb()))) {
> > + RID* rid = BB_rid(gbb->Bb());
> > + GRA_PARA_REGION* region = gra_para_region_mgr.Get(rid);
> > + Is_True((region), ("para region is NULL"));
> > + REGISTER_SET exclude_set = region->Registers_Exclude(rc);
> > + used = REGISTER_SET_Union(used, exclude_set);
> > + }
> > #endif
> >
> > return used;
> >
> >
> >
> ------------------------------------------------------------------------------
> > The Palm PDK Hot Apps Program offers developers who use the
> > Plug-In Development Kit to bring their C/C++ apps to Palm for a share
> > of $1 Million in cash or HP Products. Visit us here for more details:
> > http://p.sf.net/sfu/dev2dev-palm
> > _______________________________________________
> > Open64-devel mailing list
> > Open64-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/open64-devel
> >
>
>
> ------------------------------------------------------------------------------
> The Palm PDK Hot Apps Program offers developers who use the
> Plug-In Development Kit to bring their C/C++ apps to Palm for a share
> of $1 Million in cash or HP Products. Visit us here for more details:
> http://p.sf.net/sfu/dev2dev-palm
> _______________________________________________
> Open64-devel mailing list
> Open64-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/open64-devel
>
--
Best Regards.
Shen Ruifen
tel: 010-51266989-226
------------------------------------------------------------------------------
The Palm PDK Hot Apps Program offers developers who use the
Plug-In Development Kit to bring their C/C++ apps to Palm for a share
of $1 Million in cash or HP Products. Visit us here for more details:
http://p.sf.net/sfu/dev2dev-palm
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel