colden added a comment. In https://reviews.llvm.org/D43700#1018087, @zturner wrote:
> In https://reviews.llvm.org/D43700#1018042, @colden wrote: > > > Seems good to me! I'll give it a test on my end. > > > > One alternate implementation idea though, what if you defaulted > > EmitCodeView to the hasArg check instead of false, then removed the `else > > *EmitCodeView = false;` block on line 4999? > > > That would actually change the behavior of the cl driver, which I kind of > don't want to do since it's not necessary. It would whitelist an additional > clang option to be recognized by the cl driver. Specifically, you could then > get debug info via the cl driver without specifying /Z7 or /Zi. It makes the > possibilities more confusing, and someone will invariably try to do it and > screw something up. We already whitelist some dash options so that the cl > driver will recognize them, but it's on a case by case basis and only when > there's a strong need for it. Good point, that's a pretty good reason not to. https://reviews.llvm.org/D43700 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits