[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