[all-commits] [llvm/llvm-project] e35d50: [ADT] APInt.cpp - remove <cstring> duplicate. NFC.

[Simon Pilgrim via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e35d50b7f2938a274ec591c3ba34fcffd3613f11
      https://github.com/llvm/llvm-project/commit/e35d50b7f2938a274ec591c3ba34fcffd3613f11
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2022-10-23 (Sun, 23 Oct 2022)

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

  Log Message:
  -----------
  [ADT] APInt.cpp - remove <cstring> duplicate. NFC.

This is already included in APInt.h


  Commit: 016c83047f6ffdca8e83176733e5777176702e79
      https://github.com/llvm/llvm-project/commit/016c83047f6ffdca8e83176733e5777176702e79
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2022-10-23 (Sun, 23 Oct 2022)

  Changed paths:
    M llvm/include/llvm/ADT/APInt.h

  Log Message:
  -----------
  [ADT] APInt.h - remove <cmath> include. NFC.

We only need this for std::abs, but since we're also testing the sign of the same value, then its not really necessary.

As detailed in https://commondatastorage.googleapis.com/chromium-browser-clang/llvm-include-analysis.html - APInt.h is the generic header with the highest expanded size, due to the dependency on <cmath>


Compare: https://github.com/llvm/llvm-project/compare/6e920e8881f0...016c83047f6f