[GSoC] Unify ways to move code or check if code is safe to be moved

Hello,

I am super excited about my proposal being accepted. I would like to thank Whitney for her help during the application phase. I did schedule some tasks for community bonding period. I have started looking into analysis pass for code motion checks. I will keep you updated with the progress.

Best,
Rithik Sharma

Hi Rithik,

Welcome to LLVM. Excited to work with you for the coming summer!

Are you available on this Thursday or Friday for a quick call using webex?
Most of us are located in Toronto (GMT-4). What time will you be available?

Regards,
Whitney Tsang

graycol.gifRITHIK SHARMA —2020/05/05 02:10:58 PM—Hello, I am super excited about my proposal being accepted. I would like to thank

Hello,

This is the summary of my Community bonding period week 1, as planned I spent some time reading loop analysis passes present in lib/Analysis/, I tried to find the code motion checks in these analysis passes. I was able to find some worthy code motion checks in LoopInfo pass.

I also looked into the Phabricator review D71578 to get familiar with CodeMoverUtils unit tests. I tried to replicate and fix a test case by using bfs numbering instead of dfs locally.

I had an exceptional experience so far and thank all the mentors for being so helpful and supportive.

Best Regards,

Rithik Sharma

graycol.gif

Hello,

This is the summary of my Community bonding period week 2, as planned I spent some time reading loop transformation passes present in lib/Transforms/, I tried to find the code motion checks in transformation passes. I was able to find some worthy code motion checks in transformation passes, I also looked into the Phabricator review D71578 to get familiar with PR and writing unit tests. isSafeToMoveBefore uses DFS numbering for determining the relative position of instruction and insert point which is not always correct, I tried to replicate and fix a test case by using dominator tree level to decide the direction of code motion and posted a PR (Phabricator review D80084).

Best Regards,
Rithik Sharma

graycol.gif