[Lldb-commits] [PATCH] D44998: ObjectFileELF: Add support for arbitrarily named code sections

[Konstantin Baladurin via Phabricator via lldb-commits]

kbaladurin added inline comments.


================
Comment at: packages/Python/lldbsuite/test/arm/breakpoint-thumb-codesection/main.c:1
+__attribute__((section("__codesection")))
+int f(int a) {
----------------
clayborg wrote:
> Will this work with all compilers we currently run the test suite with? I would assume with will work with GCC and Clang at least. IF not, we might need to make a lldbtest.h file that any test case can use and use a macro here?
What compilers do we use with test suite? This construction works fine with gcc and clang. Is it enough for us?


================
Comment at: source/Core/Section.cpp:30
 
-static const char *GetSectionTypeAsCString(lldb::SectionType sect_type) {
+const char *Section::GetSectionTypeAsCString(lldb::SectionType sect_type) {
   switch (sect_type) {
----------------
clayborg wrote:
> Why did you take static off of this function? Please remove this change, or change this function to get the section type from the section itself and not require the argument.
I change it to static method to use it in `lldb-test`. There is similar static methods in `Value` and `Scalar` classes: `Value::GetValueTypeAsCString` and `Scalar::GetValueTypeAsCString`. Is non static method more preferable for us in this case?


https://reviews.llvm.org/D44998