[PATCH] D137785: [PowerPC][GISel] Add initial GlobalISel support for vector functions.

[ChenZheng via Phabricator via llvm-commits]

shchenz accepted this revision as: shchenz.
shchenz added a comment.

LGTM too with one minor question. Thanks very much.



================
Comment at: llvm/lib/Target/PowerPC/GISel/PPCRegisterBanks.td:19
+/// Vector Registers
+def VECRegBank : RegisterBank<"VEC", [VSRC]>;
----------------
Maybe too early for this question as long double is not supported in globalisel for now. I see `f128` is a type mapped to vector type register class `VRRC`, but to me, seems type `f128` should be a scalar type?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137785