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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits