[PATCH] change DAG's JumpIsExpensive check for x86 (PR23827)

Chandler Carruth chandlerc at gmail.com
Fri Jun 26 01:31:51 PDT 2015


This seems really unlikely to be a good change.

Jump instructions are not expensive on x86 in general. They certainly are not more expensive than things like sete! I think we usually want to use more jumps rather than moving things out of condition registers.

I think the bug report you're trying to fix is flat out incorrect and backwards. I've updated it.


http://reviews.llvm.org/D10741

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list