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

Martell Malone via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Nov 18 15:35:22 PST 2017


martell requested review of this revision.
martell added a comment.

Thanks for the review @rnk . I addressed the comment nit you had.
@mstorsjo  I updated this to also handle thumb on arm.

When doing that I noticed there is something really strange about the existing macro defines. I assume they should only be defined when exceptions is enabled.
This is by default in c++ mode of with -fexceptions in c mode.
I moved the defines within a check of exceptions being enabled to address that.

Apple seem to do their own thing for watchOS by just force enabling it for that target only.
Some input there would help a lot.

Also if you think we should define the macros even when exceptions are disabled could you explain why?


Repository:
  rL LLVM

https://reviews.llvm.org/D39673





More information about the cfe-commits mailing list