>> This is OK so far, but the DW_AT_alignment attribute also needs to be >> added to the checksum computation in die_checksum and >> die_checksum_ordered. > > Thanks. I see what to do in die_checksum_ordered, but die_checksum? It > seems to handle attributes by value class, and AFAICT the classes that > DW_AT_alignment could use are already covered. What am I missing? > > Here's a patch I'm about to start testing. Does it look ok?
Sorry, I read this while I wasn't in a position to reply, then totally forgot about it. Yes, you're right about die_checksum, sorry. And, for the record, it looks OK. -cary
