[Beignet] [PATCH 1/3] GBE: Remove max_limit for struct alignment

Ruiling Song ruiling.song at intel.com
Thu Nov 7 22:20:14 PST 2013


a struct may have vector field (like int8/16), max_limit is meaningless.

Signed-off-by: Ruiling Song <ruiling.song at intel.com>
---
 backend/src/llvm/llvm_passes.cpp |    4 ----
 1 file changed, 4 deletions(-)

diff --git a/backend/src/llvm/llvm_passes.cpp b/backend/src/llvm/llvm_passes.cpp
index 60c9df1..6097f80 100644
--- a/backend/src/llvm/llvm_passes.cpp
+++ b/backend/src/llvm/llvm_passes.cpp
@@ -128,8 +128,6 @@ namespace gbe
 
   uint32_t getAlignmentByte(const ir::Unit &unit, Type* Ty)
   {
-    const uint32_t MAX_ALIGN = 8; //maximum size is 8 for doubles
-
     switch (Ty->getTypeID()) {
       case Type::VoidTyID: NOT_SUPPORTED;
       case Type::VectorTyID:
@@ -153,8 +151,6 @@ namespace gbe
         for(uint32_t subtype = 0; subtype < StrTy->getNumElements(); subtype++)
         {
           maxa = std::max(getAlignmentByte(unit, StrTy->getElementType(subtype)), maxa);
-          if(maxa==MAX_ALIGN)
-            return maxa;
         }
         return maxa;
       }
-- 
1.7.9.5



More information about the Beignet mailing list