<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Thanks Steven!<div class=""><br class=""></div><div class="">I’ve committed the patch on your behalf, with a slightly modified test case.</div><div class="">This is <span style="font-family: Menlo; font-size: 11px;" class="">Committed revision 215390.</span></div><div class=""><br class=""></div><div class="">-Quentin<br class=""><div><blockquote type="cite" class=""><div class="">On Aug 11, 2014, at 2:08 PM, Steven Wu <<a href="mailto:stevenwu@apple.com" class="">stevenwu@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">Hi Quentin<br class=""><br class="">Here is a patch with testcase added according to Tim’s suggestion. Thanks for the review<br class=""><br class="">Steven<br class=""><br class=""><span id="cid:15AE96EE-C1A5-42EC-AB71-2F51BDEF528C"><postindex.patch></span><br class=""><br class=""><blockquote type="cite" class="">On Aug 8, 2014, at 5:20 PM, Quentin Colombet <<a href="mailto:qcolombet@apple.com" class="">qcolombet@apple.com</a>> wrote:<br class=""><br class="">Hi Steven,<br class=""><br class="">Could you add a few test cases? (Ideally one per instruction you are patching.)<br class=""><br class="">Thanks,<br class="">-Quentin<br class=""><br class="">On Aug 8, 2014, at 5:15 PM, Steven Wu <<a href="mailto:stevenwu@apple.com" class="">stevenwu@apple.com</a>> wrote:<br class=""><br class=""><blockquote type="cite" class="">Hi<br class=""><br class="">This patch fix a bug that sometimes register allocator may assign same register as base and writeback register in pre/post-index loads and stores for AArch64.<br class="">According to ARMv8 manual, such result will have uncertain behaviors and clang as will reject them as well. Adding earlyclobber constraints to avoid them.<br class="">Since this behavior will depend on the register allocator so I am not sure how to produce a test. I can provide a testcase if anyone want to reproduce the problem.<br class=""><br class="">Patch attached. Please review and commit it for me.<br class=""><br class="">Thanks<br class=""><br class="">Steven<br class=""><postindex.patch><br class=""><br class="">_______________________________________________<br class="">llvm-commits mailing list<br class=""><a href="mailto:llvm-commits@cs.uiuc.edu" class="">llvm-commits@cs.uiuc.edu</a><br class="">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits<br class=""></blockquote><br class=""></blockquote><br class=""></div></blockquote></div><br class=""></div></body></html>