[llvm] r276193 - [AArch64] Register AArch64LoadStoreOptimizer so it can be run by llc -run-pass. NFCI.

[Geoff Berry via llvm-commits]

Laziness? :)  Perhaps Tim can chime in if there are any downsides to 
going ahead and doing this for all the aarch64 target passes.


On 7/21/2016 5:06 AM, Diana Picus wrote:
> Is there any reason why we don't do this for other passes as well
> (e.g. AArch64BranchRelaxationPass etc) ?
>
> Diana
>
> On 21 July 2016 at 00:45, Geoff Berry via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>> Author: gberry
>> Date: Wed Jul 20 16:45:58 2016
>> New Revision: 276193
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=276193&view=rev
>> Log:
>> [AArch64] Register AArch64LoadStoreOptimizer so it can be run by llc -run-pass. NFCI.
>>
>> Modified:
>>      llvm/trunk/lib/Target/AArch64/AArch64.h
>>      llvm/trunk/lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp
>>      llvm/trunk/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp
>>      llvm/trunk/lib/Target/AArch64/AArch64TargetMachine.cpp
>>
>> Modified: llvm/trunk/lib/Target/AArch64/AArch64.h
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64.h?rev=276193&r1=276192&r2=276193&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Target/AArch64/AArch64.h (original)
>> +++ llvm/trunk/lib/Target/AArch64/AArch64.h Wed Jul 20 16:45:58 2016
>> @@ -47,6 +47,7 @@ FunctionPass *createAArch64CleanupLocalD
>>   FunctionPass *createAArch64CollectLOHPass();
>>
>>   void initializeAArch64ExpandPseudoPass(PassRegistry&);
>> +void initializeAArch64LoadStoreOptPass(PassRegistry&);
>>   } // end namespace llvm
>>
>>   #endif
>>
>> Modified: llvm/trunk/lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp?rev=276193&r1=276192&r2=276193&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp (original)
>> +++ llvm/trunk/lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp Wed Jul 20 16:45:58 2016
>> @@ -23,10 +23,6 @@
>>   #include "llvm/Support/MathExtras.h"
>>   using namespace llvm;
>>
>> -namespace llvm {
>> -void initializeAArch64ExpandPseudoPass(PassRegistry &);
>> -}
>> -
>>   #define AARCH64_EXPAND_PSEUDO_NAME "AArch64 pseudo instruction expansion pass"
>>
>>   namespace {
>>
>> Modified: llvm/trunk/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp?rev=276193&r1=276192&r2=276193&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp (original)
>> +++ llvm/trunk/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp Wed Jul 20 16:45:58 2016
>> @@ -55,10 +55,6 @@ static cl::opt<bool> EnableNarrowLdMerge
>>                                            cl::init(false),
>>                                            cl::desc("Enable narrow load merge"));
>>
>> -namespace llvm {
>> -void initializeAArch64LoadStoreOptPass(PassRegistry &);
>> -}
>> -
>>   #define AARCH64_LOAD_STORE_OPT_NAME "AArch64 load / store optimization pass"
>>
>>   namespace {
>>
>> Modified: llvm/trunk/lib/Target/AArch64/AArch64TargetMachine.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64TargetMachine.cpp?rev=276193&r1=276192&r2=276193&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Target/AArch64/AArch64TargetMachine.cpp (original)
>> +++ llvm/trunk/lib/Target/AArch64/AArch64TargetMachine.cpp Wed Jul 20 16:45:58 2016
>> @@ -116,6 +116,7 @@ extern "C" void LLVMInitializeAArch64Tar
>>     auto PR = PassRegistry::getPassRegistry();
>>     initializeGlobalISel(*PR);
>>     initializeAArch64ExpandPseudoPass(*PR);
>> +  initializeAArch64LoadStoreOptPass(*PR);
>>   }
>>
>>   //===----------------------------------------------------------------------===//
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits

-- 
Geoff Berry
Employee of Qualcomm Innovation Center, Inc.
  Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project