[clang] [clang] Set FPOptions at the beginning of CompoundStmt (PR #111654)

Serge Pavlov via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 11 07:32:19 PST 2024


================
@@ -1684,6 +1684,15 @@ class CompoundStmt final
     return hasStoredFPFeatures() ? getStoredFPFeatures() : FPOptionsOverride();
   }
 
+  /// Get FPOptions inside this statement. They may differ from the outer
+  /// options due to pragmas.
+  /// \param CurFPOptions FPOptions outside this statement.
+  FPOptions getInsideFPOptions(FPOptions CurFPOptions) const {
----------------
spavloff wrote:

I renamed it to `getActiveFPOptions`. If you have ideas how to make it better, please share.

A name that refers to `StoredFPFeatures` is a bit misleading, because AST nodes already have method `getStoredFPFeatures`, which reports the stored FP features in the form of `FPOptionsOverride`. 

https://github.com/llvm/llvm-project/pull/111654


More information about the cfe-commits mailing list