<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi Nico,<div class=""><br class=""></div><div class="">Thanks for reaching out. Michael Spencer is probably the best contact for this kind of question these days; I’ve cc’ed him.<br class=""><div><br class=""></div><div>Duncan</div><div><br class=""><blockquote type="cite" class=""><div class="">On 2020-11-12, at 09:04, Nico Weber <<a href="mailto:thakis@chromium.org" class="">thakis@chromium.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Hi,<div class=""><br class=""></div><div class=""><a href="http://reviews.llvm.org/D76801" class="">http://reviews.llvm.org/D76801</a> changed clang/lib/AST/TypePrinter.cpp to not print a space between consecutive '>'s in templates if targeting C++11 or later.<br class=""></div><div class=""><br class=""></div><div class="">I noticed recently that this affects Objective-C's @encode too, which means it has different results based on language standard, making it impossible to build part of a program with C++98 and another part with C++11 (...if both parts need @encode if some common type to agree). It also means @encode for types changed with that patch, which is an ABI break of sorts.</div><div class=""><br class=""></div><div class="">However, the change landed in March, so maybe it's already deployed in new Xcodes.</div><div class=""><br class=""></div><div class="">Should we make @encode opt out of this pretty printing change to restore the old behavior? (Several other places also broke due to that change and were changed to opt out -- debug info, for example).</div><div class=""><br class=""></div><div class="">I found this while writing tests for an unrelated change [1] and I don't know of any projects adversely affected by this.</div><div class=""><br class=""></div><div class="">Nico</div><div class=""><br class=""></div><div class=""><br class=""></div><div class="">1: See "FIXME" in <a href="https://reviews.llvm.org/D90622#change-Aht3gYP3AIRe" class="">https://reviews.llvm.org/D90622#change-Aht3gYP3AIRe</a></div></div>
</div></blockquote></div><br class=""></div></body></html>