[llvm-commits] [llvm] r168646 - in /llvm/trunk: include/llvm/Bitcode/LLVMBitCodes.h lib/Bitcode/Reader/BitcodeReader.cpp lib/Bitcode/Writer/BitcodeWriter.cpp

[Owen Anderson]

On Nov 26, 2012, at 4:43 PM, Michael Ilseman <milseman at apple.com> wrote:

> +  /// Flags for serializing FPMathOperator's
> +  /// SubclassOptionalData contents.
> +  enum FastMathFlags {
> +    FMF_UNSAFE_ALGEBRA   = 0,
> +    FMF_NO_NANS          = 1,
> +    FMF_NO_INFS          = 2,
> +    FMF_NO_SIGNED_ZEROS  = 3,
> +    FMF_ALLOW_RECIPROCAL = 4
> +  };

Is there a reason this enum isn't the same as your other fast math flag enums?  If there is a reason it needs to differ (because of encoding constraints?), please give it a name that doesn't clash.

--Owen