[PATCH] D94474: [WIP][clang][cli] Test manual header search argument generation with round-trip

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 12 10:39:22 PST 2021


dexonsmith added a comment.

I'm wondering if we can get this in incrementally without needing to list in code which options are correctly handled. Here's one way to do it:

- Add a tool, `clang-cc1-args`, that shows the diff between round-tripping when given a set of args. (This could potentially grow other functionality vs. diffing. So maybe `clang-cc1-args diff [args]` would be the usage?)
- Add `RUN` lines to tests that test the args we think should round-trip correctly and confirm there's no diff. This can be done incrementally as options get handled.
- Only once we think we have all the options covered, add the assertion from this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94474



More information about the cfe-commits mailing list