[PATCH] D18846: [safestack] Add canary to unsafe stack frames

[Peter Collingbourne via llvm-commits]

pcc added inline comments.

================
Comment at: test/CodeGen/X86/safestack_ssp.ll:2
@@ +1,3 @@
+; RUN: llc -mtriple=i386-linux < %s -o - | FileCheck --check-prefix=LINUX-I386 %s
+; RUN: llc -mtriple=x86_64-linux < %s -o - | FileCheck --check-prefix=LINUX-X64 %s
+
----------------
Well, I mainly wanted you to tell me why you made this a backend test :) We don't normally test passes this way.

Anyway, if you're just trying to provide test coverage for the codegen pipeline, it looks like the existing test `test/CodeGen/X86/safestack.ll` is already covering that, so unless you have another reason to keep this test, it seems redundant with the opt tests you added.


Repository:
  rL LLVM

http://reviews.llvm.org/D18846