Seems reasonable. I tend to prefer keeping the flag describing the behaviour driven by it (emitting the directive) rather than the nature of the platform (COFF). So something like NeedsDwarfSectionOffsetDirective instead of IsCOFF, maybe? OK with that change. http://llvm-reviews.chandlerc.com/D675