[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