[PATCH] D109966: [X86][NFC] structure-return simplificiation

Nathan Sidwell via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 5 07:58:21 PDT 2021


urnathan updated this revision to Diff 377244.
urnathan added a comment.
Herald added a subscriber: mstorsjo.

thanks for your comments.  Moving the check-which-ABI bits into the predicate does make things much simpler.  I chose to templatize it, rather than write the same thing twice.  That does mean taking a SmallVector<T> reference, rather than ArrayRef, because template deduction.  But that ends up with less work to do anyway.  I reordered the checks to get the best short-circuiting (heuristic guesswork though).

would you prefer anonymous namespace, or stick with the static fn?


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

https://reviews.llvm.org/D109966

Files:
  llvm/lib/Target/X86/X86ISelLowering.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D109966.377244.patch
Type: text/x-patch
Size: 6538 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211005/2998b3eb/attachment.bin>


More information about the llvm-commits mailing list