-
Notifications
You must be signed in to change notification settings - Fork 396
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
Fix issue in folding the conditional in the duplicated loop #7155
Fix issue in folding the conditional in the duplicated loop #7155
Conversation
LGTM, but I've already been very involved with this fix, to the point that I've been listed as a co-author of the commit. (BTW, GitHub only recognizes the So @vijaysun-omr would you mind reviewing this one? |
36ca08b
to
88c3544
Compare
It is incorrect to both version asynch check and fold the conditional in the duplicated loop, or version write barrier and fold the conditional in the duplicated loop. For asynch check, when the loops are unbiased, do not version asynch check so that the conditional in both loops can be folded away. For write barriers, the determination of whether or not to version write barriers happens after the duplicated loop has already been added to `FoldConditional`. If the conditional in the duplicated loop will be folded, do not version write barrier. Fixes eclipse-openj9/openj9#17249 Co-Authored-By: Devin Papineau <devin@ajdmp.ca> Signed-off-by: Annabelle Huo <Annabelle.Huo@ibm.com>
88c3544
to
250f161
Compare
250f161 is updated to have the correct format |
jenkins build all |
jenkins build riscv |
|
It is incorrect to both version asynch check and fold the conditional in the duplicated loop, or version write barrier and fold the conditional in the duplicated loop.
For asynch check, when the loops are unbiased, do not version asynch check so that the conditional in both loops can be folded away.
For write barriers, the determination of whether or not to version write barriers happens after the duplicated loop has already been added to
FoldConditional
. If the conditional in the duplicated loop will be folded, do not version write barrier.Fixes eclipse-openj9/openj9#17249