dexonsmith requested changes to this revision. dexonsmith added a comment. This revision now requires changes to proceed.
I agree with Saleem and Bob: `__is_target_*` is not confusing here and seems to be a straightforward spelling. It has also already shipped in LLVM 6.0.0: it would be awkward to stop supporting this syntax. Regardless, it's not clear that this patch is the right direction (i.e., we're not discussing the patch at all right now). I suggest moving the discussion back to the wider audience on cfe-dev until we have consensus for a change. Repository: rC Clang https://reviews.llvm.org/D44753 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
