]> granicus.if.org Git - llvm/commit
[ARM] Use a Changed flag to avoid making a pass's return value dependent on a compare...
authorCraig Topper <craig.topper@gmail.com>
Mon, 8 May 2017 18:02:51 +0000 (18:02 +0000)
committerCraig Topper <craig.topper@gmail.com>
Mon, 8 May 2017 18:02:51 +0000 (18:02 +0000)
commit3f67904aa9fa0364fb1dbd96f18aef7801f211a4
tree79abd6791c84c34da9419afdd7f79bf39c5bfe5b
parent0f7e2baed536064355c70c4bb049376f83175e09
[ARM] Use a Changed flag to avoid making a pass's return value dependent on a compare with a Statistic object.

Statistic compile to always be 0 in release build so this compare would always return false. And in the debug builds Statistic are global variables and remember their values across pass runs. So this compare returns true anytime the pass runs after the first time it modifies something.

This was found after reviewing all usages of comparison operators on a Statistic object. We had some internal code that did a compare with a statistic that caused a mismatch in output between debug and release builds. So we did an audit out of paranoia.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@302450 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/ARM/ARMOptimizeBarriersPass.cpp