Mesa (master): radeon/llvm: Clean up AMDILIntrinsicInfo.cpp

[Tom Stellard]

Module: Mesa
Branch: master
Commit: cc3907856ed7ec24359eea11fc2231dab680e2c4
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=cc3907856ed7ec24359eea11fc2231dab680e2c4

Author: Tom Stellard <thomas.stellard at amd.com>
Date:   Fri Jul 13 16:08:24 2012 +0000

radeon/llvm: Clean up AMDILIntrinsicInfo.cpp

---

 src/gallium/drivers/radeon/AMDILIntrinsicInfo.cpp |   88 +-------------------
 src/gallium/drivers/radeon/AMDILIntrinsicInfo.h   |    1 -
 2 files changed, 5 insertions(+), 84 deletions(-)

diff --git a/src/gallium/drivers/radeon/AMDILIntrinsicInfo.cpp b/src/gallium/drivers/radeon/AMDILIntrinsicInfo.cpp
index 678e32e..364d0ff 100644
--- a/src/gallium/drivers/radeon/AMDILIntrinsicInfo.cpp
+++ b/src/gallium/drivers/radeon/AMDILIntrinsicInfo.cpp
@@ -51,58 +51,6 @@ AMDILIntrinsicInfo::getName(unsigned int IntrID, Type **Tys,
   return Result;
 }
 
-  static bool
-checkTruncation(const char *Name, unsigned int& Len)
-{
-  const char *ptr = Name + (Len - 1);
-  while(ptr != Name && *ptr != '_') {
-    --ptr;
-  }
-  // We don't want to truncate on atomic instructions
-  // but we do want to enter the check Truncation
-  // section so that we can translate the atomic
-  // instructions if we need to.
-  if (!strncmp(Name, "__atom", 6)) {
-    return true;
-  }
-  if (strstr(ptr, "i32")
-      || strstr(ptr, "u32")
-      || strstr(ptr, "i64")
-      || strstr(ptr, "u64")
-      || strstr(ptr, "f32")
-      || strstr(ptr, "f64")
-      || strstr(ptr, "i16")
-      || strstr(ptr, "u16")
-      || strstr(ptr, "i8")
-      || strstr(ptr, "u8")) {
-    Len = (unsigned int)(ptr - Name);
-    return true;
-  }
-  return false;
-}
-
-// We don't want to support both the OpenCL 1.0 atomics
-// and the 1.1 atomics with different names, so we translate
-// the 1.0 atomics to the 1.1 naming here if needed.
-static char*
-atomTranslateIfNeeded(const char *Name, unsigned int Len) 
-{
-  char *buffer = NULL;
-  if (strncmp(Name, "__atom_", 7))  {
-    // If we are not starting with __atom_, then
-    // go ahead and continue on with the allocation.
-    buffer = new char[Len + 1];
-    memcpy(buffer, Name, Len);
-  } else {
-    buffer = new char[Len + 3];
-    memcpy(buffer, "__atomic_", 9);
-    memcpy(buffer + 9, Name + 7, Len - 7);
-    Len += 2;
-  }
-  buffer[Len] = '\0';
-  return buffer;
-}
-
 unsigned int
 AMDILIntrinsicInfo::lookupName(const char *Name, unsigned int Len) const 
 {
@@ -111,16 +59,8 @@ AMDILIntrinsicInfo::lookupName(const char *Name, unsigned int Len) const
 #undef GET_FUNCTION_RECOGNIZER
   AMDGPUIntrinsic::ID IntrinsicID
     = (AMDGPUIntrinsic::ID)Intrinsic::not_intrinsic;
-  if (checkTruncation(Name, Len)) {
-    char *buffer = atomTranslateIfNeeded(Name, Len);
-    IntrinsicID = getIntrinsicForGCCBuiltin("AMDIL", buffer);
-    delete [] buffer;
-  } else {
-    IntrinsicID = getIntrinsicForGCCBuiltin("AMDIL", Name);
-  }
-  if (!isValidIntrinsic(IntrinsicID)) {
-    return 0;
-  }
+  IntrinsicID = getIntrinsicForGCCBuiltin("AMDIL", Name);
+
   if (IntrinsicID != (AMDGPUIntrinsic::ID)Intrinsic::not_intrinsic) {
     return IntrinsicID;
   }
@@ -146,26 +86,8 @@ AMDILIntrinsicInfo::getDeclaration(Module *M, unsigned IntrID,
     Type **Tys,
     unsigned numTys) const 
 {
+  //Silence a warning
+  AttrListPtr List = getAttributes((AMDGPUIntrinsic::ID)IntrID);
+  (void)List;
   assert(!"Not implemented");
 }
-
-/// Because the code generator has to support different SC versions, 
-/// this function is added to check that the intrinsic being used
-/// is actually valid. In the case where it isn't valid, the 
-/// function call is not translated into an intrinsic and the
-/// fall back software emulated path should pick up the result.
-bool
-AMDILIntrinsicInfo::isValidIntrinsic(unsigned int IntrID) const
-{
-  const AMDILSubtarget &STM = mTM->getSubtarget<AMDILSubtarget>();
-  switch (IntrID) {
-    default:
-      return true;
-    case AMDGPUIntrinsic::AMDIL_convert_f32_i32_rpi:
-    case AMDGPUIntrinsic::AMDIL_convert_f32_i32_flr:
-    case AMDGPUIntrinsic::AMDIL_convert_f32_f16_near:
-    case AMDGPUIntrinsic::AMDIL_convert_f32_f16_neg_inf:
-    case AMDGPUIntrinsic::AMDIL_convert_f32_f16_plus_inf:
-        return STM.calVersion() >= CAL_VERSION_SC_139;
-  };
-}
diff --git a/src/gallium/drivers/radeon/AMDILIntrinsicInfo.h b/src/gallium/drivers/radeon/AMDILIntrinsicInfo.h
index 072c265..9148e49 100644
--- a/src/gallium/drivers/radeon/AMDILIntrinsicInfo.h
+++ b/src/gallium/drivers/radeon/AMDILIntrinsicInfo.h
@@ -41,7 +41,6 @@ namespace llvm {
       Function *getDeclaration(Module *M, unsigned int ID,
           Type **Tys = 0,
           unsigned int numTys = 0) const;
-      bool isValidIntrinsic(unsigned int) const;
   }; // AMDILIntrinsicInfo
 }