[PATCH] D30747: TargetInstrInfo: Provide default implementation of isTailCall().

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 8 10:52:23 PST 2017


kparzysz added a comment.

In https://reviews.llvm.org/D30747#695716, @MatzeB wrote:

> I did not touch HexagonInstrInfo::isTailCall() here as it wasn't immediately obvious whether the flags are correct there and the default implementation would work. Some target maintainer should double check and if so maybe we can get rid of the complete callback.


Hexagon has over 100 different opcodes for branches, and each of them can be used as a tail call.  We are not going to invent 100+ phony opcodes just to put flags on them.


Repository:
  rL LLVM

https://reviews.llvm.org/D30747





More information about the llvm-commits mailing list