SuccessChanges

Summary

  1. [clang][cli] Remove marshalling from Opt{In,Out}FFlag (details)
  2. [clangd] NFC, remove an extra "class" keyword. (details)
Commit d8a8e5d6240a1db809cd95106910358e69bbf299 by Jan Svoboda
[clang][cli] Remove marshalling from Opt{In,Out}FFlag

We can now express all marshalling semantics in `Opt{In,Out}FFlag` via `BoolFOption`.

This patch moves remaining `Opt{In,Out}FFlag` instances using marshalling to `BoolFOption` and removes marshalling capabilities from `Opt{In,Out}FFlag` entirely.

This simplifies the decisions developers have to make when creating new boolean options:
  * For simple cc1 flag pairs, use `Bool{,F,G}Option`.
  * For cc1 flag pairs that require complex marshalling logic, use `Opt{In,Out}FFlag` and implement marshalling manually.

Reviewed By: dexonsmith

Differential Revision: https://reviews.llvm.org/D97370
The file was modifiedclang/include/clang/Driver/Options.td (diff)
The file was modifiedclang/lib/Frontend/CompilerInvocation.cpp (diff)
Commit b218f7c4baada3d22bf22bd3ca6e849bb2ce9289 by hokein.wu
[clangd] NFC, remove an extra "class" keyword.
The file was modifiedclang-tools-extra/clangd/ClangdLSPServer.cpp (diff)