sepavloff added inline comments.
================ Comment at: clang/lib/Sema/SemaStmt.cpp:452 + : getCurCompoundScope().FPFeatures; + FPOptionsOverride FPDiff = getCurFPFeatures().diffWith(FPO); + ---------------- rjmccall wrote: > How cheap is this? Is this something it would be worthwhile to fast-path by > maintaining a bit to check for the rare case where we have an override in a > scope? Added an optimization to `diffWith`: detect the frequent case, when FPOptions is not changed, and implement fast-path for it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123952/new/ https://reviews.llvm.org/D123952 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
