<div dir="ltr">On 14 August 2013 22:31, Silviu Baranga <span dir="ltr"><<a href="mailto:Silviu.Baranga@arm.com" target="_blank">Silviu.Baranga@arm.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think the test would be easily breakable if I would add those checks.<br>
The vldm/vstm instructions were generated from fills/spills, so they might go away in the future or get replaced by some other code sequence.<br></blockquote><div><br></div><div></div></div></div><div class="gmail_extra">
So, I thought about that, and this is primarily why I think you *should* put the checks there.</div><div class="gmail_extra"><br></div><div class="gmail_extra">My point is that, on the day it gets out, this test will be irrelevant to the feature in question, and will be wasting test time for nothing.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">If you leave enough comments on the test to that regard, and put the CHECKs in place, whoever changes LLVM so that the vldm/vstm instructions won't be generated, will have the conscious decision to remove the test altogether.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">cheers,</div><div class="gmail_extra">--renato</div></div>