[PATCH] D66098: [CostModel] Model all `extractvalue`s as free.

David Greene via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 13 10:34:52 PDT 2019


greened added a comment.

LGTM, though are we sure this is true for all targets?  The comments in the referenced patch only consider X86.  I'm pretty sure it is true for common architectures like AArch64 but I'm not as sure for more exotic things.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66098





More information about the llvm-commits mailing list