thakis added inline comments.

================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:7620
+     else if (Arch == llvm::Triple::x86_64)
+       UnsupportedArches = {"AVX", "AVX2", "AVX512"};
+
----------------
Making this list opt-in instead of opt-out (ie "SupportedArchs") seems a bit 
more future-proof: We're less likely to forget to update the list when new 
archs are added.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126719/new/

https://reviews.llvm.org/D126719

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to