[all-commits] [llvm/llvm-project] d53d95: [WebAssembly] Allow inlining functions with differ...

Thomas Lively via All-commits all-commits at lists.llvm.org
Thu Aug 13 13:58:02 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: d53d952810e7afaad66a8c527fa073e8f68f6222
      https://github.com/llvm/llvm-project/commit/d53d952810e7afaad66a8c527fa073e8f68f6222
  Author: Thomas Lively <tlively at google.com>
  Date:   2020-08-13 (Thu, 13 Aug 2020)

  Changed paths:
    M llvm/lib/Target/WebAssembly/WebAssemblyTargetTransformInfo.cpp
    M llvm/lib/Target/WebAssembly/WebAssemblyTargetTransformInfo.h
    A llvm/test/Transforms/Inline/WebAssembly/inline-target-features.ll

  Log Message:
  -----------
  [WebAssembly] Allow inlining functions with different features

Allow inlining only when the Callee has a subset of the Caller's
features. In principle, we should be able to inline regardless of any
features because WebAssembly supports features at module granularity,
not function granularity, but without this restriction it would be
possible for a module to "forget" about features if all the functions
that used them were inlined.

Requested in PR46812.

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




More information about the All-commits mailing list