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.

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

[LICM]The store instruction is prevented from #69413

Closed
wants to merge 1 commit into from

Conversation

XingYuShuai
Copy link

@XingYuShuai XingYuShuai commented Oct 18, 2023

The store instruction is prevented from being mentioned outside the loop in some cases.

Please refer to issue:68776 for details.

Fixes #68776
Fixes #69279

@github-actions
Copy link

github-actions bot commented Oct 18, 2023

✅ With the latest revision this PR passed the C/C++ code formatter.

@XingYuShuai XingYuShuai force-pushed the fix-signal-before-loop branch 2 times, most recently from 87fd576 to 3aee316 Compare October 18, 2023 08:27
Copy link
Contributor

@nikic nikic left a comment

Choose a reason for hiding this comment

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

As I have already explained to you, your code is not well-defined. There is nothing to fix in LLVM, you need to fix your code to use an atomic variable instead.

being mentioned outside the loop in some cases.

Please refer to issue:68776 for details.

Fixes llvm#68776
Fixes llvm#69279
@XingYuShuai
Copy link
Author

XingYuShuai commented Oct 20, 2023

As I have already explained to you, your code is not well-defined. There is nothing to fix in LLVM, you need to fix your code to use an atomic variable instead.

@nikic

When Run_Index is atomic, there is no problem with open source version. For details about the test cases and results, see:#68776 (comment)

And I also added my patch and retested the above test cases to keep consistent with the original version.
image

Does that explain your problem? If you still have a problem, can you describe your problem in more detail? For example, in the test-code2.txt test case I provided, which variable is atomic? Thank you very much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants