r359918 - Ensure there is stack usage in stack size warning test

Matt Arsenault via cfe-commits cfe-commits at lists.llvm.org
Fri May 3 12:04:15 PDT 2019


Author: arsenm
Date: Fri May  3 12:04:14 2019
New Revision: 359918

URL: http://llvm.org/viewvc/llvm-project?rev=359918&view=rev
Log:
Ensure there is stack usage in stack size warning test

r359906 broke this because the only stack usage was from a spill which
can be avoided since the only block is a return.

Modified:
    cfe/trunk/test/Misc/backend-stack-frame-diagnostics-fallback.cpp

Modified: cfe/trunk/test/Misc/backend-stack-frame-diagnostics-fallback.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Misc/backend-stack-frame-diagnostics-fallback.cpp?rev=359918&r1=359917&r2=359918&view=diff
==============================================================================
--- cfe/trunk/test/Misc/backend-stack-frame-diagnostics-fallback.cpp (original)
+++ cfe/trunk/test/Misc/backend-stack-frame-diagnostics-fallback.cpp Fri May  3 12:04:14 2019
@@ -14,5 +14,7 @@ namespace frameSizeThunkWarning {
 
   // CHECK: warning: stack frame size of {{[0-9]+}} bytes in function 'frameSizeThunkWarning::B::f'
   // CHECK: warning: stack size limit exceeded ({{[0-9]+}}) in {{[^ ]+}}
-  void B::f() { }
+  void B::f() {
+    volatile int x = 0; // Ensure there is stack usage.
+  }
 }




More information about the cfe-commits mailing list