[PATCH] MC: Don't emit min version directives when -fno-integrated-as is on

[Filipe Cabecinhas]

BTW, cctool's "as -q" uses a clang which is "based on LLVM3.5svn”,
and r204190 was the one that implemented the *_version_min stuff.
In the next cctools version (with an updated clang,I'm guessing), "as -q"
should simply start working (unless we started emitting other new
directives in the meantime).

  Filipe

On Thu, Jan 22, 2015 at 11:03 AM, Filipe Cabecinhas <filcab at gmail.com>
wrote:

> What would be the difference between shelling out to clang -cc1as or
> simply doing a no-op on -no-integrated-as?
>
> We could also start not allowing clang (on OS X) to emit directives by
> default if the latest Xcode-shipped clang doesn't handle them. That is:
> Maybe we should say what the “ground truth” for MachO/OSX is, and which
> tools outside of llvm we should try to be compatible with.
>
> This whole problem is due to clang emitting ".macosx_version_min" and
> other directives by default, and the latest Xcode-provided clang/as -q not
> handling them.
>
> Rafael: Yes, AFAICT that is the only case that is broken (and without the
> -c, but that's basically the same case).
>
>   Filipe
>
> On Thu, Jan 22, 2015 at 10:56 AM, Rafael Espíndola <
> rafael.espindola at gmail.com> wrote:
>
>> > If cctools as isn't supported, perhaps we should make -no-integrated-as
>> > shell out to `clang -cc1as` on Darwin. Users may be relying on
>> > -no-integrated-as implying cctools as quirks, though.
>>
>> It is probably a good idea. Since cctools' as can't handle clang
>> output, the only use case that might be broken by the change is "clang
>> -c -no-integrated-as foo.s", no?
>>
>> Cheers,
>> Rafael
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150122/61e10d50/attachment.html>