[PATCH] D43005: [ARM] Error out on .arm assembler directives on windows

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added a comment.

In https://reviews.llvm.org/D43005#1020740, @mstorsjo wrote:

> In https://reviews.llvm.org/D43005#1020699, @peter.smith wrote:
>
> > To summarise:
> >
> > - It should be possible to accomplish what you want by using just the featurebits. Can you elaborate on why this doesn't work for you?
>
>
> The big question I have is, where should I initialize/override those feature bits? I see it done statically for different architecture variants in ARM.td, but here it should be set not only based on cpu but based on the OS. The existing case of setting a similar flag is done in ARMSubtarget.cpp, but when just invoking the assembler, ARMSubtarget is never constructed at all. So where should I place the code that sets the FeatureNoARM bit when OS == Windows, so that it gets set when invoking the assembler?


To clarify further - if I invoke the assembler as in the testcase I'm adding here, `llvm-mc -triple=thumbv7-win32 < %s ` (or `clang -target armv7-win32 -c file.S`, which internally remaps it to `thumbv7-win32` before it hits llvm), without manually setting the flag, what class/file should set it?


Repository:
  rL LLVM

https://reviews.llvm.org/D43005