linux-sparse.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Adding an option to sparse to disable the IR simplifications
@ 2017-03-22 19:56 Dibyendu Majumdar
  2017-03-22 20:53 ` Dibyendu Majumdar
  0 siblings, 1 reply; 6+ messages in thread
From: Dibyendu Majumdar @ 2017-03-22 19:56 UTC (permalink / raw)
  To: Linux-Sparse

Hi,

With the progress made with spars-llvm in recent weeks we are now
beginning to hit bugs in the simplifications carried out by the
linearizer. So it is useful I think to have an ability to turn off
these modifications - perhaps by a command line argument. This will
enable testing of sparse-llvm with and without the simplifications,
and help identify issues caused by the latter phase.

Thanks and Regards
Dibyendu

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Adding an option to sparse to disable the IR simplifications
  2017-03-22 19:56 Adding an option to sparse to disable the IR simplifications Dibyendu Majumdar
@ 2017-03-22 20:53 ` Dibyendu Majumdar
  2017-08-14 16:38   ` Fwd: " Dibyendu Majumdar
  0 siblings, 1 reply; 6+ messages in thread
From: Dibyendu Majumdar @ 2017-03-22 20:53 UTC (permalink / raw)
  To: Linux-Sparse

On 22 March 2017 at 19:56, Dibyendu Majumdar <mobile@majumdar.org.uk> wrote:
> With the progress made with spars-llvm in recent weeks we are now
> beginning to hit bugs in the simplifications carried out by the
> linearizer. So it is useful I think to have an ability to turn off
> these modifications - perhaps by a command line argument. This will
> enable testing of sparse-llvm with and without the simplifications,
> and help identify issues caused by the latter phase.
>

It appears that there is already an optimize flag that is set by -O
option. Perhaps this flag can be utilized to control the level of
simplifications attempted.

Thanks and Regards
Dibyendu

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Fwd: Adding an option to sparse to disable the IR simplifications
  2017-03-22 20:53 ` Dibyendu Majumdar
@ 2017-08-14 16:38   ` Dibyendu Majumdar
  2017-08-14 16:46     ` Luc Van Oostenryck
  0 siblings, 1 reply; 6+ messages in thread
From: Dibyendu Majumdar @ 2017-08-14 16:38 UTC (permalink / raw)
  To: Linux-Sparse, Christopher Li, Luc Van Oostenryck

Hi,

I suggested a while back that it might be useful to add an option to
control the simplifications.

In dmrC the simplification phase is turned off by default. But if you
supply -O1 option then it is turned on. This ensures that  I can test
both the unsimplified and simplified versions.


Regards
Dibyendu

---------- Forwarded message ----------
From: Dibyendu Majumdar <mobile@majumdar.org.uk>
Date: 22 March 2017 at 20:53
Subject: Re: Adding an option to sparse to disable the IR simplifications
To: Linux-Sparse <linux-sparse@vger.kernel.org>


On 22 March 2017 at 19:56, Dibyendu Majumdar <mobile@majumdar.org.uk> wrote:
> With the progress made with spars-llvm in recent weeks we are now
> beginning to hit bugs in the simplifications carried out by the
> linearizer. So it is useful I think to have an ability to turn off
> these modifications - perhaps by a command line argument. This will
> enable testing of sparse-llvm with and without the simplifications,
> and help identify issues caused by the latter phase.
>

It appears that there is already an optimize flag that is set by -O
option. Perhaps this flag can be utilized to control the level of
simplifications attempted.

Thanks and Regards
Dibyendu

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Adding an option to sparse to disable the IR simplifications
  2017-08-14 16:38   ` Fwd: " Dibyendu Majumdar
@ 2017-08-14 16:46     ` Luc Van Oostenryck
  2017-08-14 16:59       ` Dibyendu Majumdar
  0 siblings, 1 reply; 6+ messages in thread
From: Luc Van Oostenryck @ 2017-08-14 16:46 UTC (permalink / raw)
  To: Dibyendu Majumdar; +Cc: Linux-Sparse, Christopher Li

On Mon, Aug 14, 2017 at 6:38 PM, Dibyendu Majumdar
<mobile@majumdar.org.uk> wrote:
> Hi,
>
> I suggested a while back that it might be useful to add an option to
> control the simplifications.
>
> In dmrC the simplification phase is turned off by default. But if you
> supply -O1 option then it is turned on. This ensures that  I can test
> both the unsimplified and simplified versions.

Yes, it's something that more useful now than it was months ago.
I think I'll first move some code around though.

-- Luc

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Adding an option to sparse to disable the IR simplifications
  2017-08-14 16:46     ` Luc Van Oostenryck
@ 2017-08-14 16:59       ` Dibyendu Majumdar
  2017-08-14 17:09         ` Christopher Li
  0 siblings, 1 reply; 6+ messages in thread
From: Dibyendu Majumdar @ 2017-08-14 16:59 UTC (permalink / raw)
  To: Luc Van Oostenryck; +Cc: Linux-Sparse, Christopher Li

On 14 August 2017 at 17:46, Luc Van Oostenryck
<luc.vanoostenryck@gmail.com> wrote:
> On Mon, Aug 14, 2017 at 6:38 PM, Dibyendu Majumdar
> <mobile@majumdar.org.uk> wrote:
>> I suggested a while back that it might be useful to add an option to
>> control the simplifications.
>>
>> In dmrC the simplification phase is turned off by default. But if you
>> supply -O1 option then it is turned on. This ensures that  I can test
>> both the unsimplified and simplified versions.
>
> Yes, it's something that more useful now than it was months ago.
> I think I'll first move some code around though.
>

We can perhaps also move towards a granular level of control as in gcc
or clang - i.e. enable or disable specific simplifications.

Regards
Dibyendu

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Adding an option to sparse to disable the IR simplifications
  2017-08-14 16:59       ` Dibyendu Majumdar
@ 2017-08-14 17:09         ` Christopher Li
  0 siblings, 0 replies; 6+ messages in thread
From: Christopher Li @ 2017-08-14 17:09 UTC (permalink / raw)
  To: Dibyendu Majumdar; +Cc: Luc Van Oostenryck, Linux-Sparse

On Mon, Aug 14, 2017 at 12:59 PM, Dibyendu Majumdar
<mobile@majumdar.org.uk> wrote:
>
> We can perhaps also move towards a granular level of control as in gcc
> or clang - i.e. enable or disable specific simplifications.
>

Yes,  we can also do that. I I think the the back end should have some way
to control what kind of the optimization it want and what level it want etc.

I plan to release 0.5.1 finial on Wednesday. Then the merge windows is
open again. We can start to work on getting the IR right, getting SSA right etc.
I also have some patch want to merged as well like the ptrlist ref count.

It can even have a post release branch for experiment. After release
this branch will merge back to master.


Chris

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-08-14 17:09 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-22 19:56 Adding an option to sparse to disable the IR simplifications Dibyendu Majumdar
2017-03-22 20:53 ` Dibyendu Majumdar
2017-08-14 16:38   ` Fwd: " Dibyendu Majumdar
2017-08-14 16:46     ` Luc Van Oostenryck
2017-08-14 16:59       ` Dibyendu Majumdar
2017-08-14 17:09         ` Christopher Li

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).