[PATCH] D19274: Compilation for Intel MCU (Part 2/3)

Andrey Turetskiy via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 15 07:03:33 PDT 2016


aturetsk added inline comments.

================
Comment at: lib/Driver/Tools.cpp:3657
@@ +3656,3 @@
+  if (IsIAMCU && types::isCXX(Input.getType()))
+    D.Diag(diag::err_drv_cxx_not_supported) << getToolChain().getTriple().str();
+
----------------
aturetsk wrote:
> bruno wrote:
> > aturetsk wrote:
> > > bruno wrote:
> > > > Taking a look at this again I don't think there's a real need for a new diagnostic here; instead of adding diag::err_drv_cxx_not_supported, you can do something similar to:
> > > > 
> > > > D.Diag(diag::err_drv_clang_unsupported) << "C++ is not supported with -miamcu"
> > > > 
> > > > Otherwise, LGTM!
> > > The best thing I could come up with is this:
> > > ```
> > > D.Diag(diag::err_drv_clang_unsupported) << "C++ for IAMCU";
> > > ```
> > > This code makes the error message look like this:
> > > ```
> > > error: the clang compiler does not support 'C++ for IAMCU'
> > > ```
> > > As you can see the message is a bit crooked. Do you thinks it's better to keep it this way?
> > Sorry, but I didn't mean you should change the message. My only point here is that I don't see the need for adding a new diagnostic, you can still keep something more similar with your old intent, example:
> > 
> >   D.Diag(diag::err_drv_clang_unsupported) << "C++ for target" << getToolChain().getTriple().str();
> My concern is not the change in the message :) My concern is redundant quotation marks which look a bit ugly and out of place.
> But if you're OK with them, so would I.
Bruno, are you ok if the message will be:
```
error: the clang compiler does not support 'C++ for IAMCU'
```
If so I'll proceed with committing the changes.


http://reviews.llvm.org/D19274





More information about the cfe-commits mailing list