[PATCH] D84707: [DFSan][NewPM] Port DataFlowSanitizer to NewPM

Yuanfang Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 27 17:54:21 PDT 2020


ychen accepted this revision.
ychen added a comment.
This revision is now accepted and ready to land.

LGTM with two nits. Please wait one day or two in case other reviewers want to have a look.



================
Comment at: llvm/include/llvm/Transforms/Instrumentation/DataFlowSanitizer.h:8
+//===----------------------------------------------------------------------===//
+#ifndef LLVM_TRANSFORMS_INSTRUMENTATION_DATAFLOWSANITIZERPASS_H
+#define LLVM_TRANSFORMS_INSTRUMENTATION_DATAFLOWSANITIZERPASS_H
----------------
LLVM_TRANSFORMS_INSTRUMENTATION_DATAFLOWSANITIZER_H


================
Comment at: llvm/include/llvm/Transforms/Instrumentation/DataFlowSanitizer.h:11
+
+#include "llvm/IR/Function.h"
+#include "llvm/IR/Module.h"
----------------
Is Function.h needed?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84707/new/

https://reviews.llvm.org/D84707





More information about the llvm-commits mailing list