[PATCH] D117778: [OpaquePtrs] Add -normalize-opaque-pointers option

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

Any further thoughts on this? Some options I see:

1. Add `-normalize-opaque-pointers` to existing RUN lines. This allows them to pass when opaque pointers are enabled, but doesn't test opaque pointers in a default configuration.
2. Add both `-opaque-pointers=1` and `-opaque-pointers=0 -normalize-opaque-pointers` to RUN lines. This makes everyone test both typed and opaque pointers in a default configuration.
3. Add both `-opaque-pointers=1` and `-opaque-pointers=0` to RUN lines with different FileCheck prefixes.
4. Add `-opaque-pointers` to RUN lines and stop testing typed pointers.
5. ???

I think all of these are principally viable, with different tradeoffs. This patch currently proposed version 1.


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

https://reviews.llvm.org/D117778