[PATCH] D67251: [Object] remove struct constructor, NFC

Cyndy Ishida via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 5 17:57:07 PDT 2019


cishida created this revision.
cishida added reviewers: avl, dblaikie.
Herald added a subscriber: ributzka.
Herald added a project: LLVM.

make POD struct by removing ctors


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D67251

Files:
  llvm/include/llvm/Object/ObjectFile.h


Index: llvm/include/llvm/Object/ObjectFile.h
===================================================================
--- llvm/include/llvm/Object/ObjectFile.h
+++ llvm/include/llvm/Object/ObjectFile.h
@@ -137,11 +137,6 @@
 };
 
 struct SectionedAddress {
-  // TODO: constructors could be removed when C++14 would be adopted.
-  SectionedAddress() {}
-  SectionedAddress(uint64_t Addr, uint64_t SectIdx)
-      : Address(Addr), SectionIndex(SectIdx) {}
-
   const static uint64_t UndefSection = UINT64_MAX;
 
   uint64_t Address = 0;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67251.219015.patch
Type: text/x-patch
Size: 530 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190906/fcc9ffa8/attachment.bin>


More information about the llvm-commits mailing list