| Summary: | Enabling sisched gives Assertion `!NodePtr->isKnownSentinel()' failed | ||
|---|---|---|---|
| Product: | Mesa | Reporter: | Vedran Miletić <vedran> |
| Component: | Drivers/Gallium/radeonsi | Assignee: | Default DRI bug account <dri-devel> |
| Status: | RESOLVED FIXED | QA Contact: | Default DRI bug account <dri-devel> |
| Severity: | normal | ||
| Priority: | medium | Keywords: | bisected, regression |
| Version: | git | ||
| Hardware: | Other | ||
| OS: | All | ||
| URL: | https://llvm.org/bugs/show_bug.cgi?id=30514 | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: | Dump with R600_DEBUG=vs,tcs,tes,gs,ps,cs,sisched | ||
|
Description
Vedran Miletić
2016-09-24 20:12:16 UTC
6037e199c8a061b2e9cb22c434644a0eac659fb9 is the first bad commit commit 6037e199c8a061b2e9cb22c434644a0eac659fb9 Author: Duncan P. N. Exon Smith <dexonsmith@apple.com> Date: Sun Sep 11 16:38:18 2016 +0000 CodeGen: Turn on sentinel tracking for MachineInstr iterators This is a prep commit before fixing MachineBasicBlock::reverse_iterator invalidation semantics, ala r281167 for ilist::reverse_iterator. This changes MachineBasicBlock::Instructions to track which node is the sentinel regardless of LLVM_ENABLE_ABI_BREAKING_CHECKS. There's almost no functionality change (aside from ABI). However, in the rare configuration: #if !defined(NDEBUG) && !defined(LLVM_ENABLE_ABI_BREAKING_CHECKS) the isKnownSentinel() assertions in ilist_iterator<>::operator* suddenly have teeth for MachineInstr. If these assertions start firing for your out-of-tree backend, have a look at the suggestions in the commit message for r279314, and at some of the commits leading up to it that avoid dereferencing the end() iterator. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@281168 91177308-0d34-0410-b5e6-96231b3b80d8 Hi Vedran, please provide the output with R600_DEBUG=vs,tcs,tes,gs,ps,cs. From there, it should be possible to extract the crashing shader. Created attachment 126816 [details]
Dump with R600_DEBUG=vs,tcs,tes,gs,ps,cs,sisched
Interesting, doesn't crash without sisched. With R600_DEBUG=vs,tcs,tes,gs,ps,cs,sisched I get the following.
Reproducible with glxgears, not Talos-specific. I suspect sisched needs to be updated for recent changes in LLVM to the control flow lowering. Trying to ping Axel Davy... Fixed by https://reviews.llvm.org/D27844 LLVM r290347 |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.