[PATCH] D155729: [OptTable] Make explicitly included options override excluded ones

Joshua Batista via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 19 11:30:42 PDT 2023


bob80905 added inline comments.


================
Comment at: clang/include/clang/Driver/Options.h:40
+  Ignored = (1 << 18),
+  TargetSpecific = (1 << 19),
 };
----------------
Given that the id for these flags have changed, does it make sense to write a test that makes sure these flags with these bits still behave as expected? (Ignored and target specific, specifically).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155729



More information about the cfe-commits mailing list