[PATCH] D49256: CodeGen: Remove pipeline dependencies on StackProtector; NFC

Stephen Crane via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 12 11:34:15 PDT 2018


rinon accepted this revision.
rinon added a comment.

Thanks for updating this! LGTM.



================
Comment at: lib/CodeGen/StackProtector.cpp:516
+
+void StackProtector::copyToMachineFrameInfo(MachineFrameInfo &MFI) const {
+  if (Layout.empty())
----------------
Good call on moving this into StackProtector, it's definitely better here.


Repository:
  rL LLVM

https://reviews.llvm.org/D49256





More information about the llvm-commits mailing list