diff --git a/llvm/lib/MC/MCSectionMachO.cpp b/llvm/lib/MC/MCSectionMachO.cpp index 33f5087d1efb..b3aeb9c1789f 100644 --- a/llvm/lib/MC/MCSectionMachO.cpp +++ b/llvm/lib/MC/MCSectionMachO.cpp @@ -40,8 +40,8 @@ static const struct { /// SectionAttrDescriptors - This is an array of descriptors for section /// attributes. Unlike the SectionTypeDescriptors, this is not directly indexed -/// by attribute, instead it is searched. The last entry has a zero AttrFlag -/// value. +/// by attribute, instead it is searched. The last entry has an AttrFlagEnd +/// AttrFlag value. static const struct { unsigned AttrFlag; const char *AssemblerName, *EnumName; @@ -59,7 +59,9 @@ ENTRY(0 /*FIXME*/, S_ATTR_SOME_INSTRUCTIONS) ENTRY(0 /*FIXME*/, S_ATTR_EXT_RELOC) ENTRY(0 /*FIXME*/, S_ATTR_LOC_RELOC) #undef ENTRY - { 0, "none", 0 } + { 0, "none", 0 }, // used if section has no attributes but has a stub size +#define AttrFlagEnd 0xffffffff // non legal value, multiple attribute bits set + { AttrFlagEnd, 0, 0 } }; @@ -228,7 +230,7 @@ std::string MCSectionMachO::ParseSectionSpecifier(StringRef Spec, // In. // Look up the attribute. for (unsigned i = 0; ; ++i) { - if (SectionAttrDescriptors[i].AttrFlag == 0) + if (SectionAttrDescriptors[i].AttrFlag == AttrFlagEnd) return "mach-o section specifier has invalid attribute"; if (SectionAttrDescriptors[i].AssemblerName && diff --git a/llvm/test/MC/MachO/sections.s b/llvm/test/MC/MachO/sections.s index bcbd9f46628e..a7bcd2170f03 100644 --- a/llvm/test/MC/MachO/sections.s +++ b/llvm/test/MC/MachO/sections.s @@ -44,6 +44,7 @@ // .objc_meth_var_names .objc_selector_strs + .section __TEXT,__picsymbolstub4,symbol_stubs,none,16 .subsections_via_symbols @@ -51,20 +52,20 @@ // CHECK: ('cpusubtype', 3) // CHECK: ('filetype', 1) // CHECK: ('num_load_commands', 1) -// CHECK: ('load_commands_size', 2436) +// CHECK: ('load_commands_size', 2504) // CHECK: ('flag', 8192) // CHECK: ('load_commands', [ // CHECK: # Load Command 0 // CHECK: (('command', 1) -// CHECK: ('size', 2436) +// CHECK: ('size', 2504) // CHECK: ('segment_name', '\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('vm_addr', 0) // CHECK: ('vm_size', 0) -// CHECK: ('file_offset', 2464) +// CHECK: ('file_offset', 2532) // CHECK: ('file_size', 0) // CHECK: ('maxprot', 7) // CHECK: ('initprot', 7) -// CHECK: ('num_sections', 35) +// CHECK: ('num_sections', 36) // CHECK: ('flags', 0) // CHECK: ('sections', [ // CHECK: # Section 0 @@ -72,7 +73,7 @@ // CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -85,7 +86,7 @@ // CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -98,7 +99,7 @@ // CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -111,7 +112,7 @@ // CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -124,7 +125,7 @@ // CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 2) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -137,7 +138,7 @@ // CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 3) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -150,7 +151,7 @@ // CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 4) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -163,7 +164,7 @@ // CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -176,7 +177,7 @@ // CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -189,7 +190,7 @@ // CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -202,7 +203,7 @@ // CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -215,7 +216,7 @@ // CHECK: ('segment_name', '__DATA\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -228,7 +229,7 @@ // CHECK: ('segment_name', '__DATA\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -241,7 +242,7 @@ // CHECK: ('segment_name', '__DATA\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 2) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -254,7 +255,7 @@ // CHECK: ('segment_name', '__DATA\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 2) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -267,7 +268,7 @@ // CHECK: ('segment_name', '__DATA\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -280,7 +281,7 @@ // CHECK: ('segment_name', '__DATA\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 2) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -293,7 +294,7 @@ // CHECK: ('segment_name', '__DATA\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 2) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -306,7 +307,7 @@ // CHECK: ('segment_name', '__DATA\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -319,7 +320,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -332,7 +333,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -345,7 +346,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -358,7 +359,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -371,7 +372,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -384,7 +385,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -397,7 +398,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -410,7 +411,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -423,7 +424,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 2) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -436,7 +437,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 2) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -449,7 +450,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -462,7 +463,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -475,7 +476,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -488,7 +489,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -501,7 +502,7 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) @@ -514,13 +515,25 @@ // CHECK: ('segment_name', '__OBJC\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') // CHECK: ('address', 0) // CHECK: ('size', 0) -// CHECK: ('offset', 2464) +// CHECK: ('offset', 2532) // CHECK: ('alignment', 0) // CHECK: ('reloc_offset', 0) // CHECK: ('num_reloc', 0) // CHECK: ('flags', 0x2) // CHECK: ('reserved1', 0) // CHECK: ('reserved2', 0) +// CHECK: # Section 35 +// CHECK: (('section_name', '__picsymbolstub4') +// CHECK: ('segment_name', '__TEXT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00') +// CHECK: ('address', 0) +// CHECK: ('size', 0) +// CHECK: ('offset', 2532) +// CHECK: ('alignment', 0) +// CHECK: ('reloc_offset', 0) +// CHECK: ('num_reloc', 0) +// CHECK: ('flags', 0x8) +// CHECK: ('reserved1', 0) +// CHECK: ('reserved2', 16) // CHECK: ), // CHECK: ]) // CHECK: ),