[all-commits] [llvm/llvm-project] 2e5daa: [llvm] Update report_fatal_error calls from raw_st...

Simon Pilgrim via All-commits all-commits at lists.llvm.org
Tue Oct 5 10:42:56 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2e5daac21731eb27ef952efaad31cac2a5d8f254
      https://github.com/llvm/llvm-project/commit/2e5daac21731eb27ef952efaad31cac2a5d8f254
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-10-05 (Tue, 05 Oct 2021)

  Changed paths:
    M llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
    M llvm/lib/CodeGen/AsmPrinter/AsmPrinterInlineAsm.cpp
    M llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
    M llvm/lib/ExecutionEngine/MCJIT/MCJIT.cpp
    M llvm/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldELF.cpp
    M llvm/lib/ExecutionEngine/RuntimeDyld/Targets/RuntimeDyldCOFFThumb.h
    M llvm/lib/Object/Object.cpp
    M llvm/lib/Target/Hexagon/MCTargetDesc/HexagonMCCodeEmitter.cpp
    M llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp
    M llvm/tools/llvm-readobj/ARMWinEHPrinter.cpp
    M llvm/tools/obj2yaml/coff2yaml.cpp
    M llvm/unittests/Analysis/TargetLibraryInfoTest.cpp
    M llvm/unittests/Analysis/VectorUtilsTest.cpp

  Log Message:
  -----------
  [llvm] Update report_fatal_error calls from raw_string_ostream to use Twine(OS.str())

As described on D111049, we're trying to remove the <string> dependency from error handling and replace uses of report_fatal_error(const std::string&) with the Twine() variant which can be forward declared.

We can use the raw_string_ostream::str() method to perform the implicit flush() and return a reference to the std::string container that we can then wrap inside Twine().




More information about the All-commits mailing list