In D94619#2672518, @lebedev.ri wrote:

Could use an IR test. Is only CoroFrame.cpp affected?

I am going to commit this change since no one else seems to be reviewing it... will welcome any post commit reviews.

LGTM

Ping!

Ping

ping

Address comments

Address Comments

Updates based on feedback.

Update ppc64-sibcall-shrinkwrap.ll to currently fail since tail calling is not implemented yet.

Address Comments, remove redundant checks.

Remove trailing whitespace

Ping

Ping

Updates based on feedback

Update the test case such that it is not specific to a target.

Update test case as per feedback

Ping!

Minor updates from the previous diff

Forgot to add testcase

Ping

Add test cases for 32 and 64 bit AIX

Ping!

Made a small error in the previous diff, updated the minor typo.

Pass in the BBName Parameter to the critical edge case as well.

Hey @sckmodifier, please take a look at some comments after that one you just quoted. I added the NFC tag, but then in later discussion, it was determined it is best to remove the NFC tag since this change is in somewhat in a grey area.

Ping!

Missed one comment from Bardia, address that comment.

Address review comments

Include all of the includes from before, and update patch based on changes in D82927

Address the ASAN build failures (major change is around line 795 in LoopFuse.cpp file). Also address some issues @MaskRay pointed out in lit tests.

Address Florian 's comments.

Address Bardia's comments

Ping @ reviewers! Thanks!

Ping @ reviewers! Thanks!

Address review comments

Address Bardia's comments

Minor update after making some modifications to D80580

Applying the patch in its previous state resulted in build failures. Updated patch to resolve all build failures. Major change is in the file llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp in the gatherPeelingPreferences function.

Address Bardia's comments.

Address Bardia's comments, major change is removing the loop-fusion-peel option.

Address @Meinersbur comments. Remove the Loop Peeling files since it is unrelated to fusion. The Loop Peeling changes revision is here: D83056

@fhahn added the followup patch D82927. Also ping at reviewers of this patch! Thanks!

Some NFC modifications.

Update function description comments based on review.

Updated comments based on review.

Updated the code based on review.

@fhahn I think a follow-up patch will take a little bit of time to develop so here is a quick example.

for (i = 0; i < 10; ++i)
  a[i] = a[i] + 3;
for (j = 1; j < 10; ++j)
  b[j] = b[j] + 5;

In D80580#2055720, @fhahn wrote:

Could you elaborate why this change is desired? Is there a problem with getting the unrolling preferences and checking the peeling properties in that way? If that's the case it would be good to provide some context (e.g. in form of a follow-on patch that uses the new PeelingPreferences).