[PATCH] D79283: [PowerPC] Add missing handling for half precision

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 4 06:22:27 PDT 2020


uweigand added a comment.

> This is missing from the SystemZ back end

I don't think this change is right for SystemZ.  This would change the ABI for single-element vector types.   According to our ABI we **want** to pass those in vector registers, which requires widening to the full-width vector type.   With your change, common code would instead scalarize to the scalar type, which means the parameter would be passed in GPRs instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79283





More information about the llvm-commits mailing list