[PATCH] D47206: Code cleanup in preparation for adding LTO for wasm. NFC.

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 22 13:15:08 PDT 2018


sbc100 marked 2 inline comments as done.
sbc100 added inline comments.


================
Comment at: ELF/Config.h:199
   unsigned LTOPartitions;
-  unsigned LTOO;
+  unsigned LTOOptLevel;
   unsigned Optimize;
----------------
ruiu wrote:
> `LTOO` is an odd name but intentional. All options are named after their corresponding command line options, and this option corresponds to -lto-o, so it should be LTOO.
OK, I mostly wanted some consistency between the linkers, so I renamed the other one and I'll use that in the wasm version too.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D47206





More information about the llvm-commits mailing list