[PATCH] D139986: [clang][TypePrinter] Teach isSubstitutedDefaultArgument about integral types

[Louis Dionne via Phabricator via cfe-commits]

ldionne added a comment.

@dblaikie

We added the libc++ tests to the Clang pre-commit CI after discussing with @erichkeane since he told me breaking libc++ was a recurring pain point, and having a way to detect that would be greatly appreciated by the Clang folks. The goal was really only to help Clang developers catch more issues earlier. I believe this confusion is only the result of miscommunication within the Clang community -- it seems that not all Clang developers know equally well what tools are available to them and what each bit of infrastructure should be used for. @erichkeane @aaron.ballman Would it make sense for you folks to post something on Discourse to explain what the expectations are w.r.t. Clang pre-commit CI?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139986