This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Follow up on D87111 - Add an option for unrolling loops up to a factor - CR issues addressed.
ClosedPublic

Authored by llitchev on Sep 8 2020, 11:43 AM.

Details

Summary

Addressed some CR issues pointed out in D87111. Formatting and other nits.
The original Diff D87111 - Add an option for unrolling loops up to a factor.

Diff Detail

Event Timeline

llitchev created this revision.Sep 8 2020, 11:43 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 8 2020, 11:43 AM
llitchev requested review of this revision.Sep 8 2020, 11:43 AM
llitchev updated this revision to Diff 290558.Sep 8 2020, 11:53 AM

Some more formatting and squashing commits.

bondhugula requested changes to this revision.EditedSep 9 2020, 10:28 PM

Please mark the commit 'NFC' (in the title). Please also include the previous commit title in the commit summary and a part of it in the current title.

This revision now requires changes to proceed.Sep 9 2020, 10:28 PM
llitchev updated this revision to Diff 291027.Sep 10 2020, 10:42 AM

Fixed couple of typos and Diff title/summary issues

llitchev retitled this revision from Follow up for D87111 - CR issues addressed. to [NFC] Follow up on D87111 - Add an option for unrolling loops up to a factor - CR issues addressed..Sep 10 2020, 10:46 AM
llitchev edited the summary of this revision. (Show Details)

Updated the Title and Summary of the diff as requested.

bondhugula accepted this revision.Sep 10 2020, 10:45 PM
This revision is now accepted and ready to land.Sep 10 2020, 10:45 PM