search for: sectionelf

Is there a way to handle this entirely in the ARM AsmPrinter bits? Adding pieces to the generic MC stuff feels a bit like overkill at first impression. On a minor note: You probably want dyn_cast<> instead of cast<> so you can do something like: if (const MCSectionELF* SectionELF = dyn_cast<MCSectionELF>(Section)) { // ... } -Jim On Jun 16, 2010, at 3:44 PM, Evan Cheng wrote: > Sorry, I have been very behind on patch reviews. Bob and Jim, do you have any opinions about this? > > Evan > > On Jun 2, 2010, at 5:18 AM, Renato Golin wrote:...

Sorry, I have been very behind on patch reviews. Bob and Jim, do you have any opinions about this? Evan On Jun 2, 2010, at 5:18 AM, Renato Golin wrote: > Hi, > > Is there any interest in this patch? Is there any better way of doing this (that will be accepted mainstream)? > > I noticed my cast check was wrong (LLVM cast asserts, rather than return a null pointer). Also,

...ith clang to IR, llc to asm, gcc to object and armlink to executable. Most of the examples I tested worked fine, but the missing relocation information made it difficult to interoperate. > You probably want dyn_cast<> instead of cast<> so you can do something like: > if (const MCSectionELF* SectionELF = dyn_cast<MCSectionELF>(Section)) { >  // ... > } Precisely. I didn't want to use c++ dynamic_cast directly, but I only found out about dyn_cast after I sent the patch. Apologies. cheers, --renato