<div dir="ltr"><br><div class="gmail_extra"><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 class="">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>Relevant commit: <a href="https://github.com/eddyb/llvm/commit/c76e3d6bd6b7008795e16f4422a014d2ff7d48dd">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>