Skip to content

MCWin64EH uses wrong UNWIND_INFO struct memory layout #161

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.

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

Closed
vlmillet opened this issue Mar 1, 2020 · 1 comment
Closed

MCWin64EH uses wrong UNWIND_INFO struct memory layout #161

vlmillet opened this issue Mar 1, 2020 · 1 comment

Comments

@vlmillet
Copy link

vlmillet commented Mar 1, 2020

No description provided.

@vlmillet vlmillet closed this as completed Mar 1, 2020
@vlmillet
Copy link
Author

vlmillet commented Mar 1, 2020

not relevant

pysuxing pushed a commit to pysuxing/llvm-project that referenced this issue Jul 17, 2024
Before this patch, the loop operation condition block yielded either
empty or continue. This was replaced by a yield of a boolean value.

This change simplifies both codegen and lowering, while also being
semantically closer to the C language.

It also refactors loop op codegen tests to validate only the lowering
related to the cir.loop operation.

Fixes llvm#161
pysuxing pushed a commit to pysuxing/llvm-project that referenced this issue Jul 17, 2024
pysuxing pushed a commit to pysuxing/llvm-project that referenced this issue Jul 17, 2024
Before this patch, the loop operation condition block yielded either
empty or continue. This was replaced by a yield of a boolean value.

This change simplifies both codegen and lowering, while also being
semantically closer to the C language.

It also refactors loop op codegen tests to validate only the lowering
related to the cir.loop operation.

Fixes llvm#161
pysuxing pushed a commit to pysuxing/llvm-project that referenced this issue Jul 17, 2024
mjklemm pushed a commit to mjklemm/llvm-project that referenced this issue Sep 26, 2024
This patch fixes `TargetOp::getInnermostCapturedOmpOp()` to avoid detecting as
captured by the top target construct other constructs nested inside of a loop.
This prevents the `omp.target` verifier from incorrectly flagging valid SPMD
loops, like in the following example:

```f90
subroutine foo(n)
  implicit none
  integer, intent(in) :: n
  integer :: i, j

  !$omp target teams distribute parallel do
  do i=1,n
    !$omp simd
    do j=1,n
      call bar()
    enddo
  enddo
end subroutine foo
```
keryell pushed a commit to keryell/llvm-project that referenced this issue Oct 19, 2024
Before this patch, the loop operation condition block yielded either
empty or continue. This was replaced by a yield of a boolean value.

This change simplifies both codegen and lowering, while also being
semantically closer to the C language.

It also refactors loop op codegen tests to validate only the lowering
related to the cir.loop operation.

Fixes llvm#161
keryell pushed a commit to keryell/llvm-project that referenced this issue Oct 19, 2024
abidh pushed a commit to abidh/llvm-project that referenced this issue Feb 4, 2025
…tream_merge_20250116164428

merge main into amd-staging
xlauko pushed a commit to trailofbits/instafix-llvm that referenced this issue Mar 28, 2025
Before this patch, the loop operation condition block yielded either
empty or continue. This was replaced by a yield of a boolean value.

This change simplifies both codegen and lowering, while also being
semantically closer to the C language.

It also refactors loop op codegen tests to validate only the lowering
related to the cir.loop operation.

Fixes llvm#161
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant