* [PATCH 0/2] V2 llvm constant value pseudo size fix
@ 2017-11-13 15:27 Christopher Li
2017-11-13 20:31 ` Dibyendu Majumdar
0 siblings, 1 reply; 3+ messages in thread
From: Christopher Li @ 2017-11-13 15:27 UTC (permalink / raw)
To: Luc Van Oostenryck, Dibyendu Majumdar, Linux-Sparse
Per Luc's feed back. This is the more complete version
of the pseudo-size-value branch.
I add the second patch to allow sparse-llvm to use
the new size store in constant value pseudo.
The whole series is very simple, only two patches.
It directly address the problem of sparse-llvm wasn't
able to get the integer size from constant value pseudo.
I am consider this as better alternative compare to OP_PUSH.
- OP_PUSH break byte code compatibility at source level, this does not.
- OP_PUSH has more instruction (more memory).
- OP_PUSH only cover constant value used in function call.
this cover all usage of constant value.
It pass the "make check" test ( I have to fix a problem with
the bool type which is 1 bit integer). The V1 pass the kernel
checker test with no change to output.
Dibyendu, can you give this series some test since you report
it?
The git url:
https://git.kernel.org/pub/scm/devel/sparse/chrisl/sparse.git/log/?h=pseudo-sized-value-v2
Chris
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/2] V2 llvm constant value pseudo size fix
2017-11-13 15:27 [PATCH 0/2] V2 llvm constant value pseudo size fix Christopher Li
@ 2017-11-13 20:31 ` Dibyendu Majumdar
2017-11-13 23:23 ` Christopher Li
0 siblings, 1 reply; 3+ messages in thread
From: Dibyendu Majumdar @ 2017-11-13 20:31 UTC (permalink / raw)
To: Christopher Li; +Cc: Luc Van Oostenryck, Linux-Sparse
Hi Chris,
On 13 November 2017 at 15:27, Christopher Li <sparse@chrisli.org> wrote:
> I add the second patch to allow sparse-llvm to use
> the new size store in constant value pseudo.
>
> The whole series is very simple, only two patches.
> It directly address the problem of sparse-llvm wasn't
> able to get the integer size from constant value pseudo.
>
> Dibyendu, can you give this series some test since you report
> it?
>
I am happy to test but it might take a week or two, due to other commitments.
Regards
Dibyendu
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/2] V2 llvm constant value pseudo size fix
2017-11-13 20:31 ` Dibyendu Majumdar
@ 2017-11-13 23:23 ` Christopher Li
0 siblings, 0 replies; 3+ messages in thread
From: Christopher Li @ 2017-11-13 23:23 UTC (permalink / raw)
To: Dibyendu Majumdar; +Cc: Luc Van Oostenryck, Linux-Sparse
On Tue, Nov 14, 2017 at 4:31 AM, Dibyendu Majumdar
<mobile@majumdar.org.uk> wrote:
>> Dibyendu, can you give this series some test since you report
>> it?
>>
>
> I am happy to test but it might take a week or two, due to other commitments.
>
No problem. Thank you so much.
Chris
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-11-13 23:23 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-11-13 15:27 [PATCH 0/2] V2 llvm constant value pseudo size fix Christopher Li
2017-11-13 20:31 ` Dibyendu Majumdar
2017-11-13 23:23 ` 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).