This is an archive of the discontinued LLVM Phabricator instance.

R600/SI: Fix off by 1 error in used register count
ClosedPublic

Authored by arsenm on Sep 11 2014, 1:46 PM.

Reviewers

arsenm

Summary

The register numbers start at 0, so if only 1 register was used, this was reported as 0.

arsenm retitled this revision from to R600/SI: Fix off by 1 error in used register count .

arsenm updated this object.

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

arsenm added a subscriber: Unknown Object (MLST).

Merge test with existing test

r217636 with the other way

This revision is now accepted and ready to land.Sep 11 2014, 4:01 PM

Path

Size

lib/

Target/

R600/

2 lines

test/

CodeGen/

R600/

9 lines

Context not available.
	llvm_unreachable("Unknown register class");	llvm_unreachable("Unknown register class");
	}	}
	unsigned hwReg = RI->getEncodingValue(reg) & 0xff;	unsigned hwReg = RI->getEncodingValue(reg) & 0xff;
	unsigned maxUsed = hwReg + width - 1;	unsigned maxUsed = hwReg + width;
	if (isSGPR) {	if (isSGPR) {
	MaxSGPR = maxUsed > MaxSGPR ? maxUsed : MaxSGPR;	MaxSGPR = maxUsed > MaxSGPR ? maxUsed : MaxSGPR;
	} else {	} else {
Context not available.

	; RUN: llc -march=r600 -mcpu=SI -verify-machineinstrs< %s \| FileCheck -check-prefix=SI %s	; RUN: llc -march=r600 -mcpu=SI -verify-machineinstrs -asm-verbose < %s \| FileCheck -check-prefix=SI %s

	declare i32 @llvm.SI.tid() nounwind readnone	declare i32 @llvm.SI.tid() nounwind readnone

Context not available.
	store i32 %result, i32 addrspace(1)* %outptr, align 4	store i32 %result, i32 addrspace(1)* %outptr, align 4
	ret void	ret void
	}	}

		; SI-LABEL: @one_vgpr_used
		; SI: NumVgprs: 1
		define void @one_vgpr_used(i32 addrspace(1)* %out, i32 %x) nounwind {
		store i32 %x, i32 addrspace(1)* %out, align 4
		ret void
		}
Context not available.