[PATCH] D85494: [WebAssembly] Allow inlining functions with different features

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 12 18:31:39 PDT 2020


aheejin added a comment.

Not sure if it's worth it if there's no precedents. If people use our toolchain (clang or emscripten or rust or whatever) that wouldn't happen in the first place. Maybe it'd be sufficient to print some warning messages in `CoalesceFeaturesAndStripAtomics` if features don't match..?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85494/new/

https://reviews.llvm.org/D85494



More information about the llvm-commits mailing list