[PATCH] D78415: Fix inlining multi-block callees with type conversion.

Sean Silva via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 17 20:13:01 PDT 2020


silvas created this revision.
silvas added a reviewer: rriddle.
Herald added subscribers: llvm-commits, frgossen, grosul1, Joonsoo, liufengdb, lucyrfox, mgester, arpith-jacob, nicolasvasilache, antiagainst, shauheen, burmako, jpienaar, mehdi_amini, eraman.
Herald added a project: LLVM.

The previous code result a mismatch between block argument types and
predecessor successor args when a type conversion was needed in a
multiblock case. It was assuming the replaced result types matched the
region result types.

Also, slighly improve the debug output from the inliner and add a
debug counter <http://llvm.org/docs/ProgrammersManual.html#adding-debug-counters-to-aid-in-debugging-your-code>.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D78415

Files:
  mlir/include/mlir/Transforms/InliningUtils.h
  mlir/lib/Transforms/Inliner.cpp
  mlir/lib/Transforms/Utils/InliningUtils.cpp
  mlir/test/Transforms/inlining.mlir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78415.258470.patch
Type: text/x-patch
Size: 5908 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200418/47280919/attachment.bin>


More information about the llvm-commits mailing list