[PATCH] D130429: [lld-macho] `-exported_symbols` should hide symbols before LTO runs
Vincent Lee via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 28 11:59:46 PDT 2022
thevinster accepted this revision.
thevinster added inline comments.
This revision is now accepted and ready to land.
================
Comment at: lld/MachO/Driver.cpp:537
inputFiles.insert(file);
+ didCompile = true;
+ }
----------------
Instead of needing to set `didCompile` on every loop iteration, we could extract `lto->compile` into another variable and set `didCompile` based on the size of that variable? It's minor but feels cleaner this way.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130429/new/
https://reviews.llvm.org/D130429
More information about the llvm-commits
mailing list