[all-commits] [llvm/llvm-project] b86a9c: [mlir][irdl] Lookup symbols near dialects instead ...

[Théo Degioanni via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b86a9c5bf2fab0408a3d549995d6e2449f71a16d
      https://github.com/llvm/llvm-project/commit/b86a9c5bf2fab0408a3d549995d6e2449f71a16d
  Author: Théo Degioanni <theo.degioanni.llvm.deluge062 at simplelogin.fr>
  Date:   2024-05-31 (Fri, 31 May 2024)

  Changed paths:
    A mlir/include/mlir/Dialect/IRDL/IRDLSymbols.h
    M mlir/lib/Dialect/IRDL/CMakeLists.txt
    M mlir/lib/Dialect/IRDL/IR/IRDL.cpp
    M mlir/lib/Dialect/IRDL/IR/IRDLOps.cpp
    M mlir/lib/Dialect/IRDL/IRDLLoading.cpp
    A mlir/lib/Dialect/IRDL/IRDLSymbols.cpp
    M mlir/test/Dialect/IRDL/cmath.irdl.mlir
    M mlir/test/Dialect/IRDL/cyclic-types.irdl.mlir
    M mlir/test/Dialect/IRDL/invalid.irdl.mlir
    M mlir/test/Dialect/IRDL/testd.irdl.mlir

  Log Message:
  -----------
  [mlir][irdl] Lookup symbols near dialects instead of locally (#92819)

Because symbols cannot refer to operations outside of their symbol
tables, it was impossible to refer to operations outside of the dialect
currently being defined. This PR modifies the lookup logic to happen
relative to the symbol table containing the dialect-defining operations.
This is a bit of hack but should unblock the situation here.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications