[PATCH] D53143: [LLD] [COFF] Add and use a Wordsize field in Config. NFCI.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 11 10:34:33 PDT 2018


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM

Thank you for doing this!



================
Comment at: COFF/Config.h:87
   llvm::COFF::MachineTypes Machine = IMAGE_FILE_MACHINE_UNKNOWN;
+  size_t Wordsize = 0;
   bool Verbose = false;
----------------
I wouldn't initialize it if the initial value will never be used.


https://reviews.llvm.org/D53143





More information about the llvm-commits mailing list