[PATCH] MC: provide the ability to disable assembly parsing

[Daniel Sanders]

  I don't understand why you need the changes to MCAsmInfo. The intention of MCAsmInfo::UseIntegratedAssembler is that it is set to false when parsing inline assembly is unsupported or undesirable.

  There's also a -no-integrated-as option (NoIntegratedAssembler in LLVMTargetMachine.cpp). Can't clang pass that when it receives its own version of -no-integrated-as?


================
Comment at: include/llvm/MC/MCAsmInfo.h:310-315
@@ -309,2 +309,8 @@
 
+    /// Should we parse inline assembly?
+    /// The integrated assembler will usually parse inline assembly.  However,
+    /// when simply generating an object file, it can be useful to prevent this
+    /// (e.g. the Linux kernel uses it as a fancy preprocessor).
+    bool ValidateInlineASMSyntax;
+
   public:
----------------
Isn't this the same as UseIntegratedAssembler above?

================
Comment at: lib/CodeGen/LLVMTargetMachine.cpp:82
@@ -81,2 +81,3 @@
     TmpAsmInfo->setUseIntegratedAssembler(false);
+  TmpAsmInfo->SetValidateInlineAssembly(Options.ValidateInlineASMSyntax);
 
----------------
Why not:
  if (!Options.ValidateInlineASMSyntax)
    TmpAsmInfo->setUseIntegratedAssembler(false);


http://llvm-reviews.chandlerc.com/D2839