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

Elliot Goodrich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jun 18 11:01:13 PDT 2023


IncludeGuardian added a comment.
Herald added a subscriber: jsetoain.

This was found with IncludeGuardian 0.0.8.

The line that suggested this can be found in the output here <https://gist.github.com/IncludeGuardian/7e560c26783d5f11d7645c96db6b81d4#file-before-yaml-L4054-L4057>.

The count of preprocessing tokens before and after this change can be found in before.yaml <https://gist.github.com/IncludeGuardian/7e560c26783d5f11d7645c96db6b81d4#file-before-yaml-L3510> and after.yaml <https://gist.github.com/IncludeGuardian/7e560c26783d5f11d7645c96db6b81d4#file-after-yaml-L3510>. Note that both these were run on top of https://reviews.llvm.org/D150997, which is yet to make it into the main branch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153229



More information about the cfe-commits mailing list