[PATCH] D129935: [TableGen] Add a location for a class definition that was forward-declared

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 20 06:57:05 PDT 2022


nhaehnle added a comment.

Right you are, sorry for my confusion. The patch isn't quite right because it overrides the current meaning of the Locs array (which is to track multiclass instantiations), and that can lead to confusion e.g. with `PrintMessage`. I took the liberty of making some adjustments before committing the patch (394a388d140dc9e74178532501cddb558a589398 <https://reviews.llvm.org/rG394a388d140dc9e74178532501cddb558a589398>)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129935



More information about the llvm-commits mailing list