<div dir="ltr">Hi Rafael,<div><br></div><div style>I noticed that as a consequence of this patch, clang is issuing the following warning when assembling for arm:</div><div style><br></div><div style>"'+soft-float-abi' is not a recognized feature for this target (ignoring feature)"<br>
</div><div class="gmail_extra"><br></div><div class="gmail_extra" style>You can reproduce this (at least on Darwin, and I expect on Linux too) by creating an empty foo.s file and running:</div><div class="gmail_extra" style>
<br></div><div class="gmail_extra" style>clang -arch armv7 -c foo.s </div><div class="gmail_extra"><br></div><div class="gmail_extra" style>It looks like the cause of the warning is that both Clang::ConstructJob and ClangAs::ConstructJob are both calling getARMTargetFeatures (indirectly via getTargetFeatures), and getARMTargetFeatures always adds the abi-appropriate flag, even when the driver was being invoked as an assembler.</div>
<div class="gmail_extra" style><br></div><div class="gmail_extra" style>I'm not very familiar with the driver code, so I didn't want to jump in and undo any of the useful parts of your refactor to fix this.</div><div class="gmail_extra" style>
<br></div><div class="gmail_extra" style>Do you have any thoughts on the right way to omit this flag when assembling?</div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>Cheers,</div><div class="gmail_extra" style>
Lang.</div></div>