[PATCH] D61555: Remove BinaryOpsEnd case from ConstantFoldBinaryInstruction(...)
Cameron McInally via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun May 5 12:31:10 PDT 2019
cameron.mcinally added a comment.
I'm noticing that `lib/Analysis/InstructionSimplify.cpp` has similar switches that use the default case instead of `UnaryOpsEnd` and `BinaryOpsEnd`. Is it worth taking the time to update those for compile-time warnings?
I haven't confirmed which of the switches are candidates yet, so that will need to be done if we move forward with this plan...
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61555/new/
https://reviews.llvm.org/D61555
More information about the llvm-commits
mailing list