Skip to content

[CIR][CodeGen] supports struct copy from function call result #369

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
Merged
merged 2 commits into from
Jan 4, 2024
Merged

[CIR][CodeGen] supports struct copy from function call result #369

merged 2 commits into from
Jan 4, 2024

Conversation

gitoleg
Copy link
Collaborator

@gitoleg gitoleg commented Dec 28, 2023

This PR fixes the next case

typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}

i.e. when a struct is assigned to a function call result

bcardosolopes
bcardosolopes approved these changes Jan 4, 2024
View reviewed changes
Copy link
Member

@bcardosolopes bcardosolopes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bcardosolopes bcardosolopes merged commit 71f420e into llvm:main Jan 4, 2024
lanza pushed a commit that referenced this pull request Jan 29, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (#369)
4faaa43 
This PR fixes the next case 
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
lanza pushed a commit that referenced this pull request Mar 23, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (#369)
40ec3a5 
This PR fixes the next case 
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
eZWALT pushed a commit to eZWALT/clangir that referenced this pull request Mar 24, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (llvm#369)
5262dda 
This PR fixes the next case 
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
eZWALT pushed a commit to eZWALT/clangir that referenced this pull request Mar 24, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (llvm#369)
75b9647 
This PR fixes the next case 
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
lanza pushed a commit that referenced this pull request Apr 29, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (#369)
3943bf8 
This PR fixes the next case 
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
lanza pushed a commit that referenced this pull request Apr 29, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (#369)
8d697bd 
This PR fixes the next case 
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
eZWALT pushed a commit to eZWALT/clangir that referenced this pull request Apr 29, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (llvm#369)
fc5968e 
This PR fixes the next case 
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
lanza pushed a commit that referenced this pull request Apr 29, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (#369)
b21e81e 
This PR fixes the next case
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
bruteforceboy pushed a commit to bruteforceboy/clangir that referenced this pull request Oct 2, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (llvm#369)
6d6fe37 
This PR fixes the next case
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
Hugobros3 pushed a commit to shady-gang/clangir that referenced this pull request Oct 2, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (llvm#369)
51f1fb0 
This PR fixes the next case
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
keryell pushed a commit to keryell/clangir that referenced this pull request Oct 19, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (llvm#369)
5b01b86 
This PR fixes the next case
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
lanza pushed a commit that referenced this pull request Nov 5, 2024
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (#369)
34dbeee 
This PR fixes the next case
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
lanza pushed a commit that referenced this pull request Mar 18, 2025
@gitoleg @lanza
[CIR][CIRGen] supports struct copy from function call result (#369)
0855135 
This PR fixes the next case
```
typedef struct { } A;

A create() { A a; return a; }

void foo() {
    A a;
    a = create();
}
```
i.e. when a struct  is assigned to a function call result
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bcardosolopes bcardosolopes bcardosolopes approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gitoleg @bcardosolopes