[PATCH] D122920: [Clang][CodeGen]Fix __builtin_dump_struct missing record type field name

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 28 05:56:02 PDT 2022

erichkeane added a comment.

In D122920#3479192 <https://reviews.llvm.org/D122920#3479192>, @yihanaa wrote:

>> While we'd usually be happy to take the fix-in-hand and apply it, part of the discussion on the other thread is whether to remove `__builtin_dump_struct` entirely. Because of that, I don't think we should make substantial changes in this area until it's clear we're keeping the builtin.
> Thanks for take a look, I agree to wait for the outcome of the discussion

I think I disagree with Aaron.  While, YES, we are likely to delete this in the near future, it doesn't mean we should leave it in a broken state in the meantime.  WHILE all of this is likely to be deleted, it has value until we delete it.

  rG LLVM Github Monorepo



More information about the cfe-commits mailing list