<div dir="ltr">On 5 February 2013 23:21, Greg Fitzgerald <span dir="ltr"><<a href="mailto:garious@gmail.com" target="_blank">garious@gmail.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">

<div dir="ltr"><div><div><div><span style="font-family:arial,sans-serif;font-size:13px">> Is this for inline asm?</span><br></div></div></div><div>

<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div></div><div><span style="font-family:arial,sans-serif;font-size:13px">Yep</span></div></div></blockquote><div><br></div><div>In that case, using GAS is always a better options... for now. ;)</div>

<div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><font face="arial, sans-serif">What I'd prefer to see is if the integrated-as doesn't recognize the option, clang should report a warning and switch to the 'no-integrated-as' path.  Thoughts?</font></div>

</div></blockquote><div><br></div><div>The main problem with this is that in a large project, with different flags, some files will be compiled with integrated-as, others with GAS, and this could also change in time, creating a good headache for many people. ;)</div>
<div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><span style="font-family:arial,sans-serif;font-size:13px">I agree that's the more important question, but shouldn't we be able to throw hand-written assembly files at clang too?</span></div>
</div></blockquote><div><br></div><div style>Of course we should, but as long as it's hidden behind a flag, it'll never be properly tested and people will never fix the bugs...</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div><span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:13px">$ echo "void a(){}" | clang -x c -S - -o - | clang -x assembler -c - -integrated-as</span><br></div>


<div><stdin>:2:2: error: unknown directive</div><div>.eabi_attribute 6, 2</div></div></blockquote><div><br></div><div style>Hum, that's bad and should be fixed. Can you create a bug, pls?</div><div><br></div><div>
<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><span style="font-family:arial,sans-serif">I agree, but if you do it today, some builds will certainly choke at compile-time.  Chromium serves as a nice big workload that currently builds with a very recent version of Clang.  As is, that build will go down if you switched the default on them.</span></div>
</div></div></blockquote><div><br></div><div style>I think this is a concern. Not everyone can change to recent versions of clang as soon as they come out, but as long as the problems are just warnings, I think we can risk a bit more exposure.</div>
<div style><br></div><div style>Can you compile Chromium entirely, ignoring the warnings and hand-written ASM, with integrated-as, for ARM?</div><div style><br></div><div style>In your project configuration, you could add "-no-integrated-as" for the parts that do have inline ASM.</div>
<div style><br></div><div style>cheers,</div><div style>--renato</div></div></div></div>