[PATCH] MachineFunction is exposed to X86AsmParser.

[Rafael Ávila de Espíndola]

================
Comment at: include/llvm/MC/MCTargetAsmParser.h:102
@@ +101,3 @@
+  /// Current MCTargetOptions.
+  const MCTargetOptions *MCOptions;
+
----------------
The class small and would be easy to copy. The existence of the onMCTargetOptionSet also suggests value semantics, don't you want to store just a "MCTargetOptions MCOptions"?


================
Comment at: include/llvm/MC/MCTargetAsmParser.h:122
@@ +121,3 @@
+
+  virtual void onMCTargetOptionsSet() {}
+
----------------
This can be private. Even better, can't you also just remove setMCTargetOptions method completely and add an argument to createMCAsmParser?



http://reviews.llvm.org/D3106