Add 64-bit target check for signed AMO builtins

Author: maryammo

clang/lib/Sema/SemaPPC.cpp

@@ -89,6 +89,8 @@ static bool isPPC_64Builtin(unsigned BuiltinID) {
   case PPC::BI__builtin_ppc_fetch_and_swaplp:
   case PPC::BI__builtin_amo_lwat:
   case PPC::BI__builtin_amo_ldat:
+  case PPC::BI__builtin_amo_lwat_s:
+  case PPC::BI__builtin_amo_ldat_s:
     return true;
   }
   return false;

clang/test/CodeGen/PowerPC/builtins-amo-err.c

@@ -5,26 +5,26 @@
 
 void test_amo() {
   unsigned int *ptr1, value1;
-  // AIX32-ERROR: error: this builtin is only available on 64-bit targets
+  // AIX32-ERROR-COUNT-2: error: this builtin is only available on 64-bit targets
   __builtin_amo_lwat(ptr1, value1, 0);
   // FC-ERROR: argument value 9 is outside the valid range [0-4, 6, 8]
   __builtin_amo_lwat(ptr1, value1, 9);
 
   unsigned long int *ptr2, value2;
-  // AIX32-ERROR: error: this builtin is only available on 64-bit targets
+  // AIX32-ERROR-COUNT-2: error: this builtin is only available on 64-bit targets
   __builtin_amo_ldat(ptr2, value2, 3);
   // FC-ERROR: error: argument value 26 is outside the valid range [0-4, 6, 8]
   __builtin_amo_ldat(ptr2, value2, 26);
 
   signed int *ptr3, value3;
-  // AIX32-ERROR: error: this builtin is only available on 64-bit targets
+  // AIX32-ERROR-COUNT-2: error: this builtin is only available on 64-bit targets
   __builtin_amo_lwat_s(ptr3, value3, 0);
   // FC-ERROR: argument value 2 is outside the valid range [0, 5, 7, 8]
   __builtin_amo_lwat_s(ptr3, value3, 2);
 
-  unsigned long int *ptr4, value4;
-  // AIX32-ERROR: error: this builtin is only available on 64-bit targets
-  __builtin_amo_ldat_s(ptr4, value4, 3);
+  signed long int *ptr4, value4;
+  // AIX32-ERROR-COUNT-2: error: this builtin is only available on 64-bit targets
+  __builtin_amo_ldat_s(ptr4, value4, 5);
   // FC-ERROR: error: argument value 6 is outside the valid range [0, 5, 7, 8]
   __builtin_amo_ldat_s(ptr4, value4, 6);
 }