anemet accepted this revision.
anemet added a comment.
This revision is now accepted and ready to land.
LGTM with the nits below. Thanks!
================
Comment at: include/llvm/IR/DiagnosticHandler.h:1
+//===- DiagnosticHandler.cpp - DiagnosticHandler class for LLVM -*- C++
-*-===//
+//
----------------
DiagnosticHandler.h
================
Comment at: lib/IR/LLVMContext.cpp:197
return Remark->isEnabled();
-
return true;
----------------
Remove this whitespace change
================
Comment at: lib/IR/LLVMContextImpl.cpp:25
LLVMContextImpl::LLVMContextImpl(LLVMContext &C)
- : VoidTy(C, Type::VoidTyID),
+ : DiagHandler(llvm::make_unique<DiagnosticHandler>(nullptr)),
+ VoidTy(C, Type::VoidTyID),
----------------
No need to pass nullptr here.
https://reviews.llvm.org/D33514
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
