search

sampsyo / cs6120

advanced compilers
https://www.cs.cornell.edu/courses/cs6120/2023fa/
MIT License
752 stars 158 forks source link

Project blog: loop flattening in LLVM for Proposal #222 #253

Closed hj424 closed 3 years ago

hj424 commented 3 years ago

Hi Adrian,

Thanks for the suggestions! I have updated my blog in the latest post. Here are the keynotes of my update:

  1. Fix the writing issue.
  2. Add the checking module in the LLVM pass to make sure this pass works for real-world programs, i.e., it will not destroy the code with non-perfect nested loops.
  3. Verify that the pass works for some benchmarks from embed-iot

Thanks, Hanchen

sampsyo commented 3 years ago

Sounds good! But there are outstanding comments about nitpicky syntax things. Please request a review once those are fixed.

hj424 commented 3 years ago

Hi Adrian,

Thanks for the notification. In the latest commit, I have fixed the syntax issue and clarified the confusing part. Please take a look.

Thanks, Hanchen