This is the mail archive of the cgen@sources.redhat.com mailing list for the CGEN project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch][rfa]: Decoding (not-so) ambiguous insns in sid/sim


Frank Ch. Eigler wrote:

>
>You mean that there are actually four separate cases to consider?
>
>p0=X, p1=Y, num_insns=N
>
>	X=0 / Y=0	==> result = 0
>	X<N / Y=0	==> 0 < result < N
>	X=N / Y=0	==> result = 0
>	X!=0 / Y!=0	==> result >= N
>
>Okay.  Tricky.
>
Yup -- you got it. And also vice-versa with respect to p0 and p1.

>>Can I consider your response to be approval to commit?
>>
>
>Sure.
>
ok.

>>What about "filter-harmlessly-ambiguous-insns"? 
>>Should I remove it from insn.scm?
>>
>
>No opinion.
>
I've thought about this a bit. Since the function is no longer used, has 
a misleading name, and is very similar to the new 
"filter-non-specialized-ambiguous-insns", I think I'll remove it.

Dave


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]