hubert.reinterpretcast added inline comments.
================
Comment at: clang/test/Lexer/cxx1z-trigraphs.cpp:24
+// notri-warning@6 {{trigraph converted}} tri-warning@6 {{trigraph converted}}
+// notri-warning@12 {{backslash and newline separated by space}}
tri-warning@12 {{backslash and newline separated by space}}
+#endif
----------------
This strategy duplicates the trigraphs-enabled behaviour checks three times.
Plain `-verify` can be used along with a macro instead (without such
duplication). This also allows the explicit `-fno-trigraphs` case to be added
with little cost.
Pseudo-code:
```
// RUN: ...
// RUN: ... -ftrigraphs -DENABLED_TRIGRAPHS=1
// RUN: ... -fno-trigraphs -DENABLED_TRIGRAPHS=0
#ifdef __MVS__
#ifndef ENABLED_TRIGRAPHS
#define ENABLED_TRIGRAPHS 1
#endif
#endif
#if !ENABLED_TRIGRAPHS
// ...
#else
// ...
#endif
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85722/new/
https://reviews.llvm.org/D85722
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
