[PATCH] D154543: [llvm] Move StringExtras.h include from Error.h to Error.cpp

[Fangrui Song via Phabricator via llvm-commits]

MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

I have spot checked a few files and the additional #include looks great!
As mentioned in the other patch, it would be better landing the #include adjustment part separately so that the `llvm/Support/Error.h` change is more focused.
Also, there is lower risk if something breaks, either you or someone else reverts the patch, and causes unneeded churn to many files.


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

https://reviews.llvm.org/D154543