Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

C++ translation improvements #1482

Merged
merged 9 commits into from
Apr 24, 2024
Merged

C++ translation improvements #1482

merged 9 commits into from
Apr 24, 2024

Conversation

oxisto
Copy link
Member

@oxisto oxisto commented Mar 29, 2024 •
edited
Loading

This PR contains several C++ improvements, such as:

  • Better support for typedefs in classes (we can resolve partial FQNs now, but a real generalised solution for this is needed, see Discussion on types and type name normalisation #1533, in the long run we will handle typedefs differently anyway)
  • Support for type aliases (using myString = std::string)
  • Support for namespace aliases (namespace estd :: std)
  • Detection of const T& == T in function overloading
  • Better realisation of template types, e.g. also pointer-types of T (e.g. T*) are now realised to their concrete types (e.g. int*, if T is int)

Fixes #1481
Fixes #1489
Fixes #1483

@oxisto oxisto changed the title improve cxx C++ translation improvements Mar 29, 2024
@oxisto oxisto added the cxx Related to C/C++ language parsing label Mar 29, 2024
@oxisto oxisto linked an issue Mar 29, 2024 that may be closed by this pull request
Move connecting of declaration and definition to C++ extra pass #1484
Closed
@oxisto oxisto force-pushed the improve-cxx branch 2 times, most recently from 82f0743 to 159cf7b Compare March 31, 2024 21:29
@oxisto oxisto linked an issue Apr 1, 2024 that may be closed by this pull request
Support typedef in class definition #1489
Closed
@oxisto oxisto force-pushed the improve-cxx branch 11 times, most recently from dfcaaf9 to bb09216 Compare April 5, 2024 16:31
@oxisto oxisto changed the base branch from main to next-gen-call-resolver April 5, 2024 16:32
@oxisto oxisto force-pushed the improve-cxx branch 3 times, most recently from 812ecfb to 16e13e6 Compare April 9, 2024 10:51
Base automatically changed from next-gen-call-resolver to main April 12, 2024 07:47
@oxisto oxisto closed this Apr 12, 2024
@oxisto oxisto reopened this Apr 12, 2024
@oxisto oxisto force-pushed the improve-cxx branch 2 times, most recently from d98ade2 to 4d4202b Compare April 12, 2024 20:30
@oxisto
Trying to use a release-independent repository for CDT
3a89308 
This is some rather ugly workaround in our ivy repository layout, but it should work for all future releases.
@oxisto oxisto marked this pull request as ready for review April 13, 2024 15:14
@oxisto oxisto force-pushed the improve-cxx branch 2 times, most recently from c5a2ea8 to 24dbf47 Compare April 15, 2024 18:23
@oxisto oxisto requested a review from konradweiss April 23, 2024 21:23
@oxisto oxisto enabled auto-merge (squash) April 24, 2024 06:46
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Apr 24, 2024

Quality Gate Passed Quality Gate passed

Issues
6 New issues
0 Accepted issues

Measures
0 Security Hotspots
76.1% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@oxisto oxisto merged commit fa9b204 into main Apr 24, 2024
3 checks passed
@oxisto oxisto deleted the improve-cxx branch April 24, 2024 06:51
maximiliankaul pushed a commit that referenced this pull request Apr 30, 2024
@oxisto @maximiliankaul
C++ translation improvements (#1482)
7db0e91
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@konradweiss konradweiss konradweiss approved these changes

@peckto peckto Awaiting requested review from peckto peckto is a code owner

@KuechA KuechA Awaiting requested review from KuechA KuechA is a code owner

Assignees
No one assigned
Labels
cxx Related to C/C++ language parsing
Projects
None yet
Milestone
No milestone
2 participants
@oxisto @konradweiss