[PATCH] D146325: [PowerPC] Add statistics to show the number of entries in the TOC.

[Stefan Pintilie via Phabricator via llvm-commits]

stefanp added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp:2900
+    MCSymbol *TOCEntry =
+        lookUpOrCreateTOCEntry(TypeInfoSym, TOCType_GlobalInternal);
     const MCSymbol *TOCBaseSym =
----------------
amyk wrote:
> I might be missing something but how do we know that the type will be `TOCType_GlobalInternal`?
> I realized I am not quite sure what the `TType` means. 
The `TType` that we emit here is used for Exception Handling on AIX.
I had originally assumed that exception handling could only be local but after you asked the question I looked into it some more and realized that I was wrong. Therefore, I've fixed this code to check if this is an internal or external symbol.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146325