This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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: RFC: PATCH: Add -m8bit-idiv for x86

On Thu, Sep 16, 2010 at 9:20 AM, Paolo Bonzini <bonzini@gnu.org> wrote:
> On 09/15/2010 07:16 PM, H.J. Lu wrote:
>>>
>>> > ?I think adding an unspec to the parallel would be less hacky than
>>> > reversing
>>> > ?the div/mod.
>>> >
>>
>> There are places which check DIV/UDIV.
>
> I said "adding", like
>
> ?(parallel [(set ... (div ...))
> ? ? ? ? ? ? (set ... (mod ...))
> ? ? ? ? ? ? (unspec [] DIV_ALREADY_SPLIT)])

I agree with Paolo.

BTW: You should write:

             (unspec [(const_int 0)] DIV_ALREADY_SPLIT)

otherwise sky will fall down.

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