[PATCH] D18900: New pre-isel pass SystemZBRCTLoops

[Jonas Paulsson via llvm-commits]

jonpa added a comment.

Since there seems to be some general interest for a target independent hw-loop pass, I started out thinking that the hw-loop counter might be a simple beginning for it. So I actually first made just a 'hw-loop-count' intrinsic in the main file, but then moved it into a '-s390-' since this is still only a SystemZ pass.

I think this pass could basically be target-independent right now, so the question is if anyone would want this right now? Should I make a systemz pre-isel pass out of this or a new target-independent hw-loop pass with just the hw-loop-count intrinsic implemented?

Regardless, I don't think SystemZ needs the intrinsic, like you pointed out. For just a SystemZ pre-isel pass, it could just be removed. With a target-independent pass, just the isel patterns for it would remain.


http://reviews.llvm.org/D18900