[Lldb-commits] [PATCH] D17088: Add target and host platform enums so we're not using strings everywhere

[Pavel Labath via lldb-commits]

labath added a comment.

I like the idea. I think this could help us with the linux vs. android dilemma we've been having recently: currently, we consider android to be a flavour of linux, which means it's not possible to XFAIL tests passing on android but failing on linux. This should make expressing something like that easier.

I wonder, however, whether you need separate enums for `host` and `target`. I mean, any system can be used as one or the other, right? (Granted, nobody has tried bringing up lldb client on e.g., android, but I actually don't expect it would be so hard, and I don't see a reason to not support it here). What would you say to merging the two enums and calling it simply `os`, `platform` or something?


http://reviews.llvm.org/D17088