This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC] [NFC] rename PPCLoopPreIncPrep.cpp to PPCLoopInstrFormPrep.cpp after D67088
AbandonedPublic

Authored by shchenz on Nov 25 2019, 6:50 PM.

Details

Reviewers
jsji
nemanjai
steven.zhang
Group Reviewers
Restricted Project
Summary

This is NFC code clean work after D67088. In that patch, we extend loop instructions prep for ds/dq form.

This patch only changes the file name PPCLoopPreIncPrep.cpp to PPCLoopInstrFormPrep.cpp for better reviewing of the content change of file PPCLoopInstrFormPrep.cpp.

Diff Detail

Event Timeline

shchenz created this revision.Nov 25 2019, 6:50 PM
jsji requested changes to this revision.EditedNov 26 2019, 7:10 AM

No sure how you create the patch and review, but the patch still DOES NOT look right. So I create https://reviews.llvm.org/D70716, you can see that the diff is showing as move V there now.

Please review it there. Thanks.

This revision now requires changes to proceed.Nov 26 2019, 7:10 AM
shchenz abandoned this revision.Nov 26 2019, 4:03 PM

is it caused by the way I sent out the review? I git mv the file, git diff to get diff patch and upload the diff patch here. Not sure which step is wrong. I will abandon this patch.

jsji added a comment.Nov 26 2019, 7:23 PM

is it caused by the way I sent out the review? I git mv the file, git diff to get diff patch and upload the diff patch here. Not sure which step is wrong. I will abandon this patch.

I believe it is git diff you use that cause problem. You should use git format-patch.

llvm/lib/Target/PowerPC/PPCLoopInstrFormPrep.cpp