[PATCH] D53098: [InstCombine] Avoid instruction sinking if not profitable for other passes

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 11 12:02:23 PDT 2018


lebedev.ri added a comment.

I see that there were some perf numbers in https://bugs.llvm.org/show_bug.cgi?id=39119#c11,
but those are global, not for this patch. Anything for this change specifically?



================
Comment at: test/Transforms/InstCombine/no-sink-nonnull-args.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt < %s -instcombine -S | FileCheck %s
----------------
Please precommit the test. It isn't obvious what the diff changes here.
No existing tests change?


https://reviews.llvm.org/D53098





More information about the llvm-commits mailing list