rnk added a comment. In https://reviews.llvm.org/D39673#931487, @martell wrote:
> 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 while > giving us a flag to override which is the goal of this patch. We have to know the EH model before pre-processing, and that shouldn't rely on LLVM TargetOptions. We can probably reuse the `llvm::ExceptionHandling` enum instead of these various overlapping booleans, if that's the direction you want to go. However, I don't see how we can get away from the clang toolchain knowing the default EH model for each target. 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