-
Notifications
You must be signed in to change notification settings - Fork 13.6k
[VPlan] Compute induction end values in VPlan. #112145
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
Changes from 1 commit
e58cb96
31c1c5f
6440a91
c6cc339
c9c38c8
4fd2d0e
be373bf
071392e
b48b097
499bc8a
ec6d6e0
5eb0cbc
18b8e8c
44a0799
3715110
8a1748a
bbdc0bb
89a1b3a
d74aca2
b19f10a
07c7874
8f701a3
378caa2
a763436
d7c28cc
4265a86
e51c6fb
4bd9416
a87d409
7897ab7
f3c6387
6c8def5
1e44bdd
5f92f65
6e860aa
edfc01b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -64,6 +64,7 @@ bool VPRecipeBase::mayWriteToMemory() const { | |
case VPInstruction::FirstOrderRecurrenceSplice: | ||
case VPInstruction::LogicalAnd: | ||
case VPInstruction::PtrAdd: | ||
case VPInstruction::ResumePhi: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Independent fix, but testable only now? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I actually had to undo this change; we have to create a ResumePhi for the canonical IV in the main plan, which won't have any users but cannot be cleaned up |
||
return false; | ||
default: | ||
return true; | ||
|
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.
Worth extending the documentation of VPDerivedIVRecipe to explain that it is useful both inside and outside the vector loop, where the canonical IV operand represents the iteration index during or following the loop, respectively.
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.
Added a comment, thanks.