tstellar added a comment.

In D123850#3453750 <https://reviews.llvm.org/D123850#3453750>, @cor3ntin wrote:

> @tstellar I wasn't sure what the procedure was for backporting fixes, so I 
> made a PR. I think your suggestion of keeping the old signature works well.
> This also modifies the serialized AST, is that an issue for backports? (If it 
> is, I'm not sure i see a way to move forward with this!)

I don' t think we have a written policy about the serialized AST format like we 
do API/ABI.  What are the use cases where maintaining the AST format is 
important?  @rsmith  Do you have any opinion about changing the serialized AST 
in a bugfix release?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123850

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D123850: [Clang] Fi... Corentin Jabot via Phabricator via cfe-commits
    • [PATCH] D123850: [Clan... Tom Stellard via Phabricator via cfe-commits

Reply via email to