[PATCH] [Verifier] Rephrase visitIntrinsicFunctionCall in terms of CallSites

Philip Reames listmail at philipreames.com
Thu May 28 22:15:01 PDT 2015


The motivation is to remove a special case for invokable statepoints and share the code for intrinsic verification for CallInst and InvokeInst.

The code change is intentionally very mechanical.  I left off variable renaming - which will happen in a follow on change - to reduce the textual diff to simplify review.  The only part that really needs review is a) whether adding all the methods to CallSite is reasonable to make it closer to a drop in replacement for CallInst, and b) whether extending Write to support CallSites directly is reasonable.  (I used the CallSite * to reduce the textual diff for converting a function from CallInst to CallSite.  I'm open to changing this, but think the current version is worthwhile, even if ugly.)

http://reviews.llvm.org/D10118

Files:
  include/llvm/IR/CallSite.h
  lib/IR/Verifier.cpp

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D10118.26768.patch
Type: text/x-patch
Size: 5844 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150529/3ab6be4d/attachment.bin>


More information about the llvm-commits mailing list