rjmccall added inline comments.
================ Comment at: include/clang/Basic/AlignedExceptionObject.h:31 + case llvm::Triple::MacOSX: // Earliest supporting version is 10.14. + return llvm::VersionTuple(10U, 14U); + case llvm::Triple::IOS: ---------------- rjmccall wrote: > ldionne wrote: > > Would it make more sense to return the alignment directly here? > I agree: this should be a function returning the minimum expected exception > alignment on the current target. Also, why does this have its own header > instead of being, say, a function on `TargetInfo`? Or `ASTContext` or something similar. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61667/new/ https://reviews.llvm.org/D61667 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
