[PATCH] Add diagnostic capabilities in LLVM (frontend part)
Quentin Colombet
qcolombet at apple.com
Wed Jan 22 15:44:32 PST 2014
Ping?
@Dmitri Gribenko
The name of the added warning is clearly debatable and in particular I do not have on strong opinion on them (especially regarding the backend- prefix).
I did not find any equivalent for the inline-asm diagnostic in GCC, thus there is nothing to match here AFAICT.
Regarding the stack-size thing, the closest thing in GCC is frame-larger-than. Since we cannot, AFAIK, set an integer value to a warning in clang, we cannot match this for now.
@Hal
I do not have a good answer for the demangling of function’s name in the reporting. I believe this could be addressed later and in probably a better way by front-end people.
In other words, we can make a RFC for this after this specific plumbing lands.
The bottom line is, I do not think we should hold on this patch because of that.
Thanks for your feedbacks.
-Quentin
http://llvm-reviews.chandlerc.com/D2430
More information about the cfe-commits
mailing list