[PATCH] D40477: Enable Partial Inlining by default

Wed Mar 31 23:06:48 PDT 2021

ChuanqiXu added a comment.

In D40477#1302461 <https://reviews.llvm.org/D40477#1302461>, @vsk wrote:

> Imho CodeExtractor needs a little more work before it's ready to be on-by-default. There are two main blockers: missing debug info support, and a missing whitelist of extract-able intrinsics. The latter poses a high risk for miscompiles (see llvm.org/PR39545, llvm.org/PR39671).

What does the missing debug info support mean? If it is mean to cloning function would miss debug information, it looks like two recent patches had resolved this problem (see D86185 <https://reviews.llvm.org/D86185> and D96531 <https://reviews.llvm.org/D96531>).

Repository:
  rL LLVM

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

https://reviews.llvm.org/D40477