[PATCH] D123952: [FPEnv] Allow CompoundStmt to keep FP options
John McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jun 27 08:45:22 PDT 2022
rjmccall added inline comments.
Comment at: clang/lib/Basic/LangOptions.cpp:214
+ OverrideMask |= NAME##Mask;
+ return FPOptionsOverride(*this, OverrideMask);
Hmm. If we can assume that all of the options are single bits (which this is doing), then this algorithm is basically `FPOptions::storage_type OverrideMask = (Value ^ Base.Value);`, which makes this cheap enough to be defined inline in the header.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits