clayborg requested changes to this revision.
clayborg added inline comments.
This revision now requires changes to proceed.
================
Comment at:
packages/Python/lldbsuite/test/arm/breakpoint-thumb-codesection/main.c:1
+__attribute__((section("__codesection")))
+int f(int a) {
----------------
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?
================
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) {
----------------
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.
https://reviews.llvm.org/D44998
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
