[PATCH] D63527: InstCombine: Preserve nuw when reassociating nuw ops [2/3]
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 21 06:24:00 PDT 2019
arsenm marked an inline comment as done.
arsenm added inline comments.
================
Comment at: lib/Transforms/InstCombine/InstructionCombining.cpp:380
if (I.isAssociative() && I.isCommutative()) {
if (simplifyAssocCastAssoc(&I)) {
----------------
lebedev.ri wrote:
> So for the purpose of this patch `add`/`mul` only, correct?
Yes. Bit-ops are also handled here, but they don't have the flags to preserve
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63527/new/
https://reviews.llvm.org/D63527
More information about the llvm-commits
mailing list