[PATCH] D38776: [codeview] Implement FPO data assembler directives

David Majnemer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 10 21:35:45 PDT 2017


majnemer added a comment.

I'd also be interested in seeing a stack frame with spills and stack realignment.



================
Comment at: llvm/lib/Target/X86/X86FrameLowering.cpp:1127
+
+      if (NeedsWinCFI) {
+        // .cv_fpo_setframe $FramePtr
----------------
I think this would be more clear as `NeedsWinFPO`.


https://reviews.llvm.org/D38776





More information about the llvm-commits mailing list