Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

[FuzzMutate] Don't use index operands as sinks

Authored by igor-laevsky on Nov 23 2017, 8:14 AM.



This fixes typo in the sink generator. We can't use index operands as sinks from the arbitrary instructions. This is actually two problems: shufflevector was missing from the switch case, and ">" was used instead of ">=". I believe this check can be made less conservative, but I figured it's better to be correct for now.

Diff Detail


Event Timeline

igor-laevsky created this revision.Nov 23 2017, 8:14 AM
bogner accepted this revision.Nov 27 2017, 4:44 PM

Agreed, we should be able to make this less conservative at some point. This is good for now.

This revision is now accepted and ready to land.Nov 27 2017, 4:44 PM
This revision was automatically updated to reflect the committed changes.