[PATCH] D39673: Toolchain: Normalize dwarf, sjlj and seh eh

[Martell Malone via Phabricator via cfe-commits]

martell added a comment.

Just as a note there is still a lot to be desired here. I do not particularly like the `UseSEHExceptions` function default and the actual macro definition guards should be based on the current `ExceptionModel` because we set that in `lib/CodeGen/BackendUtil.cpp`. This way we do not need to have some silly default of x64 && windows for UseSEHExceptions and can rely on the llvm backend defaults but override within the driver we want like the apple targets do for sjlj. This does for now keep the current functionality which giving us a flag to override which is the goal of this patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D39673