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

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 16:55:12 PST 2020


davidxl added a comment.

Class Name ResultWithMessage sounds too generic. Why not keeping the InlineResult class name? The rest of the changes look reasonable.


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