[all-commits] [llvm/llvm-project] 77db83: [clang][cli] Allow users to specify a conditional ...

[Jan Svoboda via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 77db83ae997767400ffcbe0101f8ee867ebaa111
      https://github.com/llvm/llvm-project/commit/77db83ae997767400ffcbe0101f8ee867ebaa111
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2021-01-07 (Thu, 07 Jan 2021)

  Changed paths:
    M clang/include/clang/Driver/Options.td
    M clang/lib/Frontend/CompilerInvocation.cpp
    M clang/unittests/Frontend/CompilerInvocationTest.cpp
    M llvm/include/llvm/Option/OptParser.td
    M llvm/utils/TableGen/OptParserEmitter.cpp

  Log Message:
  -----------
  [clang][cli] Allow users to specify a conditional to prevent parsing options with MarshallingInfo

Depends on D84189 & D93540.

Reviewed By: Bigcheese

Differential Revision: https://reviews.llvm.org/D84674


  Commit: c6ea4d5b2c0054f3e2fa06b911ed3933fe59bc5b
      https://github.com/llvm/llvm-project/commit/c6ea4d5b2c0054f3e2fa06b911ed3933fe59bc5b
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2021-01-07 (Thu, 07 Jan 2021)

  Changed paths:
    M clang/unittests/Frontend/CompilerInvocationTest.cpp

  Log Message:
  -----------
  [clang][cli] Implement ContainsN Google Test matcher

This allows us to verify that we don't emit options multiple times.

In most cases, that would be benign, but for options with `MarshallingInfoVectorString`, emitting wrong number of arguments might change the semantics.

Reviewed By: Bigcheese

Differential Revision: https://reviews.llvm.org/D93636


Compare: https://github.com/llvm/llvm-project/compare/5471b1fa4013...c6ea4d5b2c00