This is an archive of the discontinued LLVM Phabricator instance.

[codeview] Use CVTypeVisitor instead of a hand-written switch-cases.
ClosedPublic

Authored by ruiu on Jun 15 2016, 3:56 PM.

Details

Summary

As we talked offline, I converted the hand-written switch-cases with
the visitor, and it doesn't look bad. I'll update this patch once you
submit your pending patch.

Diff Detail

Event Timeline

ruiu updated this revision to Diff 60925.Jun 15 2016, 3:56 PM
ruiu retitled this revision from to [codeview] Use CVTypeVisitor instead of a hand-written switch-cases..
ruiu updated this object.
ruiu added a reviewer: zturner.
ruiu added a subscriber: llvm-commits.
zturner edited edge metadata.Jun 15 2016, 6:34 PM
zturner added a subscriber: zturner.

Feel free to submit yours first, i will deal with the merge on my end

This revision was automatically updated to reflect the committed changes.