This is an archive of the discontinued LLVM Phabricator instance.

ms-inline-asm: Add a test case for the usage of labels in bracket expressions
ClosedPublic

Authored by • ehsan on Sep 22 2014, 12:51 PM.

Reviewers

Commits

Summary

This is a test for this patch: http://reviews.llvm.org/D5445.

• ehsan retitled this revision from to ms-inline-asm: Add a test case for the usage of labels in bracket expressions.

• ehsan updated this object.

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

• ehsan added a reviewer: rnk.

• ehsan added a subscriber: Unknown Object (MLST).

lgtm

test/CodeGen/ms-inline-asm.c
554 ↗	(On Diff #13950)	Interesting. I would've expected this to be a load, not a lea-like thing.

This revision is now accepted and ready to land.Sep 22 2014, 1:48 PM

test/CodeGen/ms-inline-asm.c
554 ↗	(On Diff #13950)	This is what MSVC does, AFAICT.

Closed by commit rL218271 (authored by @ehsan).

Path

Size

cfe/

trunk/

test/

CodeGen/

9 lines

	Show First 20 Lines • Show All 538 Lines • ▼ Show 20 Lines
	void label3() {			void label3() {
	__asm {			__asm {
	label:			label:
	mov eax, label			mov eax, label
	}			}
	// CHECK-LABEL: define void @label3			// CHECK-LABEL: define void @label3
	// CHECK: call void asm sideeffect inteldialect "{{.}}__MSASMLABEL_.3__label:\0A\09mov eax, {{.}}__MSASMLABEL_.3__label", "~{eax},~{dirflag},~{fpsr},~{flags}"()			// CHECK: call void asm sideeffect inteldialect "{{.}}__MSASMLABEL_.3__label:\0A\09mov eax, {{.}}__MSASMLABEL_.3__label", "~{eax},~{dirflag},~{fpsr},~{flags}"()
	}			}

				void label4() {
				__asm {
				label:
				mov eax, [label]
				}
				// CHECK-LABEL: define void @label4
				// CHECK: call void asm sideeffect inteldialect "{{.}}__MSASMLABEL_.4__label:\0A\09mov eax, {{.}}__MSASMLABEL_.4__label", "~{eax},~{dirflag},~{fpsr},~{flags}"()
				}