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

Reply via email to