This is an archive of the discontinued LLVM Phabricator instance.

[dwarfdump] Add missing DW_LANG_Mips_Assembler case to LanguageString()
ClosedPublic

Authored by friss on Sep 4 2014, 6:59 AM.

Details

Summary

This one should be pretty uncontroversial, just adding a missing case in a Dwarf.cpp switch. Some testcases needed updating though.

Diff Detail

Repository
rL LLVM

Event Timeline

friss updated this revision to Diff 13259.Sep 4 2014, 6:59 AM
friss retitled this revision from to [dwarfdump] Add missing DW_LANG_Mips_Assembler case to LanguageString().
friss updated this object.
friss added reviewers: dblaikie, echristo, aprantl.
friss added a subscriber: Unknown Object (MLST).
dblaikie accepted this revision.Sep 4 2014, 8:23 AM
dblaikie edited edge metadata.

Looks good to me.

This revision is now accepted and ready to land.Sep 4 2014, 8:23 AM
friss closed this revision.Sep 4 2014, 11:50 AM
friss updated this revision to Diff 13278.

Closed by commit rL217182 (authored by @friss).