[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