[PATCH] D113653: [NVPTX][AsmPrinter] Avoid removing globals before calling AsmPrinter::doFinalization()

[Andrew Savonichev via Phabricator via llvm-commits]

asavonic added inline comments.


================
Comment at: llvm/lib/Target/NVPTX/NVPTXAsmPrinter.h:310
+
+  // Do not allow base AsmPrinter doing something with globals.
+  void emitGlobalVariable(const GlobalVariable *GV) override {}
----------------
Can we keep the original comment to explain the problem? Something like:
```Do not allow base AsmPrinter doing something with globals: global variables are emitted at the beginning and base AsmPrinter will attempt to do that again in doFinalization().```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113653