diff options
| author | Alan Lawrence <alan.lawrence@arm.com> | 2015-10-30 13:27:10 +0000 |
|---|---|---|
| committer | Alan Lawrence <alalaw01@gcc.gnu.org> | 2015-10-30 13:27:10 +0000 |
| commit | 56517dda83af315590448610b4b6cd9d4f4d7527 (patch) | |
| tree | 1733c96c8fbc3c1388b70def1f87e934230e3d50 /gcc | |
| parent | 2ef543e47016ce97fabb14dc3cba52f86e8f4a16 (diff) | |
| download | gcc-56517dda83af315590448610b4b6cd9d4f4d7527.zip gcc-56517dda83af315590448610b4b6cd9d4f4d7527.tar.gz gcc-56517dda83af315590448610b4b6cd9d4f4d7527.tar.bz2 | |
[Trivial] tree-sra.c: A few comment fixes/additions.
* tree-sra.c (scalarizable_type_p): Comment variable-length arrays.
(completely_scalarize): Comment zero-length arrays.
(get_access_replacement): Correct comment re. precondition.
From-SVN: r229577
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/tree-sra.c | 4 |
2 files changed, 9 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index e6b8b27..7178c5a 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2015-10-30 Alan Lawrence <alan.lawrence@arm.com> + + * tree-sra.c (scalarizable_type_p): Comment variable-length arrays. + (completely_scalarize): Comment zero-length arrays. + (get_access_replacement): Correct comment re. precondition. + 2015-10-30 Venkataramanan Kumar <venkataramanan.kumar@amd.com> * config/i386/i386.c (get_builtin_code_for_version): Set priority diff --git a/gcc/tree-sra.c b/gcc/tree-sra.c index b6172cd..82c58f0 100644 --- a/gcc/tree-sra.c +++ b/gcc/tree-sra.c @@ -956,6 +956,7 @@ scalarizable_type_p (tree type) ; else if ((tree_to_shwi (TYPE_SIZE (type)) <= 0) || !tree_fits_shwi_p (TYPE_MAX_VALUE (TYPE_DOMAIN (type)))) + /* Variable-length array, do not allow scalarization. */ return false; tree elem = TREE_TYPE (type); @@ -1005,6 +1006,7 @@ completely_scalarize (tree base, tree decl_type, HOST_WIDE_INT offset, tree ref) tree minidx = TYPE_MIN_VALUE (TYPE_DOMAIN (decl_type)); gcc_assert (TREE_CODE (minidx) == INTEGER_CST); tree maxidx = TYPE_MAX_VALUE (TYPE_DOMAIN (decl_type)); + /* Skip (some) zero-length arrays; others have MAXIDX == MINIDX - 1. */ if (maxidx) { gcc_assert (TREE_CODE (maxidx) == INTEGER_CST); @@ -2146,7 +2148,7 @@ create_access_replacement (struct access *access) return repl; } -/* Return ACCESS scalar replacement, create it if it does not exist yet. */ +/* Return ACCESS scalar replacement, which must exist. */ static inline tree get_access_replacement (struct access *access) |