mibintc marked 4 inline comments as done. mibintc added inline comments.
================ Comment at: clang/include/clang/Basic/LangOptions.h:186 + FPM_Fast }; ---------------- rjmccall wrote: > I'm not sure I think this fusion was an improvement; the net effect was to > remove a few lines from this header and make a bunch of switches > unnecessarily non-exhaustive. I dropped the on/off enumeration and just using boolean, where needed, to show the setting, do you like this better? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78827/new/ https://reviews.llvm.org/D78827 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits