Closed edeiana closed 2 years ago
Good job Enrico and thank you for doing the work.
I have only the next two comments.
Having both addID and setID for LoopStructure is confusing. Can we have only setID with the semantics of
Why does the new module loop_id needs the headers of HELIX, DOALL, DSWP, parallelization_technique, heuristics, loop_distribution, and talkdown?
Changes based on feedback.
Created a new pass that sets the ID for every loop as metadata. Created functionality to add ,set, get loop ID metadata in LoopStructure. Changed behavior of users of loop ID, since it is now an std::optional.