[PATCH] D150997: [llvm] Split out DenseMapInfo<variant> specialization

[Elliot Goodrich via Phabricator via llvm-commits]

IncludeGuardian added a comment.

The runs of IncludeGuardian for before <https://gist.github.com/IncludeGuardian/889f08026fd63775d1694874b8199c5b#file-before-yaml> and after <https://gist.github.com/IncludeGuardian/889f08026fd63775d1694874b8199c5b#file-after-yaml> this change.

This was suggested in the analysis before on this line <https://gist.github.com/IncludeGuardian/889f08026fd63775d1694874b8199c5b#file-before-yaml-L4058-L4061>. Suggesting a reduction in 1.42% preprocessing token count across all translation units in `llvm/lib` and we ended up getting 1.44%.  It's a bit higher because not only was the include directive removed, but we removed the specialization itself from the header.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150997