search for: regusageinfocollector

...local). To achive it I have modified TargetFrameLowering::determineCalleeSaves() to return early for function which satisfies if (F->hasLocalLinkage() && !F->hasAddressTaken()) and also reflecting the fact that for local function there are no caller saved registers I am also changing RegUsageInfoCollector.cpp to not to mark regiseters as callee saved in RegMask due to CC with follwoing change in code: if (!F->hasLocalLinkage() || F->hasAddressTaken()) { const uint32_t *CallPreservedMask = TRI->getCallPreservedMask(MF, MF.getFunction()->getCallingConv()); // Set calle...

...odified TargetFrameLowering::determineCalleeSaves() to return > early for > function which satisfies if (F->hasLocalLinkage() && > !F->hasAddressTaken()) and > also reflecting the fact that for local function there are no caller saved > registers > I am also changing RegUsageInfoCollector.cpp to not to mark regiseters as > callee > saved in RegMask due to CC with follwoing change in code: > > if (!F->hasLocalLinkage() || F->hasAddressTaken()) { > const uint32_t *CallPreservedMask = > TRI->getCallPreservedMask(MF, MF.getFunction()->getCalli...

...ing::determineCalleeSaves() to return >> early for >> function which satisfies if (F->hasLocalLinkage() && >> !F->hasAddressTaken()) and >> also reflecting the fact that for local function there are no caller >> saved registers >> I am also changing RegUsageInfoCollector.cpp to not to mark regiseters as >> callee >> saved in RegMask due to CC with follwoing change in code: >> >> if (!F->hasLocalLinkage() || F->hasAddressTaken()) { >> const uint32_t *CallPreservedMask = >> TRI->getCallPreservedMask(MF, MF.ge...

...t > I have modified TargetFrameLowering::determineCalleeSaves() to return early for > function which satisfies if (F->hasLocalLinkage() && !F->hasAddressTaken()) and > also reflecting the fact that for local function there are no caller saved registers > I am also changing RegUsageInfoCollector.cpp to not to mark regiseters as callee > saved in RegMask due to CC with follwoing change in code: > > if (!F->hasLocalLinkage() || F->hasAddressTaken()) { > const uint32_t *CallPreservedMask = > TRI->getCallPreservedMask(MF, MF.getFunction()->getCallingCo...

...s() to return >>> early for >>> function which satisfies if (F->hasLocalLinkage() && >>> !F->hasAddressTaken()) and >>> also reflecting the fact that for local function there are no caller >>> saved registers >>> I am also changing RegUsageInfoCollector.cpp to not to mark regiseters >>> as callee >>> saved in RegMask due to CC with follwoing change in code: >>> >>> if (!F->hasLocalLinkage() || F->hasAddressTaken()) { >>> const uint32_t *CallPreservedMask = >>> TRI->getCal...

...t;> early for >>>> function which satisfies if (F->hasLocalLinkage() && >>>> !F->hasAddressTaken()) and >>>> also reflecting the fact that for local function there are no caller >>>> saved registers >>>> I am also changing RegUsageInfoCollector.cpp to not to mark regiseters >>>> as callee >>>> saved in RegMask due to CC with follwoing change in code: >>>> >>>> if (!F->hasLocalLinkage() || F->hasAddressTaken()) { >>>> const uint32_t *CallPreservedMask = >>>&gt...

...aves() to return early for >>>>>> function which satisfies if (F->hasLocalLinkage() && !F->hasAddressTaken()) and >>>>>> also reflecting the fact that for local function there are no caller saved registers >>>>>> I am also changing RegUsageInfoCollector.cpp to not to mark regiseters as callee >>>>>> saved in RegMask due to CC with follwoing change in code: >>>>>> >>>>>> if (!F->hasLocalLinkage() || F->hasAddressTaken()) { >>>>>> const uint32_t *CallPreservedMask =...

...nctional issue. The analysis must run in order to be compatible with other compilers. 2. IPRA passes are not enabled by default and when they are enabled many tests fail due to various reasons (mainly because the CallGraph bottom up approach). 3. The manipulated RegMasks generated using RegUsageInfoCollector are very different than the manipulated RegMasks that I need. It will be an abuse to change the current pass implementation. So new pass is required. 4. When dumping the MIR after running IPRA passes, assertion is raised because the RegMask name is unknown (same issue that I face with my solu...

...nctional issue. The analysis must run in order to be compatible with other compilers. 2. IPRA passes are not enabled by default and when they are enabled many tests fail due to various reasons (mainly because the CallGraph bottom up approach). 3. The manipulated RegMasks generated using RegUsageInfoCollector are very different than the manipulated RegMasks that I need. It will be an abuse to change the current pass implementation. So new pass is required. 4. When dumping the MIR after running IPRA passes, assertion is raised because the RegMask name is unknown (same issue that I face with my solu...

...st part for this week was to get lit and FileCheck work as you > expect them to work, specially when analysis pass prints info on stdio and > there is also a output file generated by llc or opt command. > > So here is brief summary : > > Implementation: > ============ > > RegUsageInfoCollector is now Calling Convention aware so that RegMask does > not mark callee saved register as clobbered register. Due to this register > allocator can use callee saved register for caller. > PhysicalRegisterUsageInfo.cpp renamed to RegisterUsageInfo.cpp. > StringMap used in RegisterUsageInfo...

Dear community, This is to brief you the progress of Interprocedural Register Allocation, for those who are interested to see the progress in terms of code please consider http://reviews.llvm.org/D20769 This patch contains simple infrastructure to propagate register usage information of callee to caller in call graph. The code generation order is changed to follow bottom up order on call graph ,

I have tried out the following code which examines each call site in a module for tail call and do not perform optimization in such case: On Wed, Jun 29, 2016 at 12:34 AM, Mehdi Amini <mehdi.amini at apple.com> wrote: > > On Jun 28, 2016, at 3:01 PM, Matthias Braun <matze at braunis.de> wrote: > > > On Jun 28, 2016, at 11:34 AM, Mehdi Amini via llvm-dev < >

...and FileCheck work as you >> expect them to work, specially when analysis pass prints info on stdio and >> there is also a output file generated by llc or opt command. >> >> So here is brief summary : >> >> Implementation: >> ============ >> >> RegUsageInfoCollector is now Calling Convention aware so that RegMask >> does not mark callee saved register as clobbered register. Due to this >> register allocator can use callee saved register for caller. >> PhysicalRegisterUsageInfo.cpp renamed to RegisterUsageInfo.cpp. >> StringMap used in R...

...nctional issue. The analysis must run in order to be compatible with other compilers. 2. IPRA passes are not enabled by default and when they are enabled many tests fail due to various reasons (mainly because the CallGraph bottom up approach). 3. The manipulated RegMasks generated using RegUsageInfoCollector are very different than the manipulated RegMasks that I need. It will be an abuse to change the current pass implementation. So new pass is required. 4. When dumping the MIR after running IPRA passes, assertion is raised because the RegMask name is unknown (same issue that I face with my solu...

> On Jun 28, 2016, at 3:01 PM, Matthias Braun <matze at braunis.de> wrote: > >> >> On Jun 28, 2016, at 11:34 AM, Mehdi Amini via llvm-dev <llvm-dev at lists.llvm.org <mailto:llvm-dev at lists.llvm.org>> wrote: >> >> >> >> Sent from my iPhone >> >> On Jun 28, 2016, at 2:27 PM, Matthias Braun <matze at braunis.de

...nctional issue. The analysis must run in order to be compatible with other compilers. 2. IPRA passes are not enabled by default and when they are enabled many tests fail due to various reasons (mainly because the CallGraph bottom up approach). 3. The manipulated RegMasks generated using RegUsageInfoCollector are very different than the manipulated RegMasks that I need. It will be an abuse to change the current pass implementation. So new pass is required. 4. When dumping the MIR after running IPRA passes, assertion is raised because the RegMask name is unknown (same issue that I face with my solu...

...ly when analysis pass prints info on stdio >>>> and there is also a output file generated by llc or opt command. >>>> >>>> So here is brief summary : >>>> >>>> Implementation: >>>> ============ >>>> >>>> RegUsageInfoCollector is now Calling Convention aware so that RegMask >>>> does not mark callee saved register as clobbered register. Due to this >>>> register allocator can use callee saved register for caller. >>>> PhysicalRegisterUsageInfo.cpp renamed to RegisterUsageInfo.cpp. >...

Vivek, I apologize if you took my original email as a request for implementation, I meant to be asking what is already available, I think the answer to that is the ‘preserves_most’ and ‘preserves_all’ attributes, but I will also Use ‘regmask’ if those prove to be too sub-optimal. I am still interested in figuring out the necessary and sufficient conditions For LLC to do optimal IPRA

...at gmail.com> for bringing this > to notice. > > For example if CL is only clobbered than CH should not be marked clobbered > but CX, RCX and ECX should be mark clobbered. Previously for each modified > register all of its aliases are marked clobbered by markRegClobbred() in > RegUsageInfoCollector.cpp but that is wrong because when CL is clobbered > then MRI::isPhysRegModified() will return true for CL, CX, ECX, RCX which > is correct behavior but then for CX, EXC, RCX we mark CH also clobbered as > CH is aliased to CX,ECX,RCX so markRegClobbred() is not required because > isPhys...

Hello LLVM Developers, Please feel free to send any ideas that you can think to improve current IPRA. I will work on it and if possible I will implement that. Please consider summary of work done during this week. Implementation: ============ The reviews requests has been updated to reflect the reviews. Testing: ===== To get more benefit from IPRA I experimented it with LTO and results