From: Nick Desaulniers <ndesaulniers@google.com>
Date: Wed, 10 Apr 2019 16:38:44 +0000 (+0000)
Subject: add FIXME: as per echristo
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=8b8d171265e5f3577f410f61a7dcdd113b0b0562;p=llvm

add FIXME: as per echristo

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@358102 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/include/llvm/CodeGen/MachineInstr.h b/include/llvm/CodeGen/MachineInstr.h
index 0a1b927d2e9..a85fee7bf4b 100644
--- a/include/llvm/CodeGen/MachineInstr.h
+++ b/include/llvm/CodeGen/MachineInstr.h
@@ -1016,6 +1016,8 @@ public:
            getOpcode() == TargetOpcode::INLINEASM_BR;
   }
 
+  /// FIXME: Seems like a layering violation that the AsmDialect, which is X86
+  /// specific, be attached to a generic MachineInstr.
   bool isMSInlineAsm() const {
     return isInlineAsm() && getInlineAsmDialect() == InlineAsm::AD_Intel;
   }