Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge main -> google #5500

Merged
merged 12 commits into from
Apr 17, 2021
Merged

Conversation

rsuderman
Copy link
Contributor

benvanik and others added 12 commits April 15, 2021 09:22
This was only half plumbed through before and now it's about 85% plumbed.
Probably.
This allows applications to inject a set of static libraries that can be resolved at runtime by name. This only includes the runtime portion: there's nothing in the compiler actually generating static libraries yet.

Future improvements to the driver API as part of iree-org#4298 will make it easier to inject these loaders.

Fixes iree-org#4679.
iree-org#5493 can return the inlining behavior once we have better ringbuffer
support and can measure the benefits.
Adding support for !vm.list<?> variant types.
Previously they were assuming the entire underlying storage buffer was
equal to the valid byte length which caused problems now that we are
packing multiple buffers together.
I'm suspicious that this is the proximate cause of crashes we're seeing
while destroying the Vulkan device. We think
iree-org#2659 and
iree-org#2900 are the root cause, but
reverting for now to try to get us back to green.

Example failure, starting at the PR being reverted:
https://source.cloud.google.com/results/invocations/a3093172-ae55-43bb-956c-256bb726006f/targets/iree%2Fgcp_ubuntu%2Fcmake-bazel%2Flinux%2Fx86-turing%2Fmain/log

Reverts iree-org#5467
…org#5494)

This required a minor tweak to start_as_root the flatbuffer root tables
prior to recording any data. This was discovered thanks to the gracious
help (and patience) of of the flatcc author:
dvidelabs/flatcc#179

With this all of our binary blobs embedded into the flatbuffers are now
16-byte aligned with the option to tweak it further via a vm.rodata
attribute. We don't need utf8 strings to be 16-byte aligned, for example.
…-things

Force all constants to be outlined and stored in the constant pool.
b0e2852 Merge pull request iree-org#5491 from rsuderman:main-to-google
3d9cd12 Synchronize submodules with LLVM at llvm/llvm-project@bb907b2
d23d2c5 Synchronize submodules with LLVM at llvm/llvm-project@bb907b2
7067b1e Integrate LLVM at llvm/llvm-project@bb907b2
efd715f Integrate LLVM at llvm/llvm-project@3bc88eb
8012dcd Integrate LLVM at llvm/llvm-project@3f97c66
e8f0247 Merge pull request iree-org#5466 from rsuderman:main-to-google
99ea45f Integrate LLVM at llvm/llvm-project@f28cb01
@google-cla google-cla bot added the cla: yes label Apr 17, 2021
@copybara-service copybara-service bot merged commit 32792f7 into iree-org:google Apr 17, 2021
GMNGeoffrey added a commit that referenced this pull request Apr 19, 2021
* 0adee0f Merge pull request #5520 from GMNGeoffrey:main-to-google
* e302cb3 Synchronize submodules with LLVM at llvm/llvm-project@80e166f81abd
* 6c94570 Integrate LLVM at llvm/llvm-project@80e166f81abd
* ce364ae Integrate LLVM at llvm/llvm-project@517c3aee4de5
* 32792f7 Merge pull request #5500 from rsuderman:main-to-google
This was referenced Apr 20, 2021
GMNGeoffrey added a commit that referenced this pull request Apr 22, 2021
* 97a7c8e Synchronize submodules with LLVM at llvm/llvm-project@2f69975683f5
* 3c09f24 Integrate LLVM at llvm/llvm-project@2f69975683f5
* 44d1063 Merge pull request #5551 from GMNGeoffrey:main-to-google
* 1243070 Integrate LLVM at llvm/llvm-project@1a3f88658a02
* 361e47b Integrate LLVM at llvm/llvm-project@4bb60c285cb3
* eaeb09d Integrate LLVM at llvm/llvm-project@9430efa18b02
* dc0af0c Synchronize submodules with LLVM at llvm/llvm-project@e0adf7e06a9e
* 678b1e1 Integrate LLVM at llvm/llvm-project@e0adf7e06a9e
* 98cf087 Integrate LLVM at llvm/llvm-project@01ace074fcb6
* 307cef0 Integrate LLVM at llvm/llvm-project@aa80ea8a617b
* 297f730 Integrate LLVM at llvm/llvm-project@fb69b92c7b33
* 5f1f4fe Integrate LLVM at llvm/llvm-project@f549176ad976
* bda62f0 Merge main -> google
* 0adee0f Merge pull request #5520 from GMNGeoffrey:main-to-google
* e302cb3 Synchronize submodules with LLVM at llvm/llvm-project@80e166f81abd
* 6c94570 Integrate LLVM at llvm/llvm-project@80e166f81abd
* ce364ae Integrate LLVM at llvm/llvm-project@517c3aee4de5
* 32792f7 Merge pull request #5500 from rsuderman:main-to-google
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants