diff options
author | Adam Nemet <anemet@apple.com> | 2017-03-27 19:17:25 +0000 |
---|---|---|
committer | Adam Nemet <anemet@apple.com> | 2017-03-27 19:17:25 +0000 |
commit | 484aa45153e6fe429200034380e07966fe601003 (patch) | |
tree | 6bcc904388c8c6c8138d7d94759ea2f85b3c658f /clang/lib/Sema/SemaAttr.cpp | |
parent | 97e4f472c35d16dea12372d6ec51422216d3ad17 (diff) | |
download | bcm5719-llvm-484aa45153e6fe429200034380e07966fe601003.tar.gz bcm5719-llvm-484aa45153e6fe429200034380e07966fe601003.zip |
Encapsulate FPOptions and use it consistently
Sema holds the current FPOptions which is adjusted by 'pragma STDC
FP_CONTRACT'. This then gets propagated into expression nodes as they are
built.
This encapsulates FPOptions so that this propagation happens opaquely rather
than directly with the fp_contractable on/off bit. This allows controlled
transitioning of fp_contractable to a ternary value (off, on, fast). It will
also allow adding more fast-math flags later.
This is toward moving fp-contraction=fast from an LLVM TargetOption to a
FastMathFlag in order to fix PR25721.
Differential Revision: https://reviews.llvm.org/D31166
llvm-svn: 298877
Diffstat (limited to 'clang/lib/Sema/SemaAttr.cpp')
-rw-r--r-- | clang/lib/Sema/SemaAttr.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/clang/lib/Sema/SemaAttr.cpp b/clang/lib/Sema/SemaAttr.cpp index bad9e702426..992eac9019f 100644 --- a/clang/lib/Sema/SemaAttr.cpp +++ b/clang/lib/Sema/SemaAttr.cpp @@ -450,13 +450,13 @@ void Sema::ActOnPragmaVisibility(const IdentifierInfo* VisType, void Sema::ActOnPragmaFPContract(tok::OnOffSwitch OOS) { switch (OOS) { case tok::OOS_ON: - FPFeatures.fp_contract = 1; + FPFeatures.setFPContractable(true); break; case tok::OOS_OFF: - FPFeatures.fp_contract = 0; + FPFeatures.setFPContractable(false); break; case tok::OOS_DEFAULT: - FPFeatures.fp_contract = getLangOpts().DefaultFPContract; + FPFeatures.setFPContractable(getLangOpts().DefaultFPContract); break; } } |