[PATCH] D121848: [scudo] Remove unused header includes and fix declarations

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka added inline comments.


================
Comment at: compiler-rt/lib/scudo/standalone/vector.h:14
 
-#include <string.h>
-
----------------
ddcc wrote:
> vitalybuka wrote:
> > it's used on :46
> `common.h` includes `string.h` already, so this additional include is unnecessary. We have some custom library paths in our build system, so I needed to go through each include.
Unless there is a strong reason to do so, I am for IWYU.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121848