[PATCH] D31167: Use FPContractModeKind universally

[Reid Kleckner via Phabricator via cfe-commits]

rnk added inline comments.


================
Comment at: cfe/trunk/include/clang/Basic/LangOptions.h:217
   /// Adjust BinaryOperator::FPFeatures to match the bit-field size of this.
-  unsigned fp_contract : 1;
+  LangOptions::FPContractModeKind fp_contract : 2;
 };
----------------
Please do not use bitfields with enum types, it's a good way to break the build on Windows. This change triggered this clang-cl warning:
```
C:\src\llvm-project\clang\include\clang/Basic/LangOptions.h(208,17):  warning: implicit truncation from 'clang::LangOptions::FPContractModeKind' to bit-field changes value from 2 to -2 [-Wbitfield-constant-conversion]
    fp_contract = LangOptions::FPC_Fast;
                ^ ~~~~~~~~~~~~~~~~~~~~~
```


Repository:
  rL LLVM

https://reviews.llvm.org/D31167