zahiraam added a comment. In D159486#4642317 <https://reviews.llvm.org/D159486#4642317>, @smeenai wrote:
> I don't think pre-merge testing covers Mac, only Linux and Windows (which is > presumably how the original breakage got in). If the test passes for you > locally I'd just push it and monitor the bots afterwards. (That isn't general > advice, but in this specific case, since we already have broken bots, I think > it's worth prioritizing a potential fix for them over waiting for previously > known-good configurations.) Done. Thanks. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D159486/new/ https://reviews.llvm.org/D159486 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
