[Mlir-commits] [mlir] [mlir][spirv] Rework type extension queries	(PR #160020)
    Jakub Kuderski 
    llvmlistbot at llvm.org
       
    Mon Sep 22 05:02:48 PDT 2025
    
    
  
kuhar wrote:
> Could we have a default implementation of `::getExtensions()` in `SPIRVType` class. This way we avoid repeating the same implementation in each inherited type, as they can just use the parent default. I think this also would help enforce that `::getExtensions()` is not abused as any custom implementation of `::getExtensions()` would have to be explicit.
Good idea, done. I wanted to preserve the existing API but didn't realize that all these `::getExtensions` functions in derived classes only shadow the main definition in `SPIRVType`.
https://github.com/llvm/llvm-project/pull/160020
    
    
More information about the Mlir-commits
mailing list