[PATCH] D105112: [clang] Add -fdump-record-layouts-canonical option

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 5 15:33:41 PDT 2021


thakis added a comment.

Hi, you have to bump clang/include/clang/Serialization/ASTBitCodes.h's VERSION_MAJOR after adding a LangOption, else builds with LLVM_APPEND_VC_REV=NO won't invalidate their module cache correctly. See e.g. https://reviews.llvm.org/rGb8b7a9dcdcbc Without that, tests are broken on bots with  LLVM_APPEND_VC_REV=NO , eg http://45.33.8.238/linux/50460/step_7.txt


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105112



More information about the cfe-commits mailing list