Commit 3e5009f6 by Jan Voung

Make compile without ICE_THREAD_LOCAL_HACK (avoid "Type *TLS = TLS;")

Otherwise you get: In file included from src/IceGlobalContext.cpp:21: In file included from src/IceCfg.h:21: src/IceGlobalContext.h:257:44: error: variable 'TLS' is uninitialized when used within its own initialization [-Werror,-Wuninitialized] ThreadContext *TLS = ICE_TLS_GET_FIELD(TLS); ~~~ ^~~ src/IceTLS.h:95:39: note: expanded from macro 'ICE_TLS_GET_FIELD' ^ So rename the local var to Tls. BUG=none R=stichnot@chromium.org Review URL: https://codereview.chromium.org/1030793002
parent f79d2cb6
...@@ -226,37 +226,37 @@ public: ...@@ -226,37 +226,37 @@ public:
void statsUpdateEmitted(uint32_t InstCount) { void statsUpdateEmitted(uint32_t InstCount) {
if (!getFlags().getDumpStats()) if (!getFlags().getDumpStats())
return; return;
ThreadContext *TLS = ICE_TLS_GET_FIELD(TLS); ThreadContext *Tls = ICE_TLS_GET_FIELD(TLS);
TLS->StatsFunction.update(CodeStats::CS_InstCount, InstCount); Tls->StatsFunction.update(CodeStats::CS_InstCount, InstCount);
TLS->StatsCumulative.update(CodeStats::CS_InstCount, InstCount); Tls->StatsCumulative.update(CodeStats::CS_InstCount, InstCount);
} }
void statsUpdateRegistersSaved(uint32_t Num) { void statsUpdateRegistersSaved(uint32_t Num) {
if (!getFlags().getDumpStats()) if (!getFlags().getDumpStats())
return; return;
ThreadContext *TLS = ICE_TLS_GET_FIELD(TLS); ThreadContext *Tls = ICE_TLS_GET_FIELD(TLS);
TLS->StatsFunction.update(CodeStats::CS_RegsSaved, Num); Tls->StatsFunction.update(CodeStats::CS_RegsSaved, Num);
TLS->StatsCumulative.update(CodeStats::CS_RegsSaved, Num); Tls->StatsCumulative.update(CodeStats::CS_RegsSaved, Num);
} }
void statsUpdateFrameBytes(uint32_t Bytes) { void statsUpdateFrameBytes(uint32_t Bytes) {
if (!getFlags().getDumpStats()) if (!getFlags().getDumpStats())
return; return;
ThreadContext *TLS = ICE_TLS_GET_FIELD(TLS); ThreadContext *Tls = ICE_TLS_GET_FIELD(TLS);
TLS->StatsFunction.update(CodeStats::CS_FrameByte, Bytes); Tls->StatsFunction.update(CodeStats::CS_FrameByte, Bytes);
TLS->StatsCumulative.update(CodeStats::CS_FrameByte, Bytes); Tls->StatsCumulative.update(CodeStats::CS_FrameByte, Bytes);
} }
void statsUpdateSpills() { void statsUpdateSpills() {
if (!getFlags().getDumpStats()) if (!getFlags().getDumpStats())
return; return;
ThreadContext *TLS = ICE_TLS_GET_FIELD(TLS); ThreadContext *Tls = ICE_TLS_GET_FIELD(TLS);
TLS->StatsFunction.update(CodeStats::CS_NumSpills); Tls->StatsFunction.update(CodeStats::CS_NumSpills);
TLS->StatsCumulative.update(CodeStats::CS_NumSpills); Tls->StatsCumulative.update(CodeStats::CS_NumSpills);
} }
void statsUpdateFills() { void statsUpdateFills() {
if (!getFlags().getDumpStats()) if (!getFlags().getDumpStats())
return; return;
ThreadContext *TLS = ICE_TLS_GET_FIELD(TLS); ThreadContext *Tls = ICE_TLS_GET_FIELD(TLS);
TLS->StatsFunction.update(CodeStats::CS_NumFills); Tls->StatsFunction.update(CodeStats::CS_NumFills);
TLS->StatsCumulative.update(CodeStats::CS_NumFills); Tls->StatsCumulative.update(CodeStats::CS_NumFills);
} }
// These are predefined TimerStackIdT values. // These are predefined TimerStackIdT values.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment