This is an archive of the discontinued LLVM Phabricator instance.

[X86FixupBWInsts] Prefer positive checks in the test. NFC
ClosedPublic

Authored by a.elovikov on Jan 25 2018, 5:49 AM.

Reviewers

Commits

Repository

rL LLVM

Build Status

Buildable 14237
Build 14237: arc lint + arc unit

lgtm

This revision is now accepted and ready to land.Jan 25 2018, 8:21 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

test/

CodeGen/

X86/

2 lines

	Show First 20 Lines • Show All 189 Lines • ▼ Show 20 Lines
	# This code copies r10b into r9b and then uses r9w. We would like to promote			# This code copies r10b into r9b and then uses r9w. We would like to promote
	# the copy to a 32-bit copy, but because r9w is used this is not acceptable.			# the copy to a 32-bit copy, but because r9w is used this is not acceptable.
	body: \|			body: \|
	bb.0.entry:			bb.0.entry:
	successors:			successors:
	liveins: %r9d			liveins: %r9d

	%r9b = MOV8rr undef %r10b, implicit-def %r9d, implicit killed %r9d, implicit-def %eflags			%r9b = MOV8rr undef %r10b, implicit-def %r9d, implicit killed %r9d, implicit-def %eflags
	; CHECK-NOT: MOV32rr			; CHECK: %r9b = MOV8rr undef %r10b, implicit-def %r9d, implicit killed %r9d, implicit-def %eflags
	%ax = OR16rr undef %ax, %r9w, implicit-def %eflags			%ax = OR16rr undef %ax, %r9w, implicit-def %eflags
	RETQ %ax			RETQ %ax
	...			...