[PATCH] D71209: InstCombine: Don't rewrite phi-of-bitcast when the phi has other users
Connor Abbott via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 10 05:43:57 PST 2019
cwabbott marked an inline comment as done.
cwabbott added inline comments.
================
Comment at: llvm/test/Transforms/InstCombine/pr44242.ll:1
+; RUN: opt -S -instcombine < %s | FileCheck %s
+
----------------
nikic wrote:
> cwabbott wrote:
> > lebedev.ri wrote:
> > > Please autogenerate and precommit tests.
> > Sorry, but what exactly do you mean?
> You can use `utils/update_test_checks.py` to generate the expected output. Then you should first commit just the test (with generated output without your patch). The patch will then only show the IR diff in the test, making is clearer what actually changed.
Hopefully I did that right.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71209/new/
https://reviews.llvm.org/D71209
More information about the llvm-commits
mailing list