-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[CIR] updates test for 75ea7aed #127917
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
[CIR] updates test for 75ea7aed #127917
Conversation
@llvm/pr-subscribers-clang @llvm/pr-subscribers-clangir Author: Christopher Di Bella (cjdb) ChangesFull diff: https://github.com/llvm/llvm-project/pull/127917.diff 1 Files Affected:
diff --git a/clang/test/CIR/emit-actions.cpp b/clang/test/CIR/emit-actions.cpp
index 94ddf23b34753..969c10a1fc2bc 100644
--- a/clang/test/CIR/emit-actions.cpp
+++ b/clang/test/CIR/emit-actions.cpp
@@ -11,7 +11,7 @@
int x = 1;
-// BC: @x = dso_local global i32 1
+// BC: @x = global i32 1, align 4
// ASM: x:
// ASM: .long 1
|
clang/test/CIR/emit-actions.cpp
Outdated
@@ -11,7 +11,7 @@ | |||
|
|||
int x = 1; | |||
|
|||
// BC: @x = dso_local global i32 1 | |||
// BC: @x = global i32 1, align 4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't get any buildbot notifications about this test failing. Where are you seeing this failure?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I guess I wouldn't see buildbot failures, since we don't have a buildbot with ClangIR enabled. I haven't seen it fail locally though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is happening when I run the tests inside Google's infra. I've updated the commit so that it works both inside and out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Thanks! |
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/55/builds/7301 Here is the relevant piece of the build log for the reference
|
This reverts commit 131a3cf.
No description provided.