mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-11-23 19:23:23 +01:00
add the ability to associate 'category' names with clang diagnostics
and diagnostic groups. This allows the compiler to group diagnostics together (e.g. "Logic Warning", "Format String Warning", etc) like the static analyzer does. This is not exposed through anything in the compiler yet. llvm-svn: 103050
This commit is contained in:
parent
1440201528
commit
7b17b0d5fb
@ -17,11 +17,108 @@
|
|||||||
#include "llvm/Support/Compiler.h"
|
#include "llvm/Support/Compiler.h"
|
||||||
#include "llvm/ADT/DenseSet.h"
|
#include "llvm/ADT/DenseSet.h"
|
||||||
#include "llvm/ADT/StringExtras.h"
|
#include "llvm/ADT/StringExtras.h"
|
||||||
|
#include "llvm/ADT/StringMap.h"
|
||||||
#include "llvm/ADT/VectorExtras.h"
|
#include "llvm/ADT/VectorExtras.h"
|
||||||
#include <set>
|
#include <set>
|
||||||
#include <map>
|
#include <map>
|
||||||
using namespace llvm;
|
using namespace llvm;
|
||||||
|
|
||||||
|
//===----------------------------------------------------------------------===//
|
||||||
|
// Diagnostic category computation code.
|
||||||
|
//===----------------------------------------------------------------------===//
|
||||||
|
|
||||||
|
namespace {
|
||||||
|
class DiagGroupParentMap {
|
||||||
|
std::map<const Record*, std::vector<Record*> > Mapping;
|
||||||
|
public:
|
||||||
|
DiagGroupParentMap() {
|
||||||
|
std::vector<Record*> DiagGroups
|
||||||
|
= Records.getAllDerivedDefinitions("DiagGroup");
|
||||||
|
for (unsigned i = 0, e = DiagGroups.size(); i != e; ++i) {
|
||||||
|
std::vector<Record*> SubGroups =
|
||||||
|
DiagGroups[i]->getValueAsListOfDefs("SubGroups");
|
||||||
|
for (unsigned j = 0, e = SubGroups.size(); j != e; ++j)
|
||||||
|
Mapping[SubGroups[j]].push_back(DiagGroups[i]);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
const std::vector<Record*> &getParents(const Record *Group) {
|
||||||
|
return Mapping[Group];
|
||||||
|
}
|
||||||
|
};
|
||||||
|
} // end anonymous namespace.
|
||||||
|
|
||||||
|
|
||||||
|
static std::string
|
||||||
|
getCategoryFromDiagGroup(const Record *Group,
|
||||||
|
DiagGroupParentMap &DiagGroupParents) {
|
||||||
|
// If the DiagGroup has a category, return it.
|
||||||
|
std::string CatName = Group->getValueAsString("CategoryName");
|
||||||
|
if (!CatName.empty()) return CatName;
|
||||||
|
|
||||||
|
// The diag group may the subgroup of one or more other diagnostic groups,
|
||||||
|
// check these for a category as well.
|
||||||
|
const std::vector<Record*> &Parents = DiagGroupParents.getParents(Group);
|
||||||
|
for (unsigned i = 0, e = Parents.size(); i != e; ++i) {
|
||||||
|
CatName = getCategoryFromDiagGroup(Parents[i], DiagGroupParents);
|
||||||
|
if (!CatName.empty()) return CatName;
|
||||||
|
}
|
||||||
|
return "";
|
||||||
|
}
|
||||||
|
|
||||||
|
/// getDiagnosticCategory - Return the category that the specified diagnostic
|
||||||
|
/// lives in.
|
||||||
|
static std::string getDiagnosticCategory(const Record *R,
|
||||||
|
DiagGroupParentMap &DiagGroupParents) {
|
||||||
|
// If the diagnostic itself has a category, get it.
|
||||||
|
std::string CatName = R->getValueAsString("CategoryName");
|
||||||
|
if (!CatName.empty()) return CatName;
|
||||||
|
|
||||||
|
DefInit *Group = dynamic_cast<DefInit*>(R->getValueInit("Group"));
|
||||||
|
if (Group == 0) return "";
|
||||||
|
|
||||||
|
// Check the diagnostic's diag group for a category.
|
||||||
|
return getCategoryFromDiagGroup(Group->getDef(), DiagGroupParents);
|
||||||
|
}
|
||||||
|
|
||||||
|
namespace {
|
||||||
|
class DiagCategoryIDMap {
|
||||||
|
StringMap<unsigned> CategoryIDs;
|
||||||
|
std::vector<std::string> CategoryStrings;
|
||||||
|
public:
|
||||||
|
DiagCategoryIDMap() {
|
||||||
|
DiagGroupParentMap ParentInfo;
|
||||||
|
|
||||||
|
// The zero'th category is "".
|
||||||
|
CategoryStrings.push_back("");
|
||||||
|
CategoryIDs[""] = 0;
|
||||||
|
|
||||||
|
std::vector<Record*> Diags =
|
||||||
|
Records.getAllDerivedDefinitions("Diagnostic");
|
||||||
|
for (unsigned i = 0, e = Diags.size(); i != e; ++i) {
|
||||||
|
std::string Category = getDiagnosticCategory(Diags[i], ParentInfo);
|
||||||
|
if (Category.empty()) continue; // Skip diags with no category.
|
||||||
|
|
||||||
|
unsigned &ID = CategoryIDs[Category];
|
||||||
|
if (ID != 0) continue; // Already seen.
|
||||||
|
|
||||||
|
ID = CategoryStrings.size();
|
||||||
|
CategoryStrings.push_back(Category);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
unsigned getID(StringRef CategoryString) {
|
||||||
|
return CategoryIDs[CategoryString];
|
||||||
|
}
|
||||||
|
|
||||||
|
typedef std::vector<std::string>::iterator iterator;
|
||||||
|
iterator begin() { return CategoryStrings.begin(); }
|
||||||
|
iterator end() { return CategoryStrings.end(); }
|
||||||
|
};
|
||||||
|
} // end anonymous namespace.
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
//===----------------------------------------------------------------------===//
|
//===----------------------------------------------------------------------===//
|
||||||
// Warning Tables (.inc file) generation.
|
// Warning Tables (.inc file) generation.
|
||||||
//===----------------------------------------------------------------------===//
|
//===----------------------------------------------------------------------===//
|
||||||
@ -40,6 +137,9 @@ void ClangDiagsDefsEmitter::run(raw_ostream &OS) {
|
|||||||
const std::vector<Record*> &Diags =
|
const std::vector<Record*> &Diags =
|
||||||
Records.getAllDerivedDefinitions("Diagnostic");
|
Records.getAllDerivedDefinitions("Diagnostic");
|
||||||
|
|
||||||
|
DiagCategoryIDMap CategoryIDs;
|
||||||
|
DiagGroupParentMap DGParentMap;
|
||||||
|
|
||||||
for (unsigned i = 0, e = Diags.size(); i != e; ++i) {
|
for (unsigned i = 0, e = Diags.size(); i != e; ++i) {
|
||||||
const Record &R = *Diags[i];
|
const Record &R = *Diags[i];
|
||||||
// Filter by component.
|
// Filter by component.
|
||||||
@ -67,6 +167,9 @@ void ClangDiagsDefsEmitter::run(raw_ostream &OS) {
|
|||||||
OS << ", true";
|
OS << ", true";
|
||||||
else
|
else
|
||||||
OS << ", false";
|
OS << ", false";
|
||||||
|
|
||||||
|
// Category number.
|
||||||
|
OS << ", " << CategoryIDs.getID(getDiagnosticCategory(&R, DGParentMap));
|
||||||
OS << ")\n";
|
OS << ")\n";
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@ -82,6 +185,9 @@ struct GroupInfo {
|
|||||||
};
|
};
|
||||||
|
|
||||||
void ClangDiagGroupsEmitter::run(raw_ostream &OS) {
|
void ClangDiagGroupsEmitter::run(raw_ostream &OS) {
|
||||||
|
// Compute a mapping from a DiagGroup to all of its parents.
|
||||||
|
DiagGroupParentMap DGParentMap;
|
||||||
|
|
||||||
// Invert the 1-[0/1] mapping of diags to group into a one to many mapping of
|
// Invert the 1-[0/1] mapping of diags to group into a one to many mapping of
|
||||||
// groups to diags in the group.
|
// groups to diags in the group.
|
||||||
std::map<std::string, GroupInfo> DiagsInGroup;
|
std::map<std::string, GroupInfo> DiagsInGroup;
|
||||||
@ -98,9 +204,10 @@ void ClangDiagGroupsEmitter::run(raw_ostream &OS) {
|
|||||||
|
|
||||||
// Add all DiagGroup's to the DiagsInGroup list to make sure we pick up empty
|
// Add all DiagGroup's to the DiagsInGroup list to make sure we pick up empty
|
||||||
// groups (these are warnings that GCC supports that clang never produces).
|
// groups (these are warnings that GCC supports that clang never produces).
|
||||||
Diags = Records.getAllDerivedDefinitions("DiagGroup");
|
std::vector<Record*> DiagGroups
|
||||||
for (unsigned i = 0, e = Diags.size(); i != e; ++i) {
|
= Records.getAllDerivedDefinitions("DiagGroup");
|
||||||
Record *Group = Diags[i];
|
for (unsigned i = 0, e = DiagGroups.size(); i != e; ++i) {
|
||||||
|
Record *Group = DiagGroups[i];
|
||||||
GroupInfo &GI = DiagsInGroup[Group->getValueAsString("GroupName")];
|
GroupInfo &GI = DiagsInGroup[Group->getValueAsString("GroupName")];
|
||||||
|
|
||||||
std::vector<Record*> SubGroups = Group->getValueAsListOfDefs("SubGroups");
|
std::vector<Record*> SubGroups = Group->getValueAsListOfDefs("SubGroups");
|
||||||
@ -167,4 +274,12 @@ void ClangDiagGroupsEmitter::run(raw_ostream &OS) {
|
|||||||
OS << " },\n";
|
OS << " },\n";
|
||||||
}
|
}
|
||||||
OS << "#endif // GET_DIAG_TABLE\n\n";
|
OS << "#endif // GET_DIAG_TABLE\n\n";
|
||||||
|
|
||||||
|
// Emit the category table next.
|
||||||
|
DiagCategoryIDMap CategoriesByID;
|
||||||
|
OS << "\n#ifdef GET_CATEGORY_TABLE\n";
|
||||||
|
for (DiagCategoryIDMap::iterator I = CategoriesByID.begin(),
|
||||||
|
E = CategoriesByID.end(); I != E; ++I)
|
||||||
|
OS << "CATEGORY(\"" << *I << "\")\n";
|
||||||
|
OS << "#endif // GET_CATEGORY_TABLE\n\n";
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user