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: [PATCH, i386, PR50766] Fix incorrect mem/reg operands order


On Wed, Oct 19, 2011 at 6:21 AM, Uros Bizjak <ubizjak@gmail.com> wrote:
> On Wed, Oct 19, 2011 at 3:07 PM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
>
>> Here is (almost obvous) patch, which fixes PR50766.
>>
>> ChangeLog ?entry:
>> 2011-10-19 ?Kirill Yukhin ?<kirill.yukhin@intel.com>
>>
>> ? ? ? ?* config/i386/i386.md (bmi_bextr_<mode>): Update register/
>> ? ? ? ?memory operand order.
>> ? ? ? ?(bmi2_bzhi_<mode>3): Ditto.
>> ? ? ? ?(bmi2_pdep_<mode>3): Ditto.
>> ? ? ? ?(bmi2_pext_<mode>3): Ditto.

Please mention PR #.

>> Bootstrapped, test (from bug) passing.
>>
>> Could you please have a look?
>
> Please also add the testcase from the PR. You can use ?{ dg-do
> assemble }, but you have to check for BMI2 effective target support.
>
> BTW: I can't find BMI2 instruction reference documentation, so I'm

It is in Intel AVX spec:

http://software.intel.com/en-us/avx/

> just rubberstamping the patch as obvious.
>

-- 
H.J.


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