[patch] Assert that clang's datalayout strings are in sync with llvm's

Alp Toker alp at nuanti.com
Fri Dec 20 12:40:30 PST 2013


On 20/12/2013 20:28, Rafael EspĂ­ndola wrote:
>> I can try to implement that first if you agree it is the right thing to do.
> It was so simple that I decided to try it. What do you think?

Yep, I like this check.


+          "The input file triple doesn't match the -triple option");
+


Message could be clearer, and shouldn't assume "-triple" was specified 
on the command line given that this is a backend diag.

Bonus points if you can include the text of the triples and/or a 
SourceLocation!

Alp.


>
> Cheers,
> Rafael

-- 
http://www.nuanti.com
the browser experts




More information about the cfe-commits mailing list