Problembeschreibung:

Modul1:

SI_32 search(...)
{
//LOCALS
....

VARIANTE_1: if(condition) {call eval(); return(0);}
VARIANTE_2: if(condition) {v = eval() ; return(v);}
VARIANTE_3: if(condition) {return(eval());}

//code
....
}

Modul2:
SI_32 eval(...)
{

VARIANTE_A: OBJ obj[1];
VARIANTE_B: static OBJ obj[1];

//code
....

return(obj->value);
}

Variante_1 ist der gewünschte Performance-Fall.(leider nicht zu gebrauchen,
da berechnte evaluierung nicht verwendet wird)

In Variante_2+3 und Variante_B, dass heisst sobald ich den Member-Wert in irgendeiner art zuweise, oder das objekt aus eval() statisch(global) ist
-> PERFORMANCE CRASH (Faktor 100)

Der Witz an der Sache ist,dass garantiert derselbe Code durchlaufen wird.

Hat jemand eine Idee woran das liegen kann,oder schonmal ähnliche Probleme gehabt?

Gruß,

Simon2.

(1.bin ich neu,2. kann ich wohl nur hier richtig sein, 3. geht es freundlicher)

sobald ich den Member-Wert in irgendeiner art zuweise, oder das objekt aus eval() statisch(global) ist
-> PERFORMANCE CRASH (Faktor 100)

Es sind leider zu wenig Infos, um mehr als raten zu können. Das Einzige was mir einfällt: Um was für ein Objekt handelt es sich denn, wird evtl durch den Kopierkonstruktor und/oder Zuweisungsoperator viel Zeit verschlungen?

"return eval()" müsste durch die RVO (Return-Value-Optimization) eigentlich auch performanter sein als "v=eval(); return v;".

edit: Und eine statische Variable ist nicht gleich global Das Wort "global" bezeichnet eigentlich den Sichtbarkeitsbereich, eine statische Variable in einem Funktionskörper ist aber nur in dieser Funktion sichtbar.

...3. geht es freundlicher)

Das bestimmt - leider kommt es hier nicht selten vor, dass irgendwelche Bots und/oder Trolle Zeug reinposten; da kann es schonmal passieren, dass es den Falschen erwischt. Sorry.

Desperado schrieb:

hm, wie kann ich code posten, ohne das die formatierung dahingeht?
(sorry)

Mit cpp-Flags. Entweder "unten" auf den "C/C++"-Button oder selbst entsprechend "cpp" und "/cpp" in eckige Klammern (statt ") setzen.
Aber bitte: Nicht 1000 Zeilen, sondern möglichst nur einen Ausschnitt relevanten Codes...

Gruß,

Simon2.

/*----------------------------------------------------------------------------*/

SI_32 eval(UI_16 tid,BOARD *brd)
	{
	 //LOCALS
	 //--------------------------------------

	 BTB tmp,ctl;
	 SQUARE sq64,dst;
	 PIECE  pce12;

	 SI_32 v = VBALANCE;

	 COLLECTION collect[MAXTHREADS];
	 SCORE		score[MAXTHREADS];

	 //COLLECTION *const p = &collect[tid];

	 //SETTINGS
	 //--------------------------------------

	 collect[tid].clr();

	 score[tid].balance[WHITE]=
	 score[tid].balance[BLACK]= 0;

	 //LOOP WHITE PIECELIST
	 //--------------------------------------

	 tmp = brd->wpl;

	 while(tmp)
		{
		 sq64 = bsf64(tmp);

		 pce12 = toPCE12[brd->sq[toX88[sq64]]];

		 collect[tid].mat[pce12]++;

		 ctl = p_attack(brd,sq64);

		 while(ctl)
			{
			 dst = bsf64(ctl);

			 inc_attack(tid,collect,sq64,pce12,WHITE);

			 CLB64(ctl);
			}

		 CLB64(tmp);
		}

	 //LOOP BLACK PIECELIST
	 //--------------------------------------

	 tmp = brd->bpl;

	 while(tmp)
		{
		 sq64 = bsf64(tmp);

		 pce12 = toPCE12[brd->sq[toX88[sq64]]];

		 collect[tid].mat[pce12]++;

		 ctl = p_attack(brd,sq64);

		 while(ctl)
			{
			 dst = bsf64(ctl);

			 inc_attack(tid,collect,sq64,pce12,BLACK);

			 CLB64(ctl);
			}

		 CLB64(tmp);
		}

	 //PAWNSTRUCTURE ANALYSIS
	 //--------------------------------------

	 pawnstruct(tid,collect);

	 //SCORE COLLECTED DATA
	 //--------------------------------------

	 score_material(tid,collect,score);
	 score_isolani(tid,collect,score,brd);
	 score_passer(tid,collect,score,brd);

	 //printf("\n%d ",score[tid].balance[brd->ctm]);

	 return(score[tid].balance[brd->ctm]);
	}

/*----------------------------------------------------------------------------*/

/*----------------------------------------------------------------------------*/

SI_32 search_full(UI_16 tid,BOARD *brd,SI_32 alpha,SI_32 beta,SI_16 rdp,UI_16 cdp)
	{
	 SI_32 vcur,vbst;
	 UI_16 mnb;
	 UNDO undo[1];
	 MLIST mli[1];

	 scontrol->nds++;

	 //INTERRUPT SEARCH
	 //--------------------------------------

	 if(INTERRUPT) go(scontrol);

	 //HORIZON
	 //--------------------------------------

	 if(rdp<=0) {eval(tid,brd);return(0);}

	 //GENERATE (PSEUDOLEGAL) MOVES
	 //--------------------------------------

	 gen[brd->ctm](brd,mli);

	 //LOOP MOVELIST
	 //--------------------------------------

	 for(mnb=0;mnb<mli->sze;mnb++)
		{
		 move_do(brd,undo,mli->mve[mnb]);

		 if(!incheck(brd,!brd->ctm))
			{
			 vcur = -search_full(tid,brd,-beta,-alpha,rdp-1,cdp+1);
			}

		 //update_best()

		 move_undo(brd,undo);
		}

	 return(vbst);
	}

/*----------------------------------------------------------------------------*/