[PATCH] D71563: [SCEV] Recognise the hardwareloop "loop.decrement.reg" intrinsic

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 8 08:09:33 PST 2020


samparker added a comment.

It's currently the backends responsibility to lower and handle any of the intrinsics that are inserted for hardware loops, so I don't think we should be concerned with other transforms triggering. I think the benefit of SCEV still working for these loops far outweighs the codegen effort - especially since this effort already has to be done  because of isel and machine optimisations. That said, it probably wouldn't hurt to add some tests for common transforms especially ones where the loop body would be duplicated.


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

https://reviews.llvm.org/D71563





More information about the llvm-commits mailing list