[PATCH] D13229: Add -Wc, option

Saleem Abdulrasool via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 28 19:01:57 PDT 2015


compnerd added a comment.

While I can certainly appreciate the simplification this may afford, Im not sure if adding a new option here is really that valuable.  Options being added to the frontend are expensive because they can't be changed or removed.  If gcc has a similar frontend option, we may consider it for compatibility (as was the case for -fplugin).  In this case, yes, it is harder to type options that we are passing down, but they are backend options and not meant to be used by users in the general case.  The -Xclang and -mllvm options currently are available, and make it clear enough where the option is going.  Arguably this is a bad idea since it makes it easier to pull backend options into the frontend.  Is there a stronger reason for this other than aesthetics?


http://reviews.llvm.org/D13229





More information about the cfe-commits mailing list