mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-11-23 19:23:23 +01:00
[x86] Remove the FIXME that was implemented in r214628. Managed to
forget to update the comment here... =/ llvm-svn: 214630
This commit is contained in:
parent
2c76889549
commit
b86a753f28
@ -18945,10 +18945,6 @@ static bool combineX86ShuffleChain(SDValue Op, SDValue Root, ArrayRef<int> Mask,
|
||||
/// This should never be an issue in practice as the shuffle lowering doesn't
|
||||
/// produce sequences of more than 8 instructions.
|
||||
///
|
||||
/// FIXME: Currently, we don't collapse instructions *into* PSHUFB. We should,
|
||||
/// and we should do so more aggressively than we form PSHUFB because once we
|
||||
/// have a PSHUFB, we might as well do as much shuffling as we can.
|
||||
///
|
||||
/// FIXME: We will currently miss some cases where the redundant shuffling
|
||||
/// would simplify under the threshold for PSHUFB formation because of
|
||||
/// combine-ordering. To fix this, we should do the redundant instruction
|
||||
|
Loading…
Reference in New Issue
Block a user