[PATCH] D51074: [WebAssembly] Change comments on SP writing back (NFC)

[Heejin Ahn via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL340372: [WebAssembly] Change comments on SP writing back (NFC) (authored by aheejin, committed by ).

Repository:
  rL LLVM

https://reviews.llvm.org/D51074

Files:
  llvm/trunk/lib/Target/WebAssembly/WebAssemblyFrameLowering.cpp
  llvm/trunk/lib/Target/WebAssembly/WebAssemblyFrameLowering.h


Index: llvm/trunk/lib/Target/WebAssembly/WebAssemblyFrameLowering.cpp
===================================================================
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyFrameLowering.cpp
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyFrameLowering.cpp
@@ -99,8 +99,9 @@
 }
 
 /// Returns true if the local user-space stack pointer needs to be written back
-/// to memory by this function (this is not meaningful if needsSP is false). If
-/// false, the stack red zone can be used and only a local SP is needed.
+/// to __stack_pointer global by this function (this is not meaningful if
+/// needsSP is false). If false, the stack red zone can be used and only a local
+/// SP is needed.
 bool WebAssemblyFrameLowering::needsSPWriteback(
     const MachineFunction &MF, const MachineFrameInfo &MFI) const {
   assert(needsSP(MF, MFI));
Index: llvm/trunk/lib/Target/WebAssembly/WebAssemblyFrameLowering.h
===================================================================
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyFrameLowering.h
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyFrameLowering.h
@@ -24,7 +24,8 @@
 class WebAssemblyFrameLowering final : public TargetFrameLowering {
  public:
   /// Size of the red zone for the user stack (leaf functions can use this much
-  /// space below the stack pointer without writing it back to memory).
+  /// space below the stack pointer without writing it back to __stack_pointer
+  /// global).
   // TODO: (ABI) Revisit and decide how large it should be.
   static const size_t RedZoneSize = 128;
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51074.161851.patch
Type: text/x-patch
Size: 1573 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180822/3522d04e/attachment.bin>