[PATCH] D56214: AggressiveInstCombine: Fold full mul i64 x i64 -> i128
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 3 10:38:00 PST 2019
lebedev.ri added inline comments.
================
Comment at: test/Transforms/AggressiveInstCombine/mul128.ll:1
+; RUN: opt < %s -aggressive-instcombine -S | FileCheck %s
+
----------------
chfast wrote:
> lebedev.ri wrote:
> > chfast wrote:
> > > lebedev.ri wrote:
> > > > Please use `llvm/utils/update_test_checks.py`.
> > > > And move the initial test case into another review, so this diff shows the *change* in the test output.
> > > Tests added as https://reviews.llvm.org/D56277.
> > (Yep, now this diff just needs to be based ontop of that diff, so the tests show difference)
> Can this be done in Phabricator?
Phabricator only displays the diff you upload.
If you used git for this, simply keep these two diffs as two consecutive commits,
and upload each one of them separately to their respective reviews.
If svn, no idea.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56214/new/
https://reviews.llvm.org/D56214
More information about the llvm-commits
mailing list