From f8d662aa4b554ba903be9b19620cf9621fd99525 Mon Sep 17 00:00:00 2001 From: Rafael Espindola Date: Thu, 5 Feb 2015 14:57:47 +0000 Subject: [PATCH] Add a FIXME. Thanks to Eric for the suggestion. llvm-svn: 228300 --- llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp b/llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp index 396d9e4fbfab..35860fa13375 100644 --- a/llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp +++ b/llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp @@ -181,6 +181,9 @@ getELFSectionFlags(SectionKind K, bool InCOMDAT) { if (K.isThreadLocal()) Flags |= ELF::SHF_TLS; + // FIXME: There is nothing in ELF preventing an SHF_MERGE from being + // in a comdat. We just avoid it for now because we don't print + // those .sections correctly. if (!InCOMDAT && (K.isMergeableCString() || K.isMergeableConst())) Flags |= ELF::SHF_MERGE;