[PATCH] Enable invoking the patchpoint intrinsic

Kevin Modzelewski kmod at dropbox.com
Fri Oct 17 02:16:49 PDT 2014


Ok I rebased onto r220027, but the test is passing for me.  Not that that's really a good sign for the test's robustness, but I'm curious now why it's failing.  I noticed that I had used a different triple than all of the other tests used so I fixed that -- I wonder if that makes it pass for you?  I'm assuming you're running this on a mac so that an incorrectly-specified triple could cause issues since this assumes ELF.

Separately, I also tried to make the test less brittle by getting rid of as many hard-coded label names as I could, but I couldn't get rid of the Ltmp0 one.  I wanted to check to make sure the exception data covers the entire patchpoint, since that was one of my worries going into this, though now that I understand the code better I'd be more ok with taking that out.

http://reviews.llvm.org/D5634






More information about the llvm-commits mailing list