[all-commits] [llvm/llvm-project] 9503ad: [clang] FatalErrorHandler.cpp - add explicit <stdi...

[Simon Pilgrim via All-commits]

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

  Changed paths:
    M clang/tools/libclang/FatalErrorHandler.cpp

  Log Message:
  -----------
  [clang] FatalErrorHandler.cpp - add explicit <stdio.h> include

Required for fprintf/stderr usage in the error handler, noticed while trying to remove the <string> dependency described in D111049


  Commit: 3ca232feb3526281bf9640f5f6f21e8f4c622b28
      https://github.com/llvm/llvm-project/commit/3ca232feb3526281bf9640f5f6f21e8f4c622b28
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-10-05 (Tue, 05 Oct 2021)

  Changed paths:
    M llvm/utils/TableGen/CodeEmitterGen.cpp

  Log Message:
  -----------
  [TableGen] CodeEmitterGen - emit report_fatal_error(const char*) instead of report_fatal_error(std::string&)

As described on D111049, we're trying to remove the <string> dependency from error handling. In most cases the plan is to use the Twine() variant directly but to reduce introducing additional headers for the generated files, I'm using the const char* variant here instead.


  Commit: d67935ed8e78e7ffef17b6f7193b4f11f52a143d
      https://github.com/llvm/llvm-project/commit/d67935ed8e78e7ffef17b6f7193b4f11f52a143d
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-10-05 (Tue, 05 Oct 2021)

  Changed paths:
    M llvm/lib/Support/SmallVector.cpp

  Log Message:
  -----------
  [Support] Update SmallVector report_fatal_error calls to use Twine and add missing implicit header dependency.


Compare: https://github.com/llvm/llvm-project/compare/cc697fc292b0...d67935ed8e78