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

[Mark de Wever via Phabricator via cfe-commits]

Mordante added a comment.

In D139986#4001180 <https://reviews.llvm.org/D139986#4001180>, @Michael137 wrote:

> Missed couple of test cases in libcxx
> About to fix those

There were more breakage due to this patch, which I fixed in D140272 <https://reviews.llvm.org/D140272>.

Next time please don't commit patches when the pre-commit CI is red. This build https://buildkite.com/llvm-project/libcxx-ci/builds/16118 shows the patch will break libc++. These pre-commit CI jobs were added specifically to aid the Clang developers to validate whether their changes break libc++. Libc++ heavily relies on its pre-commit CI so breaking the CI has a huge impact.
Note that when libc++ breaks there might be other projects that use the latest Clang HEAD that will be affected too. (Not likely with diagnostics, but likely when the modular build fails.)

When you have issues resolving the libc++ issues you can always reach out to us for assistance.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139986