<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Mar 28, 2013, at 5:22 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr">On 28 March 2013 12:06, Stepan Dyatkovskiy <span dir="ltr"><<a href="mailto:stpworld@narod.ru" target="_blank">stpworld@narod.ru</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'm not sure about time, since I don't know some pitfalls, may be. But I still want try to fix llvm and clang :-)<br>
</blockquote><div><br></div><div style="">Great! That's the spirit! ;)</div><div style=""><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">You say it was fixed once and rejected then. Did somebody guys remember some thread names or revisions, may be?<br>
</blockquote><div><br></div><div style="">It wasn't fixed, it had partial support and some people (including me) tried to use only to find broken stuff down the pipe. IIRC it was disabled (possibly removed from source) to avoid people doing it again. You may find some asserts along the way, too.</div>
<div style=""><br></div><div style="">I think the best way to proceed is to make Clang lower struct byval for ARM as it does on x86 and go fixing the broken tests one by one.</div><div style=""><br></div><div style="">I'm not sure how Clang folks would prefer, a huge patch at the end with it enabled by default, or a flag --use-broken-byval-on-arm (disabled by default) for now until all tests pass with it, when you can remove the flag altogether. You better ask them directly on cfe-dev@.</div></div></div></div></blockquote><br></div><div>Manman reenabled struct byval for ARM in clang svn r161790. I'm not aware of any problems with it now.</div></body></html>