[PATCH] D46084: [Fixed Point Arithmetic] Addition of the Fixed Point _Accum type

Leonard Chan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 25 11:23:48 PDT 2018

leonardchan marked 2 inline comments as done.
leonardchan added inline comments.

Comment at: include/clang/Driver/Options.td:882
+def enable_fixed_point : Flag<["-", "--"], "enable-fixed-point">, Group<fixed_point_Group>,
+                         Flags<[CC1Option]>, HelpText<"Enable fixed point types">;
ebevhan wrote:
> Shouldn't this be an `-f` flag, like `-ffixed-point`? I'm not for or against either, just wondering what anyone else thinks.
Makes sense since this flag is target independent.

  rC Clang


More information about the cfe-commits mailing list