[PATCH] D11489: [RewriteStatepointsForGC] Handle extractelement fully in the base pointer algorithm

Philip Reames listmail at philipreames.com
Tue Jul 28 18:30:32 PDT 2015


reames updated this revision to Diff 30882.
reames added a comment.

Address review comments and fix a bug I noticed.  It turns out I can't yet remove the peephole optimization code without regressing existing functionality.  This wasn't covered by the test case - bad Philip! - but I've added two new tests that show the issue.  The short version of the problem is that we were also missing generic handling for insertelement and that the peepholes were covering that too.  Gah.


http://reviews.llvm.org/D11489

Files:
  lib/Transforms/Scalar/RewriteStatepointsForGC.cpp
  test/Transforms/RewriteStatepointsForGC/base-vector.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11489.30882.patch
Type: text/x-patch
Size: 15648 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150729/fc847097/attachment.bin>


More information about the llvm-commits mailing list