[PATCH] D108091: [AggressiveInstCombine] Add shift left instruction to `TruncInstCombine` DAG
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 16 00:21:03 PDT 2021
lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.
I'm not sure how much will this catch, but LG.
@spatel ?
================
Comment at: llvm/test/Transforms/AggressiveInstCombine/trunc_shifts.ll:4
define i16 @shl_1_commute(i8 %x) {
; CHECK-LABEL: @shl_1_commute(
----------------
Pedantic nitpick: commute means `mul %x, %y <=> mul %y, %x`
Here you want to use `negative_test` in place of `not_commute`, and drop `commute`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108091/new/
https://reviews.llvm.org/D108091
More information about the llvm-commits
mailing list