-
Notifications
You must be signed in to change notification settings - Fork 631
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
Integrate LLVM at llvm/llvm-project@b0329206 #18102
Conversation
Abbreviated Benchmark Summary@ commit f26aa02a0632f949eaf61baf04ac90268f9d0423 (vs. base 09dc003df2bb3418a71cc36268d2c93adaecebe0) Data-Tiling Comparison TableClick to show
Regressed Latencies 🚩
No improved or regressed compilation metrics 🏖️ For more information: |
Signed-off-by: hanhanW <[email protected]>
Signed-off-by: hanhanW <[email protected]>
b5d41d5
to
4bc6d1c
Compare
Signed-off-by: hanhanW <[email protected]>
@@ -48,7 +48,8 @@ | |||
#pipeline_layout = #hal.pipeline.layout<push_constants = 1, sets = [ | |||
<0, bindings = [ | |||
<0, storage_buffer, ReadOnly>, | |||
<1, storage_buffer> | |||
<1, storage_buffer, ReadOnly>, | |||
<2, storage_buffer> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know which commit breaks it, but the fix is reasonable to me. It is required by l.183, which has binding(2)
while ordinal 2
not present in descriptor set layout. So the fix is adding one more storage_buffer to the set.
// Bindings are accessed by reference.
%binding0 = hal.interface.binding.subspan layout(#pipeline_layout) set(0) binding(0) alignment(64) offset(%c0) : memref<?xf32>{%dim}
%binding1 = hal.interface.binding.subspan layout(#pipeline_layout) set(0) binding(1) alignment(64) offset(%c0) : memref<?xf32>{%dim}
%binding2 = hal.interface.binding.subspan layout(#pipeline_layout) set(0) binding(2) alignment(64) offset(%c0) : memref<?xf32>{%dim}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Likely unrelated to this LLVM integrate. I thought I had the ASan job running on all (relevant) PRs and commits, but I had a bug. Fix: #18108
No local patches.