[PATCH] MachineFunction is exposed to X86AsmParser.

Evgeniy Stepanov eugenis at google.com
Thu Apr 3 01:40:49 PDT 2014



================
Comment at: lib/MC/MCContext.cpp:13
@@ -13,2 +12,3 @@
 #include "llvm/MC/MCAsmInfo.h"
+#include "llvm/MC/MCContext.h"
 #include "llvm/MC/MCDwarf.h"
----------------
Please move it back.
http://llvm.org/docs/CodingStandards.html#include-style

================
Comment at: lib/MC/MCContext.cpp:386
@@ +385,3 @@
+
+const SpecialCaseList &MCContext::getAsmInstrumentationBlacklist() const {
+  if (!AsmInstrumentationBlacklist) {
----------------
Looks like we need a setter method to support -fsanitize-backlist= in clang.
Then probably ClAsmInstrumentationBlacklist flag should live somewhere else, and use the setter method, too.



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



More information about the llvm-commits mailing list