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

[Amy Kwan via Phabricator via llvm-commits]

amyk added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp:708
+  // If we don't do this it comes out as Global.
+  if (MO.getTargetFlags() & PPCII::MO_TLSGDM_FLAG ||
+      MO.getTargetFlags() & PPCII::MO_TLSGD_FLAG)
----------------
Do we not care about the other TLS related target flags (that I suppose are mainly used for Linux)?


================
Comment at: llvm/test/CodeGen/PowerPC/ppc-TOC-stats.ll:13
+
+; The puropose of this test is to check that the statistics about the TOC are
+; are collected correctly. This test tries to have all of the different types 
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146325