[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);
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D66839.246486.patch
Type: text/x-patch
Size: 350 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200225/acc76d34/attachment-0001.bin>