[PATCH] D98400: [Test][DebugInfo] Check for backend object emission support.

Nigel Perks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 18 02:33:37 PDT 2021


nigelp-xmos added inline comments.


================
Comment at: llvm/unittests/DebugInfo/DWARF/DwarfUtils.cpp:50
 
 bool llvm::dwarf::utils::isConfigurationSupported(Triple &T) {
   initLLVMIfNeeded();
----------------
jhenderson wrote:
> Is this still in use anywhere? If not, you can delete it (and the declaration in the header).
Yes, isConfigurationSupported is used in two places that don't need to check for object emission. Thank you for the review!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98400



More information about the llvm-commits mailing list