[PATCH] D66839: Fix stack address builtin for negative numbers

[Zoe Carver via Phabricator via cfe-commits]

zoecarver updated this revision to Diff 246486.
zoecarver added a comment.

- Error if SemaBuiltinConstantArgRange returns true, not false


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66839/new/

https://reviews.llvm.org/D66839

Files:
  clang/test/Sema/builtin-stackaddress.c


Index: clang/test/Sema/builtin-stackaddress.c
===================================================================
--- clang/test/Sema/builtin-stackaddress.c
+++ clang/test/Sema/builtin-stackaddress.c
@@ -1,6 +1,6 @@
 // RUN: %clang_cc1 -fsyntax-only -verify %s
 
-void a(unsigned x) {
+void* a(unsigned x) {
 return __builtin_return_address(0);
 }
 



Index: clang/test/Sema/builtin-stackaddress.c
===================================================================
--- clang/test/Sema/builtin-stackaddress.c
+++ clang/test/Sema/builtin-stackaddress.c
@@ -1,6 +1,6 @@
 // RUN: %clang_cc1 -fsyntax-only -verify %s
 
-void a(unsigned x) {
+void* a(unsigned x) {
 return __builtin_return_address(0);
 }
 

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits