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

Mon Jul 5 16:57:57 PDT 2021

stevewan added a comment.

In D105112#2858702 <https://reviews.llvm.org/D105112#2858702>, @thakis wrote:

> 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

Posted fix rGe2904c8e0fa901adeefe579297cb2ece2757fb18 <https://reviews.llvm.org/rGe2904c8e0fa901adeefe579297cb2ece2757fb18>. Sorry for the churn.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105112