[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