[PATCH] D20939: Update for DiagnosticInfoStackSize changes

Matt Arsenault via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 2 16:31:13 PDT 2016


arsenm created this revision.
arsenm added a subscriber: cfe-commits.

http://reviews.llvm.org/D20939

Files:
  lib/CodeGen/CodeGenAction.cpp
  test/Misc/backend-resource-limit-diagnostics.cl

Index: test/Misc/backend-resource-limit-diagnostics.cl
===================================================================
--- /dev/null
+++ test/Misc/backend-resource-limit-diagnostics.cl
@@ -0,0 +1,10 @@
+// REQUIRES: amdgpu-registered-target
+// RUN: not %clang_cc1 -emit-codegen-only -triple=amdgcn-- %s 2>&1 | FileCheck %s
+
+
+// CHECK: error: local memory limit exceeded (480000) in use_huge_lds
+kernel void use_huge_lds()
+{
+    volatile local int huge[120000];
+    huge[0] = 2;
+}
Index: lib/CodeGen/CodeGenAction.cpp
===================================================================
--- lib/CodeGen/CodeGenAction.cpp
+++ lib/CodeGen/CodeGenAction.cpp
@@ -415,9 +415,10 @@
     return false;
 
   if (const Decl *ND = Gen->GetDeclForMangledName(D.getFunction().getName())) {
+    // FIXME: Shouldn't need to truncate to uint32_t
     Diags.Report(ND->getASTContext().getFullLoc(ND->getLocation()),
                  diag::warn_fe_frame_larger_than)
-        << D.getStackSize() << Decl::castToDeclContext(ND);
+      << static_cast<uint32_t>(D.getStackSize()) << Decl::castToDeclContext(ND);
     return true;
   }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D20939.59477.patch
Type: text/x-patch
Size: 1131 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160602/4de95b59/attachment.bin>


More information about the cfe-commits mailing list