From: Dibyendu Majumdar <mobile@majumdar.org.uk>
To: Luc Van Oostenryck <luc.vanoostenryck@gmail.com>
Cc: Linux-Sparse <linux-sparse@vger.kernel.org>
Subject: Re: Some thoughts on the SSA debate
Date: Wed, 13 Sep 2017 19:25:36 +0100 [thread overview]
Message-ID: <CACXZuxeO_rpOFwhjf8vb1SsiHH0NJNneKyoDwF_BzyYyz6n4kw@mail.gmail.com> (raw)
In-Reply-To: <CAExDi1QW8U=rkLX-JMzi=uQG0RTJ-bCjRT46m3CTb7LifKXMQA@mail.gmail.com>
Hi Luc,
On 12 September 2017 at 14:06, Luc Van Oostenryck
<luc.vanoostenryck@gmail.com> wrote:
> On Tue, Sep 12, 2017 at 1:47 PM, Dibyendu Majumdar
>> But the issue
>> right now is that the simplifications are not always correct even with
>> the new SSA so that I have to disable everything anyway.
>
> It's what you do but it's not what I do.
> The other part that is broken is simplify_loads() which try to convert
> more variables after each simplification/CSE steps.
> Since the code is copied from the code for the initial conversion
> it's broken in the same ways.
> I just disabled it and things are then correct.
>
I probably ought to update my project with the latest version you
have. Please let me know if you have an updated minimal changes branch
that I can apply.
Thanks and Regards
Dibyendu
next prev parent reply other threads:[~2017-09-13 18:25 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-08 10:28 Some thoughts on the SSA debate Dibyendu Majumdar
2017-09-09 2:06 ` Luc Van Oostenryck
2017-09-12 10:06 ` Dibyendu Majumdar
2017-09-12 10:48 ` Luc Van Oostenryck
2017-09-12 11:47 ` Dibyendu Majumdar
2017-09-12 13:06 ` Luc Van Oostenryck
2017-09-13 18:25 ` Dibyendu Majumdar [this message]
2017-09-13 19:10 ` Luc Van Oostenryck
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CACXZuxeO_rpOFwhjf8vb1SsiHH0NJNneKyoDwF_BzyYyz6n4kw@mail.gmail.com \
--to=mobile@majumdar.org.uk \
--cc=linux-sparse@vger.kernel.org \
--cc=luc.vanoostenryck@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).