[PATCH] D30670: [Outliner] Add tail call support

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 13 18:06:05 PDT 2017


echristo added a comment.

The boolean parameter threaded through seems a bit unwieldy. Would it be better to either a) add a call that said e.g. TII.numInstrsForCallReturn(), or b) just assume 1?

Also minor coding convention nit, single statement ifs don't need a { } :)

-eric


https://reviews.llvm.org/D30670





More information about the llvm-commits mailing list