[PATCH] D78138: [MC][COFF][ELF] Reject instructions in IMAGE_SCN_CNT_UNINITIALIZED_DATA/SHT_NOBITS sections
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 15 10:24:12 PDT 2020
MaskRay marked 4 inline comments as done.
MaskRay added inline comments.
================
Comment at: llvm/include/llvm/MC/MCSection.h:189
+ virtual StringRef getVirtualSectionKind() const { return "virtual"; }
+
----------------
rnk wrote:
> This appears to be the first non-pure virtual method, i.e. the key method. Please define it out of line to avoid duplicate debug info & vtables.
Ah, thanks for noting this.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78138/new/
https://reviews.llvm.org/D78138
More information about the llvm-commits
mailing list