Skip to content

Conversation

@alecardoso-tech
Copy link
Contributor

@alecardoso-tech alecardoso-tech commented Nov 17, 2025

Updated Multithreading sample to use enhanced barriers.

To enable it, choose <Config Option> (Enhanced Barriers) under the configuration dropdown.

@alecardoso-tech alecardoso-tech marked this pull request as ready for review November 20, 2025 00:15
Copilot finished reviewing on behalf of alecardoso-tech November 20, 2025 00:16
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the D3D12Multithreading sample to support enhanced barriers, a newer Direct3D 12 feature that provides more explicit control over resource synchronization. The sample now supports both legacy and enhanced barrier modes through build configurations.

  • Adds new build configurations for Enhanced Barriers and renames existing ones to Legacy Barriers
  • Upgrades Direct3D 12 interfaces to ID3D12Device10 and ID3D12GraphicsCommandList8
  • Implements enhanced barrier API calls with preprocessor guards for conditional compilation

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
FrameResource.h Updates device parameter to ID3D12Device10 and command list array to ID3D12GraphicsCommandList8
FrameResource.cpp Implements enhanced barrier resource creation and synchronization for shadow textures and constant buffers
D3D12Multithreading.vcxproj Adds new build configurations with USE_ENHANCED_BARRIERS preprocessor definition
D3D12Multithreading.sln Updates solution configurations to support both enhanced and legacy barrier builds
D3D12Multithreading.h Updates device interface to ID3D12Device10
D3D12Multithreading.cpp Adds enhanced barrier support for all resources and transitions, includes feature detection
readme.md Documents the new enhanced barriers feature and configuration options

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@microsoft microsoft deleted a comment from Copilot AI Nov 20, 2025
@microsoft microsoft deleted a comment from Copilot AI Nov 20, 2025
@microsoft microsoft deleted a comment from Copilot AI Nov 20, 2025
@billkris-ms billkris-ms self-requested a review November 20, 2025 21:28
@alecardoso-tech alecardoso-tech merged commit f8d07c1 into microsoft:master Nov 21, 2025
1 check passed
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.

3 participants