GGanesh added inline comments.
================ Comment at: lib/Basic/Targets.cpp:3189 break; + case CK_ZNVER1: + setFeatureEnabledImpl(Features, "adx", true); ---------------- RKSimon wrote: > Same as what I asked on D28017 - is there an accepted order that we should be > using here? Some of them seems to be chronological. Some of them are alphabetical. I personally don't have any preference as such. Alphabetical order suits a long list. I would like to know your suggestion. https://reviews.llvm.org/D28018 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits