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

Saleem Abdulrasool via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 18 18:54:28 PST 2017


compnerd added a comment.

While I love this direction (the original version really was an unintelligible pile of code), I really think that this change may be taking on too much.  Why not split it up first and do nothing else.  We could do the MS ABI implementation in a subsequent change.  This would improve the code and would not be gated on the MS ABI changes.


https://reviews.llvm.org/D28785





More information about the cfe-commits mailing list