summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBruce Forstall <brucefo@microsoft.com>2016-04-08 16:20:31 -0700
committerBruce Forstall <brucefo@microsoft.com>2016-04-08 16:20:31 -0700
commit669fdc0c3df5ba3a7c155952f110926f2f17b837 (patch)
tree0a34a0e433e0fcee27d56baae084e8d5d881491b
parent4ee08c7f344a49e27dfece22cf0ff5159158e22b (diff)
downloadcoreclr-669fdc0c3df5ba3a7c155952f110926f2f17b837.tar.gz
coreclr-669fdc0c3df5ba3a7c155952f110926f2f17b837.tar.bz2
coreclr-669fdc0c3df5ba3a7c155952f110926f2f17b837.zip
Minor JIT changes
Add a useful assert about SP to FP offsets. Add "const" to a few member functions.
-rw-r--r--src/jit/codegencommon.cpp1
-rw-r--r--src/jit/codegeninterface.h8
2 files changed, 5 insertions, 4 deletions
diff --git a/src/jit/codegencommon.cpp b/src/jit/codegencommon.cpp
index b96f5c0956..a6a3862100 100644
--- a/src/jit/codegencommon.cpp
+++ b/src/jit/codegencommon.cpp
@@ -236,6 +236,7 @@ int CodeGenInterface::genSPtoFPdelta()
delta = -genCallerSPtoInitialSPdelta() + genCallerSPtoFPdelta();
+ assert(delta >= 0);
return delta;
}
diff --git a/src/jit/codegeninterface.h b/src/jit/codegeninterface.h
index cb8cf61180..da146f2a8f 100644
--- a/src/jit/codegeninterface.h
+++ b/src/jit/codegeninterface.h
@@ -215,14 +215,14 @@ public:
// Arguments are accessed relative to the Frame Pointer (EBP), and
// locals are accessed relative to the Stack Pointer (ESP).
public:
- bool doDoubleAlign() { return m_cgDoubleAlign; }
- void setDoubleAlign(bool value) { m_cgDoubleAlign = value; }
- bool doubleAlignOrFramePointerUsed() { return isFramePointerUsed() || doDoubleAlign(); }
+ bool doDoubleAlign() const { return m_cgDoubleAlign; }
+ void setDoubleAlign(bool value) { m_cgDoubleAlign = value; }
+ bool doubleAlignOrFramePointerUsed() const { return isFramePointerUsed() || doDoubleAlign(); }
private:
bool m_cgDoubleAlign;
#else // !DOUBLE_ALIGN
public:
- bool doubleAlignOrFramePointerUsed() { return isFramePointerUsed(); }
+ bool doubleAlignOrFramePointerUsed() const { return isFramePointerUsed(); }
#endif // !DOUBLE_ALIGN