aboutsummaryrefslogtreecommitdiff
path: root/clang/lib/Basic/SourceManager.cpp
diff options
context:
space:
mode:
authorYounan Zhang <zyn7109@gmail.com>2024-08-30 16:29:18 +0800
committerGitHub <noreply@github.com>2024-08-30 16:29:18 +0800
commit0722b8ab8168d9e1aa3413a62c65878f407225ae (patch)
tree147c600330ffa168dd7e2197b3e4cedbe285f168 /clang/lib/Basic/SourceManager.cpp
parentd6dc7cf71bf52f866c092e92ce374f0266ebee1a (diff)
downloadllvm-0722b8ab8168d9e1aa3413a62c65878f407225ae.zip
llvm-0722b8ab8168d9e1aa3413a62c65878f407225ae.tar.gz
llvm-0722b8ab8168d9e1aa3413a62c65878f407225ae.tar.bz2
[Clang][NFC] Consolidate tests for default argument substitution (#105617)
Follow-up on 8ac140f39. The test `SemaTemplate/default-parm-init.cpp` was introduced since the fix #80288 and mainly did the following things: - Ensure the default arguments are properly substituted inside either the primary template & their explicit / out-of-line specializations. - Ensure the strategy doesn't mess up the substitution of a lambda expression as a default argument. The 1st is for the bug of #68490, yet it does some redundant work: each of the member functions is duplicated twice for the `sizeof` and `alignof` operators, respectively, and the principle under the hood are essentially the same. So this patch removes the duplication and reduces the 8 functions to 4 functions that reveal the same thing. The 2nd is presumably testing that the fix in #80288 doesn't impact a complicated substitution. However, that seems unnecessary & unrelated to the original issue. And more importantly, we don't have any problems with that ever. Hence, I'll remove that test from this patch. The test for default arguments is merged into `SemaTemplate/default-arguments.cpp` with a new namespace, and hopefully this could reduce the entropy of our testing cases.
Diffstat (limited to 'clang/lib/Basic/SourceManager.cpp')
0 files changed, 0 insertions, 0 deletions