[llvm-commits] x86 branch sequence optimization in LLVM code gen: please review

Umansky, Victor victor.umansky at intel.com
Thu Dec 15 10:08:48 PST 2011


Hi Bruno, 

Please see attached the patch.

It incorporates the feedback, and I yet extended it with coverage for more LLVM IR patterns related to usage of ptestz/ptestc LLVM built-ins (tests are also extended).

Best Regards,
   Victor


-----Original Message-----
From: Bruno Cardoso Lopes [mailto:bruno.cardoso at gmail.com] 
Sent: Friday, December 09, 2011 04:25
To: Umansky, Victor
Cc: Chad Rosier; Anton Korobeynikov; bruno.cardoso at gmail.com; llvm-commits at cs.uiuc.edu
Subject: Re: [llvm-commits] x86 branch sequence optimization in LLVM code gen: please review

On Wed, Dec 7, 2011 at 1:03 PM, Umansky, Victor
<victor.umansky at intel.com> wrote:
>
> Hi Chad, Anton, Bruno,
>
>
>
> Thank you for the suggestion.
>
>
>
> Unfortunately, it won't work in the case of brcond.ll file.
>
>
>
> Indeed I can introduce different "check-prefix" values in order to separate checks for "core2" case from those for "penryn" case.
>
> However, the compilation of all functions in a file will be done unconditionally for both "RUN" cases. And this will inevitably lead to the test failure (in instruction selection) when a function using "ptest" LLVM intrinsic will be processed with "-mcpu=core2" option.
>
> That's why I was not able to include the test cases for "ptest" intrinsic sequence to a file which will be compiled for a pre-Penryn target.
>
>
>
> A solution which does work is to have legacy brcond.ll LIT tests running under "-mcpu=penryn".
>
> I'm attaching the file.
>
> Are you OK with such solution?

LGTM, please resend the orignal patch with the testcase (both in the
same patch file)! Also remove the trailing CRs and generate the diff
with "svn diff" under the project root. If you have any question, the
docs may help: http://llvm.org/docs/DeveloperPolicy.html#patches

--
Bruno Cardoso Lopes
http://www.brunocardoso.cc
---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: brcond_combine.patch
Type: application/octet-stream
Size: 14571 bytes
Desc: brcond_combine.patch
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20111215/55bbd7cb/attachment.obj>


More information about the llvm-commits mailing list