[PATCH] D131400: [CodeView] Use non-qualified names for static local variables

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 8 11:01:55 PDT 2022


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

Thanks, looks good to me.



================
Comment at: llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp:3353
     Scope = MemberDecl->getScope();
   // For Fortran, the scoping portion is elided in its name so that we can
   // reference the variable in the command line of the VS debugger.
----------------
Please update the comment, something like: "For static local variables and Fortran, the scoping..."


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131400



More information about the llvm-commits mailing list