Xiangling_L accepted this revision.
Xiangling_L added a comment.
This revision is now accepted and ready to land.

Just a minor suggestion: Maybe `Enabling return and argument vector types for 
AIX` is more accurate. Otherwise, LGTM.



================
Comment at: clang/test/CodeGen/aix-altivec.c:5
+vector float foo1(vector float x) { return x; }
+// CHECK:  define <4 x float> @foo1(<4 x float> %x) #0 {
+// CHECK:  entry:
----------------
minor:
you can use `[[ATTR:#[0-9]+]]` to replace `#0`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92117

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D92117: [AIX] Ena... Xiangling Liao via Phabricator via cfe-commits

Reply via email to