[PATCH] D95344: [clang][cli] Accept strings instead of options in ImpliedByAnyOf

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 25 11:51:32 PST 2021


dexonsmith accepted this revision.
dexonsmith added a comment.
This revision is now accepted and ready to land.

LGTM.

> This somewhat weakens the guarantees that we're referring to an existing (option) record, but we can still use the option.KeyPath syntax to simulate this.

I agree that using `<option>.KeyPath` is sufficient to enforce ordering.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95344/new/

https://reviews.llvm.org/D95344



More information about the cfe-commits mailing list