<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On May 31, 2013, at 5:14 AM, Tim Northover <<a href="mailto:t.p.northover@gmail.com">t.p.northover@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><blockquote type="cite">X86: change MOV64ri64i32 into MOV32ri64<br></blockquote><br>Hmm, it seems this broke the gdb 7.5 bot:<br><a href="http://lab.llvm.org:8011/builders/clang-x86_64-ubuntu-gdb-75/builds/5399">http://lab.llvm.org:8011/builders/clang-x86_64-ubuntu-gdb-75/builds/5399</a>.<br><br>It looks like the test itself is dodgy: it consists of a few inline<br>asm blocks, marked to clobber %eax and so on. However, it expects the<br>compiler won't use those registers in *between* either.</div></blockquote><div dir="auto"><br></div><div dir="auto">This seems like a really nice invariant.</div><br><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"> Previously<br>this was true, but now on -O0 instead of:<br> movl $data, %r9d<br>we produce:<br> movl $data, %eax<br> movl %eax, %r9d<br><br>The extra instruction goes away (as it should) at higher<br>optimisations, and I think it's perfectly valid, but the test doesn't<br>like it.<br><br>Can we XFAIL individual tests on this gdb7.5 bot, or do we usually do<br>something else? I can't believe this hasn't come up before.<br></div></blockquote></div><div><br></div>We probably can, is the invariant this change breaks something that we want to break?<br><div><br></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>- Doug</div></body></html>