search for: grandmaison

...to it ?) - we modify the copy - we set the node’s new costs. But what is supposed to happen there, especially in the CostAllocator part ? Could you shed some light there ? Thanks, Arnaud From: Lang Hames [mailto:lhames at gmail.com] Sent: 10 September 2014 19:26 To: Arnaud De Grandmaison Subject: Re: Leaks in PBQPBuilderWithCoalescing::build ? Thanks Arnaud! - Lang. On Wed, Sep 10, 2014 at 11:23 AM, Arnaud A. de Grandmaison <arnaud.degrandmaison at arm.com> wrote: Hi Lang, For your information, the leak sanitizer found something when I committed my patch:...

...become null > (probably on a harvesting schedule, rather than with a direct callback as > is currently implemented))​ > pbqp_leak.diff > <https://docs.google.com/file/d/0B0jpkch3iC_7TXFVU2hCcUpfZXM/edit?usp=drive_web> > ​ > > On Wed, Sep 10, 2014 at 3:19 PM, Arnaud A. de Grandmaison < > arnaud.degrandmaison at arm.com> wrote: > >> Hi Lang, >> >> >> >> In PBQPBuilderWithCoalescing::build, around line 360, we have code >> looking like: >> >> >> >> … >> >> PBQP::Vector newCosts(g.getNodeCosts(node...

...nes for both to ensure shrink-wrapping is doing the right thing. + %0 = load i32, i32* @g1, align 4 Please use opt -instnamer to get rid of the numbered variables. Those are a pain when updating the tests :). Other than that LGTM! Cheers, -Quentin > On Nov 20, 2015, at 6:31 AM, Arnaud A. de Grandmaison <arnaud.degrandmaison at arm.com> wrote: > > +CC llvm-dev > >> -----Original Message----- >> From: Arnaud A. de Grandmaison [mailto:arnaud.degrandmaison at arm.com] >> Sent: 20 November 2015 15:28 >> To: 'qcolombet at apple.com' >> Cc: 'hai...

Hi Lang, Thanks a lot for taking time to look into this. I will test the fix soon and let you know the results. Cheers, -- Arnaud de Grandmaison On Tuesday, April 03, 2012 17:30:33 Lang Hames wrote: > Hi Arnaud, > > Apologies for the delayed reply. > > Thank you for the excellent test case - it exposed a subtle bug in the > colorability heuristic. This has been fixed in r153958. > > In case you are curious, the b...

...e in C, as well as the resulting assembly output, with and without the patch applied, for arm and x86_32 targets. You will note the loop instructions count can be reduced by 30% in several cases. The patch could probably be made smarter : I am welcoming all suggestions. Best Regards, -- Arnaud de Grandmaison -------------- next part -------------- A non-text attachment was scrubbed... Name: IndVarSimplify-nativeType.patch Type: application/octet-stream Size: 2091 bytes Desc: IndVarSimplify-nativeType.patch URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20110313/71c8b9cc/attachment.obj&gt...

...py only when needed. This had no visible effect. Although I am deriving from PBQPBuilder, the PBQP seems to be coalescing some register copy, without taking into account that the source or dest reg may have different constraints. In which part of pbqp would this be happening ? Cheers, -- Arnaud de Grandmaison On 04/05/2012 05:23 PM, Arnaud de Grandmaison wrote: > Hi Lang, > > Thanks a lot for taking time to look into this. I will test the fix soon and > let you know the results. > > Cheers, > -- > Arnaud de Grandmaison > > On Tuesday, April 03, 2012 17:30:33 Lang Hames wr...

...long as you don't have a circular chain of paired variables (a,b,c,...,a) with all-infinite spill costs you should be safe. Can you send me a PBQP graph for your failing test case? I'll see if I can figure out where the solver is going wrong. - Lang. > Cheers, > -- > Arnaud de Grandmaison > > On 04/05/2012 05:23 PM, Arnaud de Grandmaison wrote: > > Hi Lang, > > > > Thanks a lot for taking time to look into this. I will test the fix soon > and > > let you know the results. > > > > Cheers, > > -- > > Arnaud de Grandmaison > &...

...You need to make sure the Flask constraint is seen first, even before using the requirements.txt so on the command line before loading the requirements.txt. > -----Original Message----- > From: Renato Golin [mailto:renato.golin at linaro.org] > Sent: 30 May 2016 20:42 > To: Arnaud De Grandmaison > Cc: Kristof Beyls; nd; llvm-dev at lists.llvm.org > Subject: Re: LNT General Failure > > On 30 May 2016 at 12:25, Arnaud De Grandmaison > <Arnaud.DeGrandmaison at arm.com> wrote: > > I confirm this is due to the latest Flask --- Flask-0.11 was released > > this w...

A follow-up question: Is there a way to make different passes accept one command line option that will affect all of them? For example, I'd like to have a -optStrength parameter, that can be given to all of my passes simultaneously. Thanks a lot, Guy Arnaud Allard de Grandmaison wrote: > > Hi Harel, > > Several existing passes can take command line arguments. Have a look at > lib/Transforms/Scalar/LoopUnrollPass.cpp for example. Its command line > arguments are defined using the cl::opt objects. > > Best regards, > -- > Arnaud de Grandmaiso...

Hi Quentin, After shrink-wrapping was enabled as default on AArch64, llc has a seg fault when compiling the attached .ll file on AArch64. My command is llc -mcpu=cortex-a57 bug.ll Best, Haicheng -------------- next part -------------- A non-text attachment was scrubbed... Name: bug.ll Type: application/octet-stream Size: 8983 bytes Desc: not available URL:

...this approach, but did not want to dive in the constraint handling for now. The PBQP path seemed easier at first sight --- and was easy to setup. And I always wanted to give a try to the pbqp :) I will add the hook to the pbqp and propose a patch if this looks clean enough. Thanks, -- Arnaud de Grandmaison -----Original Message----- From: Jakob Stoklund Olesen [mailto:stoklund at 2pi.dk] Sent: Monday, June 06, 2011 9:28 PM To: Peter Lawrence Cc: Arnaud Allard de Grandmaison; llvmdev at cs.uiuc.edu Subject: Re: [LLVMdev] PBQP & register pairing On Jun 6, 2011, at 12:14 PM, Peter Lawrence wrote:...

On Mar 13, 2011, at 2:01 PM, Arnaud Allard de Grandmaison wrote: > Hi all, > > The IndVarSimplify pass seems to be too aggressive when it enlarge the induction variable type ; this can pessimize the generated code when the new induction variable size is not natively supported by the target. This is probably not an issue for x86_64, which suppor...

You could make the cl:opt object visible outside one of your passes, and use it in your other passes : in other words, it is a global variable shared between several files. Best regards, -- Arnaud de Grandmaison -----Original Message----- From: llvmdev-bounces at cs.uiuc.edu [mailto:llvmdev-bounces at cs.uiuc.edu] On Behalf Of guyadini Sent: Thursday, January 27, 2011 12:38 PM To: llvmdev at cs.uiuc.edu Subject: Re: [LLVMdev] Passing command line arguments to optimization passes A follow-up question: Is...

Hi Lang, I have reduced the testcase as much as possible. The log of the run and the dumped graphes are attached. Cheers, -- Arnaud de Grandmaison On Tuesday, March 27, 2012 01:20:35 Lang Hames wrote: > Hi Arnaud, > > Thanks for attaching those files. I'll take a look at them. > > Commit r153483 adds an option to the PBQP allocator, > "-pbqp-dump-graphs", to dump the PBQP graph for each round of each > fu...

I hope I've fixed this properly just now in http://llvm.org/viewvc/llvm-project?view=revision&revision=271274. Fingers crossed that the bots will come back now... On 30 May 2016, at 21:01, Arnaud De Grandmaison <Arnaud.DeGrandmaison at arm.com<mailto:Arnaud.DeGrandmaison at arm.com>> wrote: I honestly do not know how to fix that --- I would otherwise I've committed a fix. I've been able to hack it locally exploiting the very same lit limitation then the one we're stumbling on (i....

...ud, That sounds great. I look forward to seeing a patch. You may also look forward to big performance improvements in the PBQP allocator: I'm working on updates which will improve compile speeds and massively reduce memory use. Regards, Lang. On Tue, Jun 7, 2011 at 7:02 PM, Arnaud Allard de Grandmaison < Arnaud.AllardDeGrandMaison at dibcom.com> wrote: > > I also considered this approach, but did not want to dive in the constraint > handling for now. > > The PBQP path seemed easier at first sight --- and was easy to setup. And I > always wanted to give a try to the pbqp :...

...ons. With the fix from r153958 applied the solver now finds a zero-cost solution for the test case you sent me. This should translate to a valid register allocation for your test case. Please try it out and let me know if it works for you. Cheers, Lang. On Tue, Mar 27, 2012 at 5:05 AM, Arnaud de Grandmaison < arnaud.allarddegrandmaison at parrot.com> wrote: > Hi Lang, > > I have reduced the testcase as much as possible. The log of the run and the > dumped graphes are attached. > > Cheers, > -- > Arnaud de Grandmaison > > On Tuesday, March 27, 2012 01:20:35 Lang Ham...

...Beyls via llvm-dev <llvm-dev at lists.llvm.org> wrote: > > I hope I've fixed this properly just now in http://llvm.org/viewvc/llvm-project?view=revision&revision=271274. > Fingers crossed that the bots will come back now... > >> On 30 May 2016, at 21:01, Arnaud De Grandmaison <Arnaud.DeGrandmaison at arm.com> wrote: >> >> I honestly do not know how to fix that --- I would otherwise I've committed a fix. >> >> I've been able to hack it locally exploiting the very same lit limitation then the one we're stumbling on (i.e it does...

...with my patch : the induction variable should be counting up, and it could be zero extended before the call to f. This again points to Isel, but to a different area, as everything is taking place in the same BB. Is this some known issue ? I could not find a bug report matching this. -- Arnaud de Grandmaison -----Original Message----- From: Eli Friedman [mailto:eli.friedman at gmail.com] Sent: Sunday, March 13, 2011 11:08 PM To: Arnaud Allard de Grandmaison Cc: llvmdev at cs.uiuc.edu Subject: Re: [LLVMdev] IndVarSimplify too aggressive ? On Sun, Mar 13, 2011 at 5:01 PM, Arnaud Allard de Grandmaison &...

Oops. Forgot the mailing list in the reply. -- Arnaud de Grandmaison ________________________________________ From: Arnaud Allard de Grandmaison Sent: Thursday, June 03, 2010 9:57 PM To: Jakob Stoklund Olesen Subject: RE: [LLVMdev] Unused argument registers can not be reused ? I have finally been able to make some progress : the bug has been introduced/revealed by...