<div dir="ltr">Sure If no one objects I'll disable this for just the ARM target. <div><br></div><div>-Nirav</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 29, 2017 at 12:23 PM, Evgeny Astigeevich via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">eastig added a comment.<br>
<br>
Hi Nirav,<br>
<br>
Could you please revert the changes? They affected Arm targets (Thumb2 code).<br>
The following sequence of stores:<br>
<br>
  MOVS     r0,#0xe5<br>
  STRB     r0,[r6,#0x1e5]<br>
  MOVS     r0,#0xe4<br>
  STRB     r0,[r6,#0x1e4]<br>
  MOVS     r0,#0xe6<br>
  STRB     r0,[r6,#0x1e6]<br>
  MOVS     r0,#0xe7<br>
  STRB     r0,[r6,#0x1e7]<br>
<br>
is optimised into<br>
<br>
  MVN      r0,#0x1b<br>
  STR      r0,[r6,#0x1e4]<br>
<br>
causing incorrect data to be written.<br>
<br>
We are working on a reproducer.<br>
<br>
Thanks,<br>
Evgeny Astigeevich<br>
The Arm Compiler Optimisation team<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D33675" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D33675</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>