This is an archive of the discontinued LLVM Phabricator instance.

[X86][InlineAsm][Ms Compatibility]Prefer variable name over a register when the two collides
ClosedPublic

Authored by coby on Jun 27 2017, 11:54 PM.

Reviewers

Commits

Summary

On MS-style, the following snippet:

int eax;
__asm mov eax, ebx

should yield loading of ebx, into the location pointed by the variable eax

This patch sees to it.

Currently, a reg-to-reg move would have been invoked.

This revision is now accepted and ready to land.Jul 23 2017, 12:19 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

cfe/

trunk/

test/

CodeGen/

12 lines

Property	Old Value	New Value
svn:eol-style	null	native \ No newline at end of property
svn:keywords	null	Author Date Id Rev URL \ No newline at end of property
svn:mime-type	null	text/plain \ No newline at end of property

				// REQUIRES: x86-registered-target
				// RUN: %clang_cc1 %s -triple i386-apple-darwin10 -fasm-blocks -emit-llvm -o - \| FileCheck %s

				void t() {
				int eax;
				int Ecx;
				__asm mov eax, ebx
				// CHECK: mov $0, ebx
				__asm add ecx, Ecx
				// CHECK: add ecx, $1
				}