[PATCH] D137091: [llvm-readobj] Rename JSON Flag fields to be more consistent

Hans Wennborg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 21 05:29:19 PDT 2023


hans added a comment.

This broke some Chromium builds: https://bugs.chromium.org/p/chromium/issues/detail?id=1426287

It turns out we have a tool that consumes this JSON. Maybe that one can be made to accept the encoding both before and after this change, but there may be others.

That raises the question of what the compatibility story is here. Are we okay with breaking all consumers of this JSON?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137091



More information about the llvm-commits mailing list