This revision was automatically updated to reflect the committed changes.
Closed by commit rG50f24331fd91: Add diagnostic option backing field for
-fansi-escape-codes (authored by dang).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82874/new/
https
dang updated this revision to Diff 275187.
dang added a comment.
Make the tablegen description of the new option more consistent with other
flags.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82874/new/
https://reviews.llvm.org/D82874
Files:
c
dang updated this revision to Diff 274541.
dang added a comment.
Move the call to `UseANSIEscapeCodes` to `CreateFromArgs`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82874/new/
https://reviews.llvm.org/D82874
Files:
clang/include/clang/Basic/
Bigcheese accepted this revision.
Bigcheese added a comment.
This revision is now accepted and ready to land.
lgtm with the fix I mentioned.
Comment at: clang/lib/Frontend/CompilerInvocation.cpp:3629
#undef MERGE_ASSIGN_VALUE
+ llvm::sys::Process::UseANSIEscapeCodes(Diagnosti
dang created this revision.
dang added a reviewer: Bigcheese.
Herald added subscribers: cfe-commits, dexonsmith.
Herald added a project: clang.
dang edited the summary of this revision.
Keep track of -fansi-escape-codes in DiagnosticOptions and move the option to
the new option parsing system.
D