<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Dec 6, 2015 at 10:23 AM, Marina Yatsina <span dir="ltr"><<a href="mailto:marina.yatsina@intel.com" target="_blank">marina.yatsina@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">myatsina added a comment.<br>
<br>
Sean, if I understand correctly, you, as well as Reid, support producing a warning, though the spec explicitly mentions that the memory operand is ignored, and though MSVC and ICC do not produce such warnings, right?<br>
If so, how about some kind of warning like "Memory operand ignored, assuming ES:(R|E)SI" or something of that sort?<br></blockquote><div><br></div><div>I think that if the spec is willing to include a paragraph warning about this, the assembler probably should warn about it as well. I would say "memory operand is only for determining the size, <span style="font-size:13px">ES:(R|E)DI will be used for the destination</span>" (as it says in the spec) instead of "memory operand ignored, assuming ES:(R|E)SI".</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class=""><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D14827" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14827</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div></div>