Skip to content

Conversation

@kazutakahirata
Copy link
Contributor

No description provided.

@llvmbot llvmbot added backend:DirectX llvm:analysis Includes value tracking, cost tables and constant folding labels May 12, 2025
@llvmbot
Copy link
Member

llvmbot commented May 12, 2025

@llvm/pr-subscribers-backend-directx

@llvm/pr-subscribers-llvm-analysis

Author: Kazu Hirata (kazutakahirata)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/139488.diff

1 Files Affected:

  • (modified) llvm/lib/Analysis/DXILResource.cpp (+1-3)
diff --git a/llvm/lib/Analysis/DXILResource.cpp b/llvm/lib/Analysis/DXILResource.cpp index 1602b24e7d9af..baba7d2c75e58 100644 --- a/llvm/lib/Analysis/DXILResource.cpp +++ b/llvm/lib/Analysis/DXILResource.cpp @@ -955,9 +955,7 @@ void DXILResourceBindingInfo::populate(Module &M, DXILResourceTypeMap &DRTM) { // for each binding type and used spaces. Bindings are sorted by resource // class, space, and lower bound register slot. BindingSpaces *BS = &SRVSpaces; - for (unsigned I = 0, E = Bindings.size(); I != E; ++I) { - Binding &B = Bindings[I]; - + for (const Binding &B : Bindings) { if (BS->RC != B.RC) // move to the next resource class spaces BS = &getBindingSpaces(B.RC); 
@kazutakahirata kazutakahirata merged commit 4c9ffb7 into llvm:main May 12, 2025
14 checks passed
@kazutakahirata kazutakahirata deleted the cleanup_001_tidy_modernize-loop-convert_llvm branch May 12, 2025 04:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend:DirectX llvm:analysis Includes value tracking, cost tables and constant folding

3 participants