Commit d147b27
authored
[ET-VK] Fix pipeline indexing bug in batch create_pipelines (#17192)
Refactored Context::flush() by extracting two new public functions:
- wait_for_queue(): blocks until the GPU queue is idle
- clear_resources(): clears command/descriptor pools and cleanup lists
This allows callers to use these operations independently. ComputeGraph's
destructor now uses these functions directly and wraps them in try/catch
blocks to ensure it never throws exceptions, following C++ best practices
for destructors.
Authored with Claude.
Differential Revision: [D92171362](https://our.internmc.facebook.com/intern/diff/D92171362/)
[ghstack-poisoned]1 parent 10d79a4 commit d147b27
1 file changed
Lines changed: 21 additions & 10 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
459 | 459 | | |
460 | 460 | | |
461 | 461 | | |
462 | | - | |
| 462 | + | |
| 463 | + | |
| 464 | + | |
| 465 | + | |
| 466 | + | |
| 467 | + | |
| 468 | + | |
| 469 | + | |
| 470 | + | |
| 471 | + | |
| 472 | + | |
| 473 | + | |
| 474 | + | |
| 475 | + | |
| 476 | + | |
| 477 | + | |
463 | 478 | | |
464 | 479 | | |
465 | 480 | | |
| |||
474 | 489 | | |
475 | 490 | | |
476 | 491 | | |
477 | | - | |
| 492 | + | |
478 | 493 | | |
479 | 494 | | |
480 | 495 | | |
| |||
513 | 528 | | |
514 | 529 | | |
515 | 530 | | |
516 | | - | |
517 | | - | |
518 | | - | |
519 | | - | |
520 | | - | |
521 | | - | |
522 | | - | |
523 | | - | |
| 531 | + | |
| 532 | + | |
| 533 | + | |
| 534 | + | |
524 | 535 | | |
525 | 536 | | |
526 | 537 | | |
| |||
0 commit comments