[PATCH] D26149: [DAGCombiner] Match load by bytes idiom and fold it into a single load

Artur Pilipenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 30 04:48:27 PST 2016


apilipenko added a comment.

We see load by bytes (with optional bswap) patterns in Java code originating from ByteBuffers from the standard library. This is the main motivator for the change. I don't have any numbers for C/C++ code but I expect it's not uncommon.

As of matching patterns like rotate, we don't see it the the code we are interested about, so it's more like "this is where this code can evolve in the future".


https://reviews.llvm.org/D26149





More information about the llvm-commits mailing list