[PATCH] D14121: Thumb state not being passed through to LLVM triple when using clang -cc1as

[Alexandros Lamprineas via cfe-commits]

labrinea added a comment.

In http://reviews.llvm.org/D14121#276389, @t.p.northover wrote:

> If you're on Linux or something you need "clang -target x86_64-apple-darwin -arch armv7 -c tmp.s". Another mess I keep meaning to fix.
>
> I suspect the reason for this hack is that we've already changed the triple to "thumbv7-apple-iosN" by this point (because -arch armv7 compiles to Thumb), so it needs undoing for .s files. It might be reasonably easy to push the TY_PP_Asm check back into the Darwin codepath, or it might be horrible. So much has changed since 2011.


I just tried that and I get :
-cc1 -triple thumbv7-apple-ios5.0.0
-cc1as -triple thumbv7-apple-ios5.0.0
which means you were right.

My previosu revision gives:
-cc1 -triple thumbv7-apple-ios5.0.0
-cc1as -triple armv7-apple-ios5.0.0
so I think it should be ok.


http://reviews.llvm.org/D14121