[PATCH] D62656: Make SwitchInstProfUpdateWrapper safer

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 30 10:59:48 PDT 2019


davidxl added inline comments.


================
Comment at: llvm/lib/IR/Instructions.cpp:3907
+
+  if (ProfileData->getNumOperands() != SI.getNumSuccessors() + 1) {
+    State = Invalid;
----------------
should we fix the underlying problem that leads to this inconsistent state?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62656/new/

https://reviews.llvm.org/D62656





More information about the llvm-commits mailing list