This is an archive of the discontinued LLVM Phabricator instance.

Fix Android build after r266267
Needs ReviewPublic

Authored by ovyalov on Apr 13 2016, 6:58 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

Fix Android build after r266267

Diff Detail

Event Timeline

ovyalov updated this revision to Diff 53653.Apr 13 2016, 6:58 PM
ovyalov retitled this revision from to Fix Android build after r266267.
ovyalov updated this object.
ovyalov added a reviewer: granata.enrico.
ovyalov added a subscriber: lldb-commits.

Why is this necessary? stdout is a local variable defined in this scope. Why would the android g++ have problems with this?

Anyway, if you have to avoid using stdout as a name, maybe name it std_out as that better reflects its meaning.

Why is this necessary? stdout is a local variable defined in this scope. Why would the android g++ have problems with this?

Anyway, if you have to avoid using stdout as a name, maybe name it std_out as that better reflects its meaning.

My understanding that NDK toolchain exposes stdout global variable that is caused name collision here.

labath added a subscriber: labath.Apr 14 2016, 11:07 AM

Judging by the cryptic error message, and the fact that a global variable should not cause an issue, i suspect stdout was a macro in this case...

granata.enrico resigned from this revision.Nov 29 2016, 10:05 AM
granata.enrico removed a reviewer: granata.enrico.

I am not an Apple employee working on LLDB anymore