diff options
author | Rafael Espindola <rafael.espindola@gmail.com> | 2011-05-24 03:10:31 +0000 |
---|---|---|
committer | Rafael Espindola <rafael.espindola@gmail.com> | 2011-05-24 03:10:31 +0000 |
commit | 0d018b1155e2aa365e10a0c6aafa1da907dbdc3d (patch) | |
tree | 3a19e18b3aceb1b7a0eb1cdc0d0344212c32b23a /llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp | |
parent | 0f33be1b8797a36053782fdfa54e07810d64fe1e (diff) | |
download | bcm5719-llvm-0d018b1155e2aa365e10a0c6aafa1da907dbdc3d.tar.gz bcm5719-llvm-0d018b1155e2aa365e10a0c6aafa1da907dbdc3d.zip |
Explain FIXME.
llvm-svn: 131952
Diffstat (limited to 'llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp')
-rw-r--r-- | llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp b/llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp index ff88b6f2e99..e9cfe43e6d4 100644 --- a/llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp +++ b/llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp @@ -225,10 +225,11 @@ void TargetLoweringObjectFileELF::emitPersonalityValue(MCStreamer &Streamer, static SectionKind getELFKindForNamedSection(StringRef Name, SectionKind K) { - // FIXME: Why is this here? Codegen is should not be in the business - // of figuring section flags. If the user wrote section(".eh_frame"), - // we should just pass that to MC which will defer to the assembly - // or use its default if producing an object file. + // N.B.: The defaults used in here are no the same ones used in MC. + // We follow gcc, MC follows gas. For example, given ".section .eh_frame", + // both gas and MC will produce a section with no flags. Given + // section(".eh_frame") gcc will produce + // .section .eh_frame,"a",@progbits if (Name.empty() || Name[0] != '.') return K; // Some lame default implementation based on some magic section names. |