[PATCH] D62117: Make replaceSymbol a member function of Symbol.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 19 19:37:24 PDT 2019


MaskRay added inline comments.


================
Comment at: lld/ELF/Symbols.h:434
 
-static size_t getSymbolSize(const Symbol &Sym) {
-  switch (Sym.kind()) {
-  case Symbol::CommonKind:
+size_t Symbol::getSymbolSize() const {
+  switch (kind()) {
----------------
Consider renaming this since it is now a member function.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62117/new/

https://reviews.llvm.org/D62117





More information about the llvm-commits mailing list