[PATCH] D111971: [clang] Allocate 2 bits to store the constexpr specifier kind when serializing

Adam Czachorowski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 18 04:46:36 PDT 2021


adamcz accepted this revision.
adamcz added a comment.
This revision is now accepted and ready to land.

Fix LGTM

As for the test, I suggest putting it in clang/test/AST/, maybe in ast-dump-constant-expr? You can write a CHECK-NEXT: thingy to verify that consteval is preserved. The test already runs both with and without AST reader/writer.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111971



More information about the cfe-commits mailing list