[PATCH] D84707: [DFSan][NewPM] Port DataFlowSanitizer to NewPM
    Yuanfang Chen via Phabricator via cfe-commits 
    cfe-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 cfe-commits
mailing list