[PATCH] D46789: [StackProtector] Don't crash when passing post-SP IR back in

Tim Shen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 22 14:28:28 PDT 2018


timshen added a comment.

My feeling is that getSDagStackGuard/getSSPStackGuardCheck gets mismatched with shouldEmitSDCheck/SupportsSelectionDAGSP. Would you explain the invariance between them if you already investigated it? It seems shouldEmitSDCheck needs to be fixed too, but the current patch doesn't fix it.

Also, the function getSSPStackGuardCheck is weird, since we already have getSDagStackGuard.


Repository:
  rL LLVM

https://reviews.llvm.org/D46789





More information about the llvm-commits mailing list