[PATCH] add a cl::opt override for TargetLowering's JumpIsExpensive

Kevin B. Smith kevin.b.smith at intel.com
Tue Jun 30 13:16:57 PDT 2015


Seems like for the unit test that it would be better to not be quite so specific.  For the JUMPY case, maybe
CHECK-DAG: jl
CHECK-DAG: je
would be better, and a little less fragile because it seems like the ordering of the jumps could be arbitrary.
For the NOJMP case, maybe just check for one set, the or, and then a je or jne?


http://reviews.llvm.org/D10846

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






More information about the llvm-commits mailing list