[PATCH] D87161: [DemandedBits][BDCE] Add support for min/max intrinsics
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 10 09:03:15 PDT 2020
spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: llvm/lib/Analysis/DemandedBits.cpp:181-182
+ break;
}
break;
case Instruction::Add:
----------------
Nit: this is where LLVM formatting rules break down. I couldn't tell that we were in a switch-within-an-if-within-a-swich from that single brace/indent. At the least, I'd add braces on the 'if'.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87161/new/
https://reviews.llvm.org/D87161
More information about the llvm-commits
mailing list