[PATCH] D79215: [llvm][NFC] Inliner: simplify inlining decision logic

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 2 00:29:38 PDT 2020


davidxl added a comment.

I don't feel strongly to add the new interface -- my original suggestion was to return Optional<InlineCost> without dropping it to None.

On the other hand, I do like the new return type though -- using Optional<..> and let client fiddle with hasValue and getValue to figure out the inline decisions is not friendly. Having a new return type does make the user code cleaner.

Having said that, I don't mind either way.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79215





More information about the llvm-commits mailing list