Fix for https://bugs.llvm.org/show_bug.cgi?id=41971. Make the combineVectorSizedSetCCEquality() transform more conservative by checking that the bitcast to the vector type will be cheap/free for both operands. I'm considering it cheap if it's a constant, a load or already a vector. I've dropped the explicit check for f128 because it should fall out naturally (in the cases where it'd be detrimental).
Details
Details
- Reviewers
- spatel - RKSimon - craig.topper 
- Commits
- rZORG0d0162efa758: [X86] Don't compare i128 through vector if construction not cheap (PR41971)
 rG0d0162efa758: [X86] Don't compare i128 through vector if construction not cheap (PR41971)
 rG15df05152d3d: [X86] Don't compare i128 through vector if construction not cheap (PR41971)
 rL361352: [X86] Don't compare i128 through vector if construction not cheap (PR41971)
Diff Detail
Diff Detail