summaryrefslogtreecommitdiff
path: root/gcc/builtins.c
diff options
context:
space:
mode:
authorMartin Sebor <msebor@redhat.com>2020-08-30 15:10:44 -0600
committerMartin Sebor <msebor@redhat.com>2020-08-30 15:11:48 -0600
commit6ccadc4c0486ff011a32c74de1a31148acb3cbe2 (patch)
tree83a984530c8523cf8565da5aa1fe8da7a9d20f92 /gcc/builtins.c
parenta240e83ce9d92786ac9a15ab815b58197b85ada2 (diff)
Use get_size_range instead of get_range to obtain range of valid sizes.
gcc/ChangeLog: * builtins.c (access_ref::access_ref): Call get_size_range instead of get_range. gcc/testsuite/ChangeLog: * gcc.dg/Wstringop-overread-3.c: New test.
Diffstat (limited to 'gcc/builtins.c')
-rw-r--r--gcc/builtins.c9
1 files changed, 7 insertions, 2 deletions
diff --git a/gcc/builtins.c b/gcc/builtins.c
index df121f98b95..bc35b071f02 100644
--- a/gcc/builtins.c
+++ b/gcc/builtins.c
@@ -214,8 +214,13 @@ access_ref::access_ref (tree bound /* = NULL_TREE */,
/* When BOUND is nonnull and a range can be extracted from it,
set the bounds of the access to reflect both it and MINACCESS.
BNDRNG[0] is the size of the minimum access. */
- if (bound && get_range (bound, UNSIGNED, bndrng))
- bndrng[0] = bndrng[0] > 0 && minaccess ? 1 : 0;
+ tree rng[2];
+ if (bound && get_size_range (bound, rng, true))
+ {
+ bndrng[0] = wi::to_offset (rng[0]);
+ bndrng[1] = wi::to_offset (rng[1]);
+ bndrng[0] = bndrng[0] > 0 && minaccess ? 1 : 0;
+ }
}
/* Return true if NAME starts with __builtin_ or __sync_. */