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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to