From 1583124e4fdfa4c60f7cfa4063e3ef47087927ca Mon Sep 17 00:00:00 2001 From: Martin Sebor Date: Tue, 28 Aug 2018 00:10:46 +0000 Subject: [PATCH] PR tree-optimization/87112 - ICE in fold_binary_loc on strnlen of mixed integer types gcc/ChangeLog: PR tree-optimization/87112 * builtins.c (expand_builtin_strnlen): Convert c_strlen result to the type of the bound argument. gcc/testsuite/ChangeLog: PR tree-optimization/87112 * gcc.dg/pr87112.c: New test. From-SVN: r263900 --- gcc/ChangeLog | 6 ++++++ gcc/builtins.c | 5 ++++- gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/gcc.dg/pr87112.c | 31 +++++++++++++++++++++++++++++++ 4 files changed, 46 insertions(+), 1 deletion(-) create mode 100644 gcc/testsuite/gcc.dg/pr87112.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index a8ab83580b5..2449672c62f 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2018-08-27 Martin Sebor + + PR tree-optimization/87112 + * builtins.c (expand_builtin_strnlen): Convert c_strlen result to + the type of the bound argument. + 2018-08-27 Jeff Law * tree-ssa-dse.c (compute_trims): Handle case where the reference's diff --git a/gcc/builtins.c b/gcc/builtins.c index c4b52b9c627..eb69a409272 100644 --- a/gcc/builtins.c +++ b/gcc/builtins.c @@ -2970,6 +2970,10 @@ expand_builtin_strnlen (tree exp, rtx target, machine_mode target_mode) tree func = get_callee_fndecl (exp); tree len = c_strlen (src, 0); + /* FIXME: Change c_strlen() to return sizetype instead of ssizetype + so these conversions aren't necessary. */ + if (len) + len = fold_convert_loc (loc, TREE_TYPE (bound), len); if (TREE_CODE (bound) == INTEGER_CST) { @@ -2984,7 +2988,6 @@ expand_builtin_strnlen (tree exp, rtx target, machine_mode target_mode) if (!len || TREE_CODE (len) != INTEGER_CST) return NULL_RTX; - len = fold_convert_loc (loc, size_type_node, len); len = fold_build2_loc (loc, MIN_EXPR, size_type_node, len, bound); return expand_expr (len, target, target_mode, EXPAND_NORMAL); } diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 8ecb60b2e17..92f15401a22 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2018-08-27 Martin Sebor + + PR tree-optimization/87112 + * gcc.dg/pr87112.c: New test. + 2018-08-27 David Malcolm PR c++/63392 diff --git a/gcc/testsuite/gcc.dg/pr87112.c b/gcc/testsuite/gcc.dg/pr87112.c new file mode 100644 index 00000000000..7510956be4e --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr87112.c @@ -0,0 +1,31 @@ +/* PR tree-optimization/87112 - ICE due to strnlen mixing integer types + { dg-do compile } + { dg-options "-Os -Wall" } */ + +typedef __SIZE_TYPE__ size_t; + +extern size_t strnlen (const char*, size_t); + +size_t fi (int i) +{ + int n = i & 3; + return strnlen ("int", n); +} + +size_t fui (unsigned i) +{ + unsigned n = i & 3; + return strnlen ("unsigned", n); +} + +size_t fl (long i) +{ + long n = i & 3; + return strnlen ("long", n); +} + +size_t fsz (size_t i) +{ + size_t n = i & 3; + return strnlen ("size_t", n); +}