[PATCH] D17736: [SSP] Remove llvm.stackprotectorcheck.

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 7 16:16:15 PDT 2016


echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.

Oh, and long as you add the inline comments, fix the style stuff, etc, I think we're good to go.

-eric


================
Comment at: test/Assembler/auto_upgrade_intrinsics.ll:57
@@ -56,3 +56,2 @@
 ; CHECK: @llvm.objectsize.i32.p0i8
-; CHECK-DAG: declare i32 @llvm.objectsize.i32.p0i8
   %s = call i32 @llvm.objectsize.i32(i8* getelementptr inbounds ([60 x i8], [60 x i8]* @a, i32 0, i32 0), i1 false)
----------------
echristo wrote:
> Hmm?
Nevermind, I see it now. That's frustrating, can you add a comment that the declare is for the objectsize upgrade and why?


http://reviews.llvm.org/D17736





More information about the llvm-commits mailing list