[PATCH] D153550: [DWARFVerifier] Fix verification of nameless variables.

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 22 09:37:57 PDT 2023


JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.

I can't speak for the committee, but my person interpretation of the standard is that these variables should not be included. I get that from both the order in which those points are define as well as the general spirit behind the accelerated access (i.e. looking things up by name).

It's fine to patch the verifier like this, but let's start a thread on the `dwarf-discuss` mailing list about the generation side.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153550



More information about the llvm-commits mailing list