<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 19, 2016 at 3:30 PM, Eddy B. <span dir="ltr"><<a href="mailto:edy.burt@gmail.com" target="_blank">edy.burt@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><span class=""><br><div class="gmail_quote">2016-01-20 1:18 GMT+02:00 David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On Tue, Jan 19, 2016 at 3:16 PM, Eddy B. <span dir="ltr"><<a href="mailto:edy.burt@gmail.com" target="_blank">edy.burt@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span>> Do all tests need it? Align is just for non-default alignment (> align(1)) so it may be OK for the tests to omit it & it's probably not right to /require/ an align attribute on a byval parameter.<br>
<br>
</span>I've had to fix some of the tests which checked that the right<br>
alignment was applied,<br></blockquote><div><br></div></span><div>Rough order of magnitude number of tests that needed to be fixed? (10s? 100s?)</div></div></div></div></blockquote><div> </div></div></span>Relevant commit: <a href="https://github.com/eddyb/llvm/commit/c76e3d6bd6b7008795e16f4422a014d2ff7d48dd" target="_blank">https://github.com/eddyb/llvm/commit/c76e3d6bd6b7008795e16f4422a014d2ff7d48dd</a></div><div class="gmail_extra">It's just 4 tests, but I had to split each one because they were getting ran against</div><div class="gmail_extra">multiple targets, and those needed different alignment attributes.</div></div></blockquote><div><br>Only casually glanced at that, but do we need to test it with different alignments on different targets? Or can we just pick one alignment to test on all of them, do you think? (if you're not sure, that's OK - I/we can look into it & see what the tests are trying to do, etc) </div></div><br></div></div>