[PATCH] D28637: [PPC] Inline expansion of memcmp

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 16 10:51:39 PST 2017


nemanjai added a comment.

> Well, but on SystemZ we will also emit branches; memcmp will generally expand to a loop over CLC instructions.  It's just that this is hidden behind a CLC_LOOP pseudo at the DAG level, which then gets expanded by a custom inserter.

The original implementation was very similar but the community felt that this isn't the best approach for valid reasons (i.e. no DAG combines, etc.). Please refer to: http://lists.llvm.org/pipermail/llvm-dev/2016-December/108589.html


https://reviews.llvm.org/D28637





More information about the llvm-commits mailing list