You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have a few finds by the fuzzer that affect LLVM (tested using a1d77ca).
I think they're all duplicates but I'm attaching them all in a .zip so it can be confirmed.
Thanks @patrick-rivos, it looks like all test cases had dead instructions, which would be considered by the legacy cost model but not the VPlan-based one (because VPlan-based DCE would clean them up before). Should be fixed now.
@JonPsson1's case was a different issue and should also be fixed.
Summary:
Update collectValuesToIgnore to also ignore dead instructions in the
loop. Such instructions will be removed by VPlan-based DCE and won't be
considered by the VPlan-based cost model.
This closes a gap between the legacy and VPlan-based cost model. In
practice with the default pipelines, there shouldn't be any dead
instructions in loops reaching LoopVectorize, but it is easy to generate
such cases by hand or automatically via fuzzers.
Fixes#99701.
Test Plan:
Reviewers:
Subscribers:
Tasks:
Tags:
Differential Revision: https://phabricator.intern.facebook.com/D60250593
Uh oh!
There was an error while loading. Please reload this page.
I have a few finds by the fuzzer that affect LLVM (tested using a1d77ca).
I think they're all duplicates but I'm attaching them all in a .zip so it can be confirmed.
Testcase:
Command/backtrace:
Godbolt: https://godbolt.org/z/a7KPnMh94
Zip with all finds reduced:
finds.zip
Related: #92555 @fhahn
Found via fuzzer.
The text was updated successfully, but these errors were encountered: