When I (hack) fixed the fact that Subzero fails to load from a constant source because it interprets it as an offset, I did so by routing all loads into sz::createLoad, and in there, making sure to bitcast constant sources to variables. I thought this covered all loads, but it turns out that one of our optimization passes optimizes stores by removing intermediate stores. This would sometimes replace a variable source with a constant one, so I applied a similar hack here to avoid doing replace in this case. Bug: b/148272103 Change-Id: I809688e2c79fa1b62918d3b484a8e2a9e601bd90 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/41269 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Tested-by:Antonio Maiorano <amaiorano@google.com> Reviewed-by:
Nicolas Capens <nicolascapens@google.com>
| Name |
Last commit
|
Last update |
|---|---|---|
| .vscode | Loading commit data... | |
| build | Loading commit data... | |
| build_overrides | Loading commit data... | |
| docs | Loading commit data... | |
| extensions | Loading commit data... | |
| include | Loading commit data... | |
| src | Loading commit data... | |
| tests | Loading commit data... | |
| third_party | Loading commit data... | |
| .clang-format | Loading commit data... | |
| .dir-locals.el | Loading commit data... | |
| .gitignore | Loading commit data... | |
| .gitmodules | Loading commit data... | |
| .travis.yml | Loading commit data... | |
| AUTHORS.txt | Loading commit data... | |
| Android.bp | Loading commit data... | |
| BUILD.gn | Loading commit data... | |
| CMakeLists.txt | Loading commit data... | |
| CMakeSettings.json | Loading commit data... | |
| CONTRIBUTING.txt | Loading commit data... | |
| CONTRIBUTORS.txt | Loading commit data... | |
| LICENSE.txt | Loading commit data... | |
| OWNERS | Loading commit data... | |
| README.md | Loading commit data... | |
| SwiftShader.sln | Loading commit data... |