[PATCH] D36654: [X86] Correct two places that seemed to be mishandling X86ISD::UMUL

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 13 12:44:35 PDT 2017

Previous message: [PATCH] D36654: [X86] Correct two places that seemed to be mishandling X86ISD::UMUL
Next message: [PATCH] D36654: [X86] Correct two places that seemed to be mishandling X86ISD::UMUL
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

craig.topper added a comment.

I'm actually not sure why any of the "flag" setting instructions are in computeKnownBitsForTargetNode. You can remove them all and no tests fail.


https://reviews.llvm.org/D36654

Previous message: [PATCH] D36654: [X86] Correct two places that seemed to be mishandling X86ISD::UMUL
Next message: [PATCH] D36654: [X86] Correct two places that seemed to be mishandling X86ISD::UMUL
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list