diff options
| author | Tim Foley <tfoleyNV@users.noreply.github.com> | 2018-02-13 10:22:54 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-02-13 10:22:54 -0800 |
| commit | 32549707cc9aa67dbc19cbdc0490ffebc8ec253c (patch) | |
| tree | 4a8d50d9e42d73e045cd6cddf5e5879a43ce7f6b /tests/reflection/reflect-imported-code.hlsl | |
| parent | 214a1fced7c53b81c00bec67fa2b91a357d5ece4 (diff) | |
Fix a bug in IR use-def information (#406)
The basic problem here is that when unlinking an `IRUse` from the linked list of uses, there were several cases where I was failing to set the `prevLink` field of the next node to match the `prevLink` field of the node being removed. That doesn't show up when walking the linked list of uses forward, but it breaks it whenever you have subsequent unlinking operations.
This change fixes the bugs of that kind I could find, and also adds a debug validation method to try to avoid breaking it again. I also made more access to `IRUse` go through accessor methods rather than using fields directly, to try to avoid this kind of error. I stopped short of making anything `private`, because I tend to find that it creates more hassles than it avoids.
A few other fixes along the way:
- Made the `List<T>` type default-initialize elements when you resize it. I hadn't realized we weren't doing that.
- Add a standalone `dumpIR(IRGlobalValue*)` so help when debugging issues.
Diffstat (limited to 'tests/reflection/reflect-imported-code.hlsl')
0 files changed, 0 insertions, 0 deletions
