[PATCH] Move the personality function from LandingPadInst to Function

Reid Kleckner rnk at google.com
Mon Jun 15 15:48:14 PDT 2015


I'm happy with this, but we should find another reviewer or two. I'd try Anton for EH stuff and John and Rafael for IR changes.


================
Comment at: lib/Bitcode/Reader/BitcodeReader.cpp:4063
@@ +4062,3 @@
+
+        F->setPersonalityFn(cast<Constant>(PersFn));
+      }
----------------
What should we do with bitcode that has multiple landing pads with mismatched personalities? Obviously, it's invalid IR, but now we might not detect that anymore, right? Seems like if the parent function has a personality and it doesn't match the one we're adding, we should error.

http://reviews.llvm.org/D10429

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list