This is an archive of the discontinued LLVM Phabricator instance.

R600/SI: Fix verifier error with pseudo store instructions.
ClosedPublic

Authored by arsenm on Apr 30 2014, 2:52 PM.

Reviewers

Summary

Use i32 instead of specifying SReg_32. When this is
the pseudo INDIRECT_BASE_ADDR, this would give a bogus
verifier error.

No test since I for some reason haven't reproduced this on trunk without a set of patches that I've been working on.

arsenm retitled this revision from to R600/SI: Fix verifier error with pseudo store instructions..

arsenm updated this object.

arsenm edited the test plan for this revision. (Show Details)

arsenm added a subscriber: Unknown Object (MLST).

r207770

This revision is now accepted and ready to land.May 1 2014, 9:47 AM

Path

Size

lib/

Target/

R600/

2 lines

Context not available.
	}]>;	}]>;

	def FRAMEri32 : Operand<iPTR> {	def FRAMEri32 : Operand<iPTR> {
	let MIOperandInfo = (ops SReg_32:$ptr, i32imm:$index);	let MIOperandInfo = (ops i32:$ptr, i32imm:$index);
	}	}

	//===----------------------------------------------------------------------===//	//===----------------------------------------------------------------------===//
Context not available.