[PATCH] D21418: [codeview] Use CVTypeVisitor instead of a hand-written switch-cases.

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 15 18:34:43 PDT 2016


Feel free to submit yours first, i will deal with the merge on my end
On Wed, Jun 15, 2016 at 3:56 PM Rui Ueyama <ruiu at google.com> wrote:

> ruiu created this revision.
> ruiu added a reviewer: zturner.
> ruiu added a subscriber: llvm-commits.
>
> 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.
>
> http://reviews.llvm.org/D21418
>
> Files:
>   include/llvm/DebugInfo/PDB/Raw/TpiStream.h
>   lib/DebugInfo/PDB/Raw/TpiStream.cpp
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160616/932966d3/attachment.html>


More information about the llvm-commits mailing list