One of our internal builders choked on free in Demangle.cpp.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
LGTM, if it is moved into Demangle.cpp. I don't think we want to be adding unnecessary dependencies to headers, when a .cpp file change would suffice.
include/llvm/Demangle/Demangle.h | ||
---|---|---|
13 | Shouldn't this be in Demangle.cpp, since the only usage is there? |
Shouldn't this be in Demangle.cpp, since the only usage is there?