rofirrim wrote:

This change now builds on top of:

- https://github.com/llvm/llvm-project/pull/155849
- https://github.com/llvm/llvm-project/pull/155848

I added a new class `OMPCanonicalLoopSequenceTransformationDirective` which 
inherits from a common class `OMPLoopTransformationDirective` which is also a 
base class of `OMPCanonicalLoopNestTransformationDirective` (what used to be 
called `OMPLoopTransformationDirective`). This class only contains the number 
of (top-level) loop nests generated along with a some convenience methods for 
consistency.

`OMPFuseDirective` now inherits from 
`OMPCanonicalLoopSequenceTransformationDirective` as I think it doesn't make 
sense to make it inherit from `OMPLoopBasedDirective` like it used in an 
earlier version of this patch.

I have also squashed the changes for the Fuse implementation because it was 
getting hard to handle.

Please @Meinersbur, @alexey-bataev let me know if the refactoring seems 
reasonable.

https://github.com/llvm/llvm-project/pull/139293
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to