[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