Open lyazj opened 1 month ago
@llvm/issue-subscribers-clang-frontend
Author: 小小佳的云朵儿~ (lyazj)
Hi!
This issue may be a good introductory issue for people new to working on LLVM. If you would like to work on this issue, your first steps are:
test/
create fine-grained testing targets, so you can e.g. use make check-clang-ast
to only run Clang's AST tests.git clang-format HEAD~1
to format your changes.If you have any further questions about this issue, don't hesitate to ask via a comment in the thread below.
@llvm/issue-subscribers-good-first-issue
Author: 小小佳的云朵儿~ (lyazj)
Working on this to submit a patch with those additional test cases :)
This is just a robustness enhancement suggestion.
https://github.com/llvm/llvm-project/issues/79748
I triggered this issue and found the solution done in https://github.com/llvm/llvm-project/issues/79748. However, the test cases attached to https://github.com/agozillon/llvm-project/commit/323660ae742aecb524898f956590d599e8869f7f do not include those for reference qualifiers, in addition to cv-qualifiers. As for the completeness consideration, I suggest to add more test cases like: