[PATCH] D67435: Make SwitchInstProfUpdateWrapper strict permanently

Yevgeny Rouban via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 11 03:44:29 PDT 2019


yrouban created this revision.
yrouban added reviewers: davidxl, nikic, reames, chandlerc, eraman.
Herald added a subscriber: hiraditya.
Herald added a project: LLVM.

We have been using //-switch-inst-prof-update-wrapper-strict// set to //true// by default for some time.
It is time to remove the safety stuff and make //SwitchInstProfUpdateWrapper// intolerant to inconsistencies in !prof branch_weights metadata of //SwitchInst//.
This patch gets rid of the //Invalid// state of //SwitchInstProfUpdateWrapper// and the option //-switch-inst-prof-update-wrapper-strict//.
So there is only two states: //changed// and //unchanged//.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D67435

Files:
  llvm/include/llvm/IR/Instructions.h
  llvm/lib/IR/Instructions.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67435.219682.patch
Type: text/x-patch
Size: 4590 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190911/a2dce373/attachment.bin>


More information about the llvm-commits mailing list