[all-commits] [llvm/llvm-project] f86209: [FLANG][MLIR] Update all module symbol references ...

agozillon via All-commits all-commits at lists.llvm.org
Thu Feb 2 03:00:03 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f86209fc80d8a646ed5802d58f4ed6934d980d4e
      https://github.com/llvm/llvm-project/commit/f86209fc80d8a646ed5802d58f4ed6934d980d4e
  Author: Andrew Gozillon <Andrew.Gozillon at amd.com>
  Date:   2023-02-02 (Thu, 02 Feb 2023)

  Changed paths:
    M flang/lib/Optimizer/Transforms/ExternalNameConversion.cpp
    M flang/test/Fir/external-mangling.fir

  Log Message:
  -----------
  [FLANG][MLIR] Update all module symbol references after changing FuncOp symbol during external name mangling

This fixes an issue where the symbols for operations that were not directly
handled by the rewriting in ExternalNameConversion.cpp were not updated
accurately when a FuncOp symbol was modified. Resulting in a name
mismatch between the FuncOp and the operation holding a symbol to
the FuncOp.

This fix works by updating all of the symbols relating to a FuncOp in a
module, this did not show up as an issue previously as fir::CallOps were
getting specific handling and only fir::CallOps were being tested. So
as the more larger case is now being handled the specific handling for
fir::CallOps has been removed (but is still handled by the fix).

Reviewers:
clementval

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




More information about the All-commits mailing list