<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:#0563C1;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:#954F72;
        text-decoration:underline;}
span.EmailStyle17
        {mso-style-type:personal-compose;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-family:"Calibri",sans-serif;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.25in 1.0in 1.25in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72">
<div class="WordSection1">
<p class="MsoNormal">Hi all,<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">I’ve encountered an issue with x86 Intel asm syntax when using certain variable names.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">If you look at the following example, where I try to do a mov to a memory location named “flags2”, llvm- mc works fine:<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">>cat test_good.s<o:p></o:p></p>
<p class="MsoNormal">mov eax, flags2<o:p></o:p></p>
<p class="MsoNormal">>llvm-mc.exe -x86-asm-syntax=intel test_good.s -o -<o:p></o:p></p>
<p class="MsoNormal">        .text<o:p></o:p></p>
<p class="MsoNormal">        movl    flags2, %eax<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">But if the memory location is named “flags”, llvm-mc fails:<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">>cat test_bad.s<o:p></o:p></p>
<p class="MsoNormal">mov eax, flags<o:p></o:p></p>
<p class="MsoNormal">>llvm-mc.exe -x86-asm-syntax=intel test_bad.s -o -<o:p></o:p></p>
<p class="MsoNormal">test_bad.s:1:1: error: invalid operand for instruction<o:p></o:p></p>
<p class="MsoNormal">mov eax, flags<o:p></o:p></p>
<p class="MsoNormal">^<o:p></o:p></p>
<p class="MsoNormal">        .text<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">After investigation, I saw that the memory location named “flags” was matched to the EFLAGS register in the
<span style="background:white;mso-highlight:white">MatchRegisterName</span>() function in the generated X86GenAsmMatcher.inc.<o:p></o:p></p>
<p class="MsoNormal"><span style="background:white;mso-highlight:white"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="background:white;mso-highlight:white">case 'f': // 1 string to match.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="background:white;mso-highlight:white">      if (memcmp(Name.data()+1, "lags", 4))<o:p></o:p></span></p>
<p class="MsoNormal"><span style="background:white;mso-highlight:white">        break;<o:p></o:p></span></p>
<p class="MsoNormal"><span style="background:white;mso-highlight:white">      return 25;      // "flags"</span><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">So basically, what I’m seeing with “flags” (which should be a legit variable name) is that the X86AsmParser creates a reference to an implicit register instead of a reference to memory.<o:p></o:p></p>
<p class="MsoNormal">There are additional issues here as well - what if we compile to SSE, but use a variable named “ZMM0” which is a register in AVX-512? Should this be allowed?<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">We probably need some way to mark the registers (using attributes or predicates?) so that we’d know which ones are part of the legal set of registers that can be referenced in the architecture we’re compiling too.<o:p></o:p></p>
<p class="MsoNormal">Do you think this is a good approach?<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thanks,<o:p></o:p></p>
<p class="MsoNormal">Marina<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<p>---------------------------------------------------------------------<br>
Intel Israel (74) Limited</p>

<p>This e-mail and any attachments may contain confidential material for<br>
the sole use of the intended recipient(s). Any review or distribution<br>
by others is strictly prohibited. If you are not the intended<br>
recipient, please contact the sender and delete all copies.</p></body>
</html>