[llvm] [DirectX] Add support for heap resources to `DXILResourceMap` (PR #216454)

[Ashley Coleman via llvm-commits]

================
@@ -452,13 +460,12 @@ class ResourceInfo {
   getAnnotateProps(Module &M, dxil::ResourceTypeInfo &RTI) const;
 
   bool operator==(const ResourceInfo &RHS) const {
-    return std::tie(Binding, HandleTy, Symbol, Name, HeapResourceID) ==
-           std::tie(RHS.Binding, RHS.HandleTy, RHS.Symbol, RHS.Name,
-                    RHS.HeapResourceID);
+    return std::tie(BindingOrHeapID, HandleTy, Symbol, Name) ==
+           std::tie(RHS.BindingOrHeapID, RHS.HandleTy, RHS.Symbol, RHS.Name);
   }
   bool operator!=(const ResourceInfo &RHS) const { return !(*this == RHS); }
   bool operator<(const ResourceInfo &RHS) const {
-    return Binding < RHS.Binding;
+    return BindingOrHeapID < RHS.BindingOrHeapID;
----------------
V-FEXrt wrote:

Not a change request:

How does this work when they hold different things?

https://github.com/llvm/llvm-project/pull/216454