dmgreen added a subscriber: rsmith. dmgreen accepted this revision. dmgreen added a comment.
Yeah, OK. D67678 <https://reviews.llvm.org/D67678> is the patch that will change the default, but only to "int", not to "none" for the moment. An earlier version of that patch had a different way of setting the default, in lib/Basic/Targets/ARM.cpp. I'm not sure that's better or worse than this, they both look fine to me. Under that condition, LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67160/new/ https://reviews.llvm.org/D67160 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
