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

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 7 18:10:28 PST 2017


MatzeB added a comment.

Is it possible to avoid the code dealing with `ReturnIDs` and the new `TailCall` instruction type, simply by taking the last instruction of an outlined sequence and calling `TII::isTailCall()` (or `MachineInstr::isReturn()`) on it to determine whether we outline a sequence ending in one?



================
Comment at: lib/CodeGen/MachineOutliner.cpp:520
 
+  unsigned operator[](size_t i) {
+    return Str[i];
----------------
This can be `const`.


https://reviews.llvm.org/D30670





More information about the llvm-commits mailing list