aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM aside from a tiny naming nit. Thank you for the cleanup! ================ Comment at: clang/include/clang/Basic/TargetInfo.h:895-896 bool useObjCFPRetForRealType(FloatModeKind T) const { - assert(T <= FloatModeKind::Last && - "T value is larger than RealTypeUsesObjCFPRetMask can handle"); - return RealTypeUsesObjCFPRetMask & (1 << (int)T); + int val = llvm::BitmaskEnumDetail::Underlying(T); + return RealTypeUsesObjCFPRetMask & val; } ---------------- Alternatively, you can get rid of the local variable entirely. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128182/new/ https://reviews.llvm.org/D128182 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits