* Reverting OP_SYMADDR in sparse-llvm
@ 2017-03-21 10:59 Dibyendu Majumdar
2017-03-21 12:10 ` Luc Van Oostenryck
0 siblings, 1 reply; 2+ messages in thread
From: Dibyendu Majumdar @ 2017-03-21 10:59 UTC (permalink / raw)
To: Linux-Sparse
Hi Luc,
In the last two sets of patches I have noticed that you have reverted
the change related to OP_SYMADDR. I would like to understand what fix
in sparse-llvm made this change unnecessary.
Thanks and Regards
Dibyendu
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: Reverting OP_SYMADDR in sparse-llvm
2017-03-21 10:59 Reverting OP_SYMADDR in sparse-llvm Dibyendu Majumdar
@ 2017-03-21 12:10 ` Luc Van Oostenryck
0 siblings, 0 replies; 2+ messages in thread
From: Luc Van Oostenryck @ 2017-03-21 12:10 UTC (permalink / raw)
To: Dibyendu Majumdar; +Cc: Linux-Sparse
On Tue, Mar 21, 2017 at 10:59:47AM +0000, Dibyendu Majumdar wrote:
> Hi Luc,
>
> In the last two sets of patches I have noticed that you have reverted
> the change related to OP_SYMADDR. I would like to understand what fix
> in sparse-llvm made this change unnecessary.
Yes, I forgot to state that in the changelog.
It's not that it is now unnecessary but there are case where
it's wrong.
It's certainly one of the things I'll look further because
I think that it is better to have these SYMADDR.
-- Luc
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-03-21 12:12 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-21 10:59 Reverting OP_SYMADDR in sparse-llvm Dibyendu Majumdar
2017-03-21 12:10 ` Luc Van Oostenryck
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).