erichkeane added a comment. In D151537#4382811 <https://reviews.llvm.org/D151537#4382811>, @LuoYuanke wrote:
> In D151537#4380763 <https://reviews.llvm.org/D151537#4380763>, @erichkeane > wrote: > >> I don't really see the justification here? Why do this change? If the >> intent is to just test a newer architecture, we can add tests for that, not >> change existing ones. > > KNL is deprecated and it is better to remove KNL related code in clang/llvm. IS it deprecated in Clang? I see an RFC proposing it, but I don't see acceptance of that yet. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151537/new/ https://reviews.llvm.org/D151537 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits