[PATCH] D26906: [ELF] Convert EhFrameHeader to input section
George Rimar via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 21 02:48:30 PST 2016
grimar added inline comments.
================
Comment at: ELF/SyntheticSections.cpp:39
using namespace llvm;
+using namespace llvm::dwarf;
using namespace llvm::ELF;
----------------
I think it is not ordered. Because for example we have
```
using namespace llvm;
using namespace llvm::dwarf;
using namespace llvm::object;
using namespace llvm::support::endian;
using namespace llvm::ELF;
```
In OutputSections.cpp file, so looks we separate uppercase and lowercase elements.
================
Comment at: ELF/Writer.cpp:986
+ In<ELFT>::EhFrameHdr, In<ELFT>::Dynamic});
}
----------------
Not sure if this is formatted (looks a bit strange in phab-view for me).
Repository:
rL LLVM
https://reviews.llvm.org/D26906
More information about the llvm-commits
mailing list