https://github.com/hokein closed https://github.com/llvm/llvm-project/pull/79296
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hokein edited https://github.com/llvm/llvm-project/pull/79296
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hokein wrote:
> Sorry for missing this!
>
> Would be nice to have a testcase but I see it's hard to get into the crashing
> state (maybe even not possible via clang). Seems worthwhile to be robust to
> such conditions and getDefaultArgRange appears less fragile.
This fix was needed for the in
https://github.com/sam-mccall approved this pull request.
Sorry for missing this!
Would be nice to have a testcase but I see it's hard to get into the crashing
state (maybe even not possible via clang).
Seems worthwhile to be robust to such conditions and getDefaultArgRange appears
less fragil
https://github.com/hokein edited https://github.com/llvm/llvm-project/pull/79296
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Haojian Wu (hokein)
Changes
source location in AliasTemplateDeductionGuideTransform.
I don't have a reproducible testcase, but this should be a safe and
non-functional change. We have checked the `hasDefaultArg` before calling
getDefault
https://github.com/hokein created
https://github.com/llvm/llvm-project/pull/79296
source location in AliasTemplateDeductionGuideTransform.
I don't have a reproducible testcase, but this should be a safe and
non-functional change. We have checked the `hasDefaultArg` before calling
getDefaultAr