[Mlir-commits] [mlir] [MLIR][DLTI] Enable types as keys in	DLTI-query utils (PR #105995)
    Adam Siemieniuk 
    llvmlistbot at llvm.org
       
    Mon Aug 26 02:52:37 PDT 2024
    
    
  
================
@@ -424,8 +424,8 @@ getClosestQueryable(Operation *op) {
   return std::pair(queryable, op);
 }
 
-FailureOr<Attribute> dlti::query(Operation *op, ArrayRef<StringAttr> keys,
-                                 bool emitError) {
+FailureOr<Attribute>
+dlti::query(Operation *op, ArrayRef<DataLayoutEntryKey> keys, bool emitError) {
----------------
adam-smnk wrote:
Unrelated to this PR so feel free to ignore for now but just came to my mind.
What's the desired behavior when `keys` is empty? Right now it works but it's not covered by any test.
https://github.com/llvm/llvm-project/pull/105995
    
    
More information about the Mlir-commits
mailing list