[llvm] 57b259a - [Passes] Enable the relative lookup table converter pass on aarch64

Martin Storsjö via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 14 03:16:18 PDT 2021


Author: Martin Storsjö
Date: 2021-04-14T13:15:41+03:00
New Revision: 57b259a852a6383880f5d0875d848420bb3c2945

URL: https://github.com/llvm/llvm-project/commit/57b259a852a6383880f5d0875d848420bb3c2945
DIFF: https://github.com/llvm/llvm-project/commit/57b259a852a6383880f5d0875d848420bb3c2945.diff

LOG: [Passes] Enable the relative lookup table converter pass on aarch64

After d5c5cf5ce8d921fc8c5e1b608c298a1ffa688d37, it should work fine
for aarch64 on COFF too. (It was disabled when the patch was
(re)applied in e96df3e531f506eea75da0f13d0f8aa9a267f975, pending
that fix.)

Added: 
    

Modified: 
    llvm/include/llvm/CodeGen/BasicTTIImpl.h

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/CodeGen/BasicTTIImpl.h b/llvm/include/llvm/CodeGen/BasicTTIImpl.h
index c4ebaacd3271..7e7205967460 100644
--- a/llvm/include/llvm/CodeGen/BasicTTIImpl.h
+++ b/llvm/include/llvm/CodeGen/BasicTTIImpl.h
@@ -397,11 +397,6 @@ class BasicTTIImplBase : public TargetTransformInfoImplCRTPBase<T> {
     if (!TargetTriple.isArch64Bit())
       return false;
 
-    // TODO: Triggers an issue in aarch64, so temporarily disable it.
-    // See https://reviews.llvm.org/D99572 for more information.
-    if (TargetTriple.getArch() == Triple::aarch64)
-      return false;
-
     return true;
   }
 


        


More information about the llvm-commits mailing list