[PATCH] D85079: [lld-macho] Ensure .tbss sections are also considered as ZeroFilled

Jez Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 3 10:12:31 PDT 2020


int3 added inline comments.


================
Comment at: lld/MachO/InputSection.h:39
 inline bool isZeroFill(uint8_t flags) {
-  return (flags & llvm::MachO::SECTION_TYPE) == llvm::MachO::S_ZEROFILL;
+  return llvm::MachO::isVirtualSection(flags & llvm::MachO::SECTION_TYPE);
+}
----------------
MaskRay wrote:
> If you `using namespace llvm` then you can omit `llvm::`
I think we should avoid `using namespace` inside header files...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85079



More information about the llvm-commits mailing list