[PATCH] D13165: [ARM] Avoid redundant checks for isThumb1Only() after supportsTailCall()

Saleem Abdulrasool via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 25 21:55:29 PDT 2015


compnerd accepted this revision.
compnerd added a comment.
This revision is now accepted and ready to land.

This is correct as is.  Might be nice to have IsEligibleForTailCall assert that TCO is available on the subtarget (currently shouldn't be possible to trip on since the only caller only calls it if the subtarget supports TCO and the MF is not marked with "disable-tail-calls").  A second assert for !Thumb1 with a comment pointer to initSubtargetFeatures would be nice too.


http://reviews.llvm.org/D13165





More information about the llvm-commits mailing list