[PATCH] D78672: [Debuginfo] Remove redundand variable from getAttributeValue()

Greg Clayton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 23 12:28:31 PDT 2020


clayborg accepted this revision.
clayborg added a comment.

Yes, much cleaner! I look forward to seeing any perf optimizations in the future if they prove to be faster. It really depends on how many workflows directly access attributes. It is probably quite common for symbolication where it will look for low and high pc and name.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78672





More information about the llvm-commits mailing list