fhahn added a reviewer: rengolin.
fhahn added a comment.

I like the idea. However for all backends, except Arm and AArch64, we would 
have to maintain another list of CPU names. At least for the targets which 
implement `isValidCPUName`, we could add an array with valid names and use 
that.  That's still not ideal, but I do not think we should hold back this 
patch until all targets use the same infrastructure to handle CPUs. As for 
tests, we should test the note for all backends I think.



================
Comment at: test/Misc/target-parser.c:3
 // CHECK: error: unknown target CPU 'not-a-cpu'
+// //CHECK: note: valid target CPU values are:
----------------
drop one set of //


Repository:
  rC Clang

https://reviews.llvm.org/D42978



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

Reply via email to