IF/ENDIF fix does not work on large nested if/endif block

[linnaea]

Current IF/ENDIF fix only handles the innermost block, this will still trigger the IF/ENDIF out of range error if the block is enclosed in another IF/ENDIF block.

beignet/backend/src/backend/gen_insn_selection.cpp

Lines 2696 to 2708 in 6553d2c

	if (this->ctx.getIFENDIFFix() &&
	!this->block->insnList.empty() &&
	this->block->insnList.size() % 1000 == 0 &&
	this->block->endifLabel.value() != 0) {
	this->curr.flag = 0;
	this->curr.subFlag = 1;
	ir::LabelIndex jip = this->block->endifLabel;
	this->ENDIF(GenRegister::immd(0), jip);
	this->push();
	this->curr.predicate = GEN_PREDICATE_NORMAL;
	this->IF(GenRegister::immd(0), jip, jip);
	this->pop();
	}

This might be the cause of some assertions on hashcat.

Removing the assertion on gen_program.cpp as a workaround.

[linnaea]

One possible solution would be to move the code in the large block somewhere else and insert a pair of JMPI instructions.

Might need to rebuild ir::Function.

[ryoqun]

Oh, cool is this worked on at this fork? I've experienced this error when trying solana-labs/solana-perf-libs#58 on my macbook, which has a bit old integrated OpenCL intel chip. :)

[linnaea]

Oh, cool is this worked on at this fork? I've experienced this error when trying solana-labs/solana-perf-libs#58 on my macbook, which has a bit old integrated OpenCL intel chip. :)

Intel's iGPUs are slow and inefficient beyond belief when the workload is cryptography related, a Pentium E5400 can outperform 2 Haswell GT3, plus the iGPU's power draw counts towards the package power limit of the CPU and may cause the CPU to throttle, so overall you might be able to get better performance running CPU-only.

[ryoqun]

@linnaea Thanks for really quick reply! Yeah, I wasn't expecting some great number... But hearing that makes me more certain for the sad fact... Maybe I should just be satisfied that I just can build and barely run the OpenCL thingy on my environment, targeting both nvidia and intel.