[PATCH] D100152: Support: Drop the no-op initializer for SignpostEmitterImpl::Signposts, NFC

Duncan P. N. Exon Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 8 16:15:31 PDT 2021


dexonsmith created this revision.
dexonsmith added reviewers: dsanders, bogner.
Herald added a subscriber: hiraditya.
dexonsmith requested review of this revision.
Herald added a project: LLVM.

This is a DenseMap, which has its own initializer; we don't need to explicitly
call the default constructor here.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D100152

Files:
  llvm/lib/Support/Signposts.cpp


Index: llvm/lib/Support/Signposts.cpp
===================================================================
--- llvm/lib/Support/Signposts.cpp
+++ llvm/lib/Support/Signposts.cpp
@@ -59,7 +59,7 @@
   }
 
 public:
-  SignpostEmitterImpl() : SignpostLog(LogCreator(), LogDeleter), Signposts() {}
+  SignpostEmitterImpl() : SignpostLog(LogCreator(), LogDeleter) {}
 
   bool isEnabled() const {
     if (SIGNPOSTS_AVAILABLE())


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D100152.336255.patch
Type: text/x-patch
Size: 421 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210408/779564a1/attachment.bin>


More information about the llvm-commits mailing list