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

Zoe Carver via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 25 09:12:00 PST 2020


zoecarver added a comment.

I was using `SemaBuiltinConstantArgRange` incorrectly. I was returning an error for `!SemaBuiltinConstantArgRange` instead of `SemaBuiltinConstantArgRange`. Also, I only modified the fail tests so I wasn't able to catch the error. Updated and am running both the Sema and CodeGen tests now. Should fix the error.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66839





More information about the cfe-commits mailing list