<div dir="ltr">Hi Daniel,<div><br></div><div>Yes, looks good to me too.</div><div><br></div><div>I'm wondering if this is the only case where you can do that. If there are other cases (other load types), than we might think of a better way of checking for similarity, but for now, I think your condition is ok.</div>
<div><br></div><div>cheers,</div><div>--renato</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 13 August 2013 21:03, Daniel Stewart <span dir="ltr"><<a href="mailto:stewartd@codeaurora.org" target="_blank">stewartd@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal"><span style="color:#1f497d">Just wanted to bump this request again to see if I can get this patch reviewed.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="color:#1f497d">This patch tries to allow schedules of loads to be near each other even if one is a t2LDRBi8 load and the other is a t2LDRBi12 load. Currently, loads must be of the exact same type to be considered near each other. It is advantageous in cases where they loads are not of the exact same type for them to at least be tested for nearness. In the case of ARM thumb  code, addresses which are +1 and -1 offset from a base address could be considered near each other, but currently are not due to the use of differing opcodes to encode the positive and negative offsets. <u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="color:#1f497d">Daniel Stewart<u></u><u></u></span></p><p class="MsoNormal"><span style="color:#1f497d">--<u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d">Qualcomm Innovation Center, Inc is a member of Code Aurora Forum, hosted by The Linux Foundation<u></u><u></u></span></p><p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
<div><div style="border:none;border-top:solid #b5c4df 1.0pt;padding:3.0pt 0in 0in 0in"><p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> <a href="mailto:llvm-commits-bounces@cs.uiuc.edu" target="_blank">llvm-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:llvm-commits-bounces@cs.uiuc.edu" target="_blank">llvm-commits-bounces@cs.uiuc.edu</a>] <b>On Behalf Of </b>Daniel Stewart<br>
<b>Sent:</b> Friday, August 02, 2013 2:28 PM<br><b>To:</b> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br><b>Subject:</b> [PATCH] Add Thumb2 load byte instructions in check for same address<u></u><u></u></span></p>
</div></div><div class="im"><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">When determining if two different loads are from the same base address, this patch allows one load to use a t2LDRi8 address mode and another to use a t2LDRi12 address mode. The current implementation is very conservative and this allows the case of differing Thumb2 byte loads to be considered. Allowing these differing modes instead of forcing the exact same opcode is useful for situations where one opcodes loads from a base address+1 and a second opcode loads for a base address-1.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Daniel Stewart<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p></div></div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>