[PATCH] D31548: [IR] Redesign the case iterator in SwitchInst to actually be an iterator and to expose a handle to represent the actual case rather than having the iterator return a reference to itself.

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc updated this revision to Diff 94463.
chandlerc added a comment.

Rebase, enhance with better testing of const iterators, fix a couple of minor
issues, and re-clang-format everything.

Also, ping. I'd love to get this landed.


https://reviews.llvm.org/D31548

Files:
  include/llvm/ADT/iterator.h
  include/llvm/Analysis/CFGPrinter.h
  include/llvm/IR/Instructions.h
  lib/Analysis/InlineCost.cpp
  lib/Analysis/LazyValueInfo.cpp
  lib/Analysis/SparsePropagation.cpp
  lib/Bitcode/Writer/BitcodeWriter.cpp
  lib/CodeGen/CodeGenPrepare.cpp
  lib/ExecutionEngine/Interpreter/Execution.cpp
  lib/IR/AsmWriter.cpp
  lib/IR/Instructions.cpp
  lib/Transforms/Coroutines/CoroSplit.cpp
  lib/Transforms/InstCombine/InstructionCombining.cpp
  lib/Transforms/Scalar/CorrelatedValuePropagation.cpp
  lib/Transforms/Scalar/GVN.cpp
  lib/Transforms/Scalar/JumpThreading.cpp
  lib/Transforms/Scalar/LoopUnrollPass.cpp
  lib/Transforms/Scalar/LoopUnswitch.cpp
  lib/Transforms/Scalar/LowerExpectIntrinsic.cpp
  lib/Transforms/Scalar/NewGVN.cpp
  lib/Transforms/Scalar/SCCP.cpp
  lib/Transforms/Utils/CloneFunction.cpp
  lib/Transforms/Utils/Evaluator.cpp
  lib/Transforms/Utils/Local.cpp
  lib/Transforms/Utils/LowerSwitch.cpp
  lib/Transforms/Utils/SimplifyCFG.cpp
  tools/llvm-diff/DifferenceEngine.cpp
  unittests/IR/InstructionsTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31548.94463.patch
Type: text/x-patch
Size: 44352 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170407/212fa49f/attachment.bin>