[PATCH] D55192: [PowerPC] VSX register support for inline assembly

Jinsong Ji via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 4 08:36:50 PST 2018


jsji requested changes to this revision.
jsji added a comment.
This revision now requires changes to proceed.

RegNum are wrong!



================
Comment at: clang/lib/Basic/Targets/PPC.cpp:416
+const TargetInfo::AddlRegName GCCAddlRegNames[] = {
+    {{"vs0"}, 0},   {{"vs1"}, 1},   {{"vs2"}, 2},   {{"vs3"}, 3}, 
+    {{"vs4"}, 4},   {{"vs5"}, 5},   {{"vs6"}, 6},   {{"vs7"}, 7},
----------------
RegNum is wrong here!
"vs0" should be mapped to RegNum of "f0", which is 33. 
"vs32" should be mapped to RegNum of "v0", which is 78.


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

https://reviews.llvm.org/D55192





More information about the llvm-commits mailing list