[PATCH] D72744: [NFC] Refactor InlineResult for readability

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 21:31:35 PST 2020


davidxl added a comment.

InlineResult --> inlining related result (viability, etc) -- it captures two pieces of information:  1) inline decision and 2) when decision is 'no',  related inline analysis that leads to the no decision.  The class name seems fine.  The patch makes the 'decision' part more explicit, and also fixes some bug in missing the right analysis message.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72744





More information about the llvm-commits mailing list