wdx727 wrote: > Thanks for the patch. I think this PR should be split up a bit, so that it's > easier to review distinct sections. In particular the areas related to > yaml2obj/obj2yaml, llvm-readobj and llvm-objdump could probably be their own > PR, where we can focus on reviewing how tools read the data.
Thanks for your advice. We will split the PR into smaller ones. However, modifying the bb_address_map section requires synchronously updating yaml2obj/obj2yaml, llvm-readobj, and llvm-objdump to ensure that the test cases pass. We may not be able to submit separate PRs for yaml2obj/obj2yaml, llvm-readobj, and llvm-objdump. https://github.com/llvm/llvm-project/pull/139008 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
