[PATCH] D43093: [FastISel] Sink local value materializations to first use

[Reid Kleckner via Phabricator via llvm-commits]

rnk added a comment.

In https://reviews.llvm.org/D43093#1016883, @MatzeB wrote:

> Looks good from what I can see, though I'm not super experienced with fast-isel either. Does this work correctly in the case when fast-isel aborts while selecting a block?


Thanks for reading, though. :) I believe the mid-block abort case came up during testing and is covered by the existing suite. FastISel often aborts on calls, flushes the local value area, goes to sdag to select a single call instruction, and then continues fastisel. This all seems to work. When it aborts, it flushes the local value area, which now triggers sinking.


https://reviews.llvm.org/D43093