diff options
| author | Martin Kroeker <martin@ruby.chemie.uni-freiburg.de> | 2018-12-30 20:07:01 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-12-30 20:07:01 +0100 |
| commit | 8643521127b613d2d1fa328c3d1a5666eca78614 (patch) | |
| tree | 0f3f33d7a83eef3eec2cc8c2f83dd0e54ca7b646 /driver | |
| parent | ccd5945d382ed44b97180a9ed0c7337f577d30d4 (diff) | |
| parent | 5a720cf9cac5266079c06032fb2ab36da4ed84f5 (diff) | |
| download | openblas-8643521127b613d2d1fa328c3d1a5666eca78614.tar.gz openblas-8643521127b613d2d1fa328c3d1a5666eca78614.tar.bz2 openblas-8643521127b613d2d1fa328c3d1a5666eca78614.zip | |
Merge pull request #1943 from martin-frbg/issue1748
Re-enable loop unrolling in trmv and remove the scary warning
Diffstat (limited to 'driver')
| -rw-r--r-- | driver/level2/trmv_U.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/driver/level2/trmv_U.c b/driver/level2/trmv_U.c index 7f8895e7f..90ffb7370 100644 --- a/driver/level2/trmv_U.c +++ b/driver/level2/trmv_U.c @@ -54,16 +54,12 @@ int CNAME(BLASLONG m, FLOAT *a, BLASLONG lda, FLOAT *b, BLASLONG incb, FLOAT *bu COPY_K(m, b, incb, buffer, 1); } -/*FIXME the GEMV unrolling performed here was found to be broken, see issue 1332 */ -/* Multiplying DTB size by 100 is just a quick-and-dirty hack to disable it for now[B */ + for (is = 0; is < m; is += DTB_ENTRIES){ - for (is = 0; is < m; is += DTB_ENTRIES * 100){ - - min_i = MIN(m - is, DTB_ENTRIES * 100); + min_i = MIN(m - is, DTB_ENTRIES); #ifndef TRANSA - if (is > 0){ -fprintf(stderr,"WARNING unrolling of the trmv_U loop may give wrong results\n"); + if (is > 0){ GEMV_N(is, min_i, 0, dp1, a + is * lda, lda, B + is, 1, |