[PATCH] D75384: OpenMP for loop fusion

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a subscriber: jmellorcrummey.
jdoerfert added a comment.

In D75384#1933294 <https://reviews.llvm.org/D75384#1933294>, @lebedev.ri wrote:

> Hmm. Random question: are these optimizations friendly to the openmp tooling?


Not yet but it's on my list. The plan was to start improving this once we have an "intrusive" transformation, e.g., parallel region merging I have downstream. As of now, I expect us to emit remarks that a tool can use to match the code and events seen at runimte with the input. Similarly, we need to deal with source location changes, update source locations when we merge calls, etc. (there is a TODO in OpenMPOpt already). Feedback on this is very welcome. I also wanted the input of @jmellorcrummey on that.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75384/new/

https://reviews.llvm.org/D75384