> Can you please split the patch to three independent parts, prefetchw
> handling, rdseed and adxx. I got lost in the patch the second time.
Sure!
> I propose we start with prefetch. You are still changing prefetch
> patterns, while it should be enough to only change prefetch expander.
Actually, t
On Tue, Jul 24, 2012 at 2:09 PM, Kirill Yukhin wrote:
> Uros, Jakub, thanks fr the inputs!
>
> I've refactored the patch appliying (hopefully) all of your inputs.
>
> It is attached.
Can you please split the patch to three independent parts, prefetchw
handling, rdseed and adxx. I got lost in the
On Thu, Jul 19, 2012 at 11:51 AM, Jakub Jelinek wrote:
> I think it is fine to change existing mm3dnow.h header, but not in the way
> it has been changed in the patch. The thing is that unlike the newly added
> intrinsics headers, mm3dnow.h is still publicly includable header, and just
> includi
On Thu, Jul 19, 2012 at 11:51 AM, Jakub Jelinek wrote:
>> >> You have a mnemonic clash here. prefetchw is not good name for a new
>> >> instruction, it clashes with existing 3dnow name. Intel will need to
>> >> fix the spec, you probably won't be able to change prefetchw encoding
>> >> in binutil
On Thu, Jul 19, 2012 at 11:23:09AM +0200, Uros Bizjak wrote:
> >> You have a mnemonic clash here. prefetchw is not good name for a new
> >> instruction, it clashes with existing 3dnow name. Intel will need to
> >> fix the spec, you probably won't be able to change prefetchw encoding
> >> in binutil
> - else
> -return "prefetchw\t%a0";
> -}
> + "prefetch\t%a0"
>
> You have a mnemonic clash here. prefetchw is not good name for a new
> instruction, it clashes with existing 3dnow name. Intel will need to
> fix the spec, you probably won't be able to change prefetchw encoding
> in binutils.
On Thu, Jul 19, 2012 at 11:10 AM, Jakub Jelinek wrote:
> On Thu, Jul 19, 2012 at 11:03:25AM +0200, Uros Bizjak wrote:
>> + /* Set CF to specified value. */
>> + emit_insn (gen_addqi3_cc(
>> + gen_reg_rtx(QImode),
>> + op1,
>> + constm1_rtx));
>>
On Thu, Jul 19, 2012 at 11:03:25AM +0200, Uros Bizjak wrote:
> + /* Set CF to specified value. */
> + emit_insn (gen_addqi3_cc(
> + gen_reg_rtx(QImode),
> + op1,
> + constm1_rtx));
> +
>
> This should be in the *.md expander.
Note also that the
Hello!
> This patch adds new intrinsics for new ADCX, ADOX, RDSEED and
> PREFETCHW instructions, introduced here:
> http://software.intel.com/en-us/avx/
+/* Copyright (C) 2012 Free Software Foundation, Inc.
+ Contributed by Michael Zolotukhin.
Please remove "Contributed by" lines from headers.
Hi,
This patch adds new intrinsics for new ADCX, ADOX, RDSEED and
PREFETCHW instructions, introduced here:
http://software.intel.com/en-us/avx/
Bootstrapped on x86-64, testing is in progress.
Is it ok for trunk?
Changelog entry:
2012-07-17 Michael Zolotukhin
* common/config/i386/i386
10 matches
Mail list logo