[all-commits] [llvm/llvm-project] f43e67: [mlir] Allow SymbolTable to update existing symbols

Tres via All-commits all-commits at lists.llvm.org
Tue Dec 15 15:45:04 PST 2020


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f43e67cc6c6f6e0da925039b28b54e44fc751267
      https://github.com/llvm/llvm-project/commit/f43e67cc6c6f6e0da925039b28b54e44fc751267
  Author: Tres Popp <tpopp at google.com>
  Date:   2020-12-16 (Wed, 16 Dec 2020)

  Changed paths:
    M mlir/include/mlir/IR/SymbolTable.h
    M mlir/lib/IR/SymbolTable.cpp

  Log Message:
  -----------
  [mlir] Allow SymbolTable to update existing symbols

Previous behavior would fail if inserting an operation that already
existed. Now SymbolTable::insert can also be used as a way to make a
symbol's name unique even after insertion.

Further TODOs have been left over naming and consistent behavior
considerations.

Differential Revision: https://reviews.llvm.org/D93349




More information about the All-commits mailing list