This is an archive of the discontinued LLVM Phabricator instance.

Recognize dwarf::DW_ATE_UTF in DwarfUnit isUnsignedDIType
ClosedPublic

Authored by dblaikie on May 16 2014, 2:46 PM.

Details

Summary

This patch fixes an assert violation triggered by the following clang test case:

// compile with -std=c++11 -g
const char16_t c = 0;
char16_t test() { return c; }

David, you wrote you could help me turn this into an LLVM test case?

Diff Detail

Event Timeline

stephant updated this revision to Diff 9496.May 16 2014, 2:46 PM
stephant retitled this revision from to Recognize dwarf::DW_ATE_UTF in DwarfUnit isUnsignedDIType .
stephant updated this object.
stephant edited the test plan for this revision. (Show Details)
stephant added a reviewer: dblaikie.
stephant added a subscriber: Unknown Object (MLST).
dblaikie commandeered this revision.May 16 2014, 3:00 PM
dblaikie accepted this revision.
dblaikie edited reviewers, added: stephant; removed: dblaikie.
dblaikie added a reviewer: dblaikie.

Committed with a test case in r209037.

This revision is now accepted and ready to land.May 16 2014, 3:00 PM
dblaikie closed this revision.May 16 2014, 3:00 PM