Skip to content

Disable test that is failing CI #67875

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
wants to merge 1 commit into from

Conversation

tbkka
Copy link
Contributor

@tbkka tbkka commented Aug 10, 2023

Hopefully, someone can fix this soon and we can re-enable it.

/Users/ec2-user/jenkins/workspace/oss-swift_tools-RA_stdlib-DA_test-device-non_executable/swift/test/IRGen/debug_scope_distinct.swift:12:11: error: CHECK: expected string not found in input
// CHECK: call void @llvm.dbg.value(metadata {{.*}} %[[ARG0]], metadata ![[VAR1:[0-9]+]], metadata !DIExpression(DW_OP_LLVM_fragment, 0, 64)), !dbg ![[LOC1:[0-9]+]]
          ^
<stdin>:1033:42: note: scanning from here
 %.sroa.013.0.copyload = load ptr, ptr %0, align 4, !dbg !305
                                         ^
<stdin>:1033:42: note: with "ARG0" equal to "\\.sroa\\.013\\.0\\.copyload"
 %.sroa.013.0.copyload = load ptr, ptr %0, align 4, !dbg !305
                                         ^
<stdin>:1040:2: note: possible intended match here
 call void @llvm.dbg.value(metadata i64 %4, metadata !299, metadata !DIExpression(DW_OP_LLVM_fragment, 40, 64)), !dbg !306
 ^

The input shows that line 1034 is this:

         1034:  call void @llvm.dbg.value(metadata ptr %.sroa.013.0.copyload, metadata !299, metadata !DIExpression(DW_OP_LLVM_fragment, 0, 32)), !dbg !306

which differs in 32 vs. 64. So I wonder if this is really an issue with integer size?

Link to a failing test:

https://ci.swift.org/job/oss-swift_tools-RA_stdlib-DA_test-device-non_executable/4234/

Hopefully, someone can fix this soon and we can re-enable it.

```
/Users/ec2-user/jenkins/workspace/oss-swift_tools-RA_stdlib-DA_test-device-non_executable/swift/test/IRGen/debug_scope_distinct.swift:12:11: error: CHECK: expected string not found in input
// CHECK: call void @llvm.dbg.value(metadata {{.*}} %[[ARG0]], metadata ![[VAR1:[0-9]+]], metadata !DIExpression(DW_OP_LLVM_fragment, 0, 64)), !dbg ![[LOC1:[0-9]+]]
          ^
<stdin>:1033:42: note: scanning from here
 %.sroa.013.0.copyload = load ptr, ptr %0, align 4, !dbg !305
                                         ^
<stdin>:1033:42: note: with "ARG0" equal to "\\.sroa\\.013\\.0\\.copyload"
 %.sroa.013.0.copyload = load ptr, ptr %0, align 4, !dbg !305
                                         ^
<stdin>:1040:2: note: possible intended match here
 call void @llvm.dbg.value(metadata i64 %4, metadata !299, metadata !DIExpression(DW_OP_LLVM_fragment, 40, 64)), !dbg !306
 ^
```

The input shows that line 1034 is this:
```
         1034:  call void @llvm.dbg.value(metadata ptr %.sroa.013.0.copyload, metadata !299, metadata !DIExpression(DW_OP_LLVM_fragment, 0, 32)), !dbg !306
```
which differs in 32 vs. 64.  So I wonder if this is really an issue with integer size?

Link to a failing test:

https://ci.swift.org/job/oss-swift_tools-RA_stdlib-DA_test-device-non_executable/4234/
@tbkka
Copy link
Contributor Author

tbkka commented Aug 10, 2023

CC: @asavonic

@tbkka
Copy link
Contributor Author

tbkka commented Aug 10, 2023

Test was introduced in #66744 and was recently updated in #67699

@tbkka tbkka requested a review from ahoppen August 10, 2023 20:17
@tbkka
Copy link
Contributor Author

tbkka commented Aug 10, 2023

@swift-ci Please test

@asavonic
Copy link
Contributor

@tbkka, can you please confirm that the test still fails after #67699? The link above shows the test failing for watchos-armv7k which should be disabled by REQUIRES: CPU=arm64 || CPU=x86_64 || CPU=arm64e.

@tbkka
Copy link
Contributor Author

tbkka commented Aug 14, 2023

Looks like it's passing after #67699, so closing this.

@tbkka tbkka closed this Aug 14, 2023
@tbkka tbkka deleted the tbkka-disable-failing-test branch August 1, 2024 16:37
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

Successfully merging this pull request may close these issues.

2 participants