[PATCH] D28785: Split exception.cpp implementation into different files for different runtimes

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 16 15:25:46 PST 2017


EricWF created this revision.

`exception.cpp` is a bloody mess. It's full of confusing `#ifdef` branches for each different ABI library we support, and it's getting unmaintainable.

This patch breaks down `exception.cpp` into multiple different header files, roughly one per implementation.


https://reviews.llvm.org/D28785

Files:
  src/exception.cpp
  src/support/runtime/exception_fallback.ipp
  src/support/runtime/exception_glibcxx.ipp
  src/support/runtime/exception_libcxxabi.ipp
  src/support/runtime/exception_libcxxrt.ipp
  src/support/runtime/exception_msvc.ipp
  src/support/runtime/exception_pointer_cxxabi.ipp
  src/support/runtime/exception_pointer_glibcxx.ipp
  src/support/runtime/exception_pointer_unimplemented.ipp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28785.84605.patch
Type: text/x-patch
Size: 23992 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170116/d8d25eaa/attachment-0001.bin>


More information about the cfe-commits mailing list