Displaying 1 result from an estimated 1 matches for "rl367485".
Did you mean:
367485
2020 Feb 05
3
IndVarSimplify: getBackedgeTakenCount and Release vs Assert
...ion'
After digging around, it seems that following change is the culprit:
-----
Author: Philip Reames <listmail at philipreames.com> 2019-08-01 03:16:08
Committer: Philip Reames <listmail at philipreames.com> 2019-08-01 03:16:08
Fix a release-only build warning triggered by rL367485
llvm-svn: 367499
[..]
+#ifndef NDEBUG
+ // Used below for a consistency check only
const SCEV *BackedgeTakenCount = SE->getBackedgeTakenCount(L);
+#endif
-----
It seems that the 'SE->getBackedgeTakenCount(L)' call has sideeffects.. Is that to be expected ?
Is the cor...