[PATCH] D95851: [MC][ELF] Support for zero flag section groups

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 13 15:07:43 PST 2021


MaskRay added inline comments.


================
Comment at: llvm/include/llvm/MC/MCContext.h:490
                                 unsigned Flags, unsigned EntrySize,
-                                const Twine &Group) {
-      return getELFSection(Section, Type, Flags, EntrySize, Group,
+                                const Twine &Group, bool IsComdat = true) {
+      return getELFSection(Section, Type, Flags, EntrySize, Group, IsComdat,
----------------
`Group == ""` means a non-SHF_GROUP section. `IsComdat = true` can be misleading for such use cases.

We should pick a variable which defaults to false here.

NoDeduplicates is a PE-COFF specific name, and we should stick with ELF naming here. So a concise name indicating zero section group flags should be used.


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

https://reviews.llvm.org/D95851



More information about the llvm-commits mailing list