[PATCH] D49575: Merge demangler changes over to libcxxabi

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 20 09:44:22 PDT 2018


zturner added a subscriber: erik.pilkington.
zturner added a comment.

Yea, it's fine either way.  I was just looking at it from the perspective
of minimizing differences from the master copy in LLVM.  Definitely having
headers be self-contained is worthy, but it comes at the (very small)
expense of adding an extra difference from the master copy.  That said, I
don't feel strongly, so if you want me to add it, that's fine.


https://reviews.llvm.org/D49575





More information about the llvm-commits mailing list