MDString
- Rename member function size(). New name is length().
- Store string beginning and length. Earlier it used to store string end.
llvm-svn: 76841
diff --git a/llvm/include/llvm/LLVMContext.h b/llvm/include/llvm/LLVMContext.h
index 134311e..0751ccd 100644
--- a/llvm/include/llvm/LLVMContext.h
+++ b/llvm/include/llvm/LLVMContext.h
@@ -232,7 +232,7 @@
MDNode* getMDNode(Value* const* Vals, unsigned NumVals);
// MDString accessors
- MDString* getMDString(const char *StrBegin, const char *StrEnd);
+ MDString* getMDString(const char *StrBegin, unsigned Length);
MDString* getMDString(const std::string &Str);
// FunctionType accessors
diff --git a/llvm/include/llvm/MDNode.h b/llvm/include/llvm/MDNode.h
index 14bfe57..06cc909 100644
--- a/llvm/include/llvm/MDNode.h
+++ b/llvm/include/llvm/MDNode.h
@@ -64,16 +64,17 @@
///
class MDString : public MetadataBase {
MDString(const MDString &); // DO NOT IMPLEMENT
- const char *StrBegin, *StrEnd;
+ const char *StrBegin;
+ unsigned StrLength;
friend class LLVMContextImpl;
protected:
- explicit MDString(const char *begin, const char *end)
+ explicit MDString(const char *begin, unsigned l)
: MetadataBase(Type::MetadataTy, Value::MDStringVal),
- StrBegin(begin), StrEnd(end) {}
+ StrBegin(begin), StrLength(l) {}
public:
- intptr_t size() const { return StrEnd - StrBegin; }
+ unsigned length() const { return StrLength; }
/// begin() - Pointer to the first byte of the string.
///
@@ -81,7 +82,7 @@
/// end() - Pointer to one byte past the end of the string.
///
- const char *end() const { return StrEnd; }
+ const char *end() const { return StrBegin + length(); }
/// Methods for support type inquiry through isa, cast, and dyn_cast:
static inline bool classof(const MDString *) { return true; }
diff --git a/llvm/lib/AsmParser/LLParser.cpp b/llvm/lib/AsmParser/LLParser.cpp
index b09f23f..94224a6 100644
--- a/llvm/lib/AsmParser/LLParser.cpp
+++ b/llvm/lib/AsmParser/LLParser.cpp
@@ -373,7 +373,7 @@
bool LLParser::ParseMDString(MetadataBase *&MDS) {
std::string Str;
if (ParseStringConstant(Str)) return true;
- MDS = Context.getMDString(Str.data(), Str.data() + Str.size());
+ MDS = Context.getMDString(Str.data(), Str.size());
return false;
}
diff --git a/llvm/lib/Bitcode/Reader/BitcodeReader.cpp b/llvm/lib/Bitcode/Reader/BitcodeReader.cpp
index 8393cd4..01b3eb3 100644
--- a/llvm/lib/Bitcode/Reader/BitcodeReader.cpp
+++ b/llvm/lib/Bitcode/Reader/BitcodeReader.cpp
@@ -758,7 +758,7 @@
for (unsigned i = 0; i != MDStringLength; ++i)
String[i] = Record[i];
Value *V =
- Context.getMDString(String.c_str(), String.c_str() + MDStringLength);
+ Context.getMDString(String.c_str(), MDStringLength);
ValueList.AssignValue(V, NextValueNo++);
break;
}
diff --git a/llvm/lib/Bitcode/Writer/BitcodeWriter.cpp b/llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
index 7d745ff..b312420 100644
--- a/llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
+++ b/llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
@@ -512,7 +512,7 @@
// Code: [strchar x N]
const char *StrBegin = MDS->begin();
- for (unsigned i = 0, e = MDS->size(); i != e; ++i)
+ for (unsigned i = 0, e = MDS->length(); i != e; ++i)
Record.push_back(StrBegin[i]);
// Emit the finished record.
diff --git a/llvm/lib/VMCore/AsmWriter.cpp b/llvm/lib/VMCore/AsmWriter.cpp
index ae55d5b..73310a3 100644
--- a/llvm/lib/VMCore/AsmWriter.cpp
+++ b/llvm/lib/VMCore/AsmWriter.cpp
@@ -1139,7 +1139,7 @@
if (const MDString *MDS = dyn_cast<MDString>(V)) {
Out << "!\"";
- PrintEscapedString(MDS->begin(), MDS->size(), Out);
+ PrintEscapedString(MDS->begin(), MDS->length(), Out);
Out << '"';
return;
}
@@ -1983,7 +1983,7 @@
TypePrinter.print(MDS->getType(), OS);
OS << ' ';
OS << "!\"";
- PrintEscapedString(MDS->begin(), MDS->size(), OS);
+ PrintEscapedString(MDS->begin(), MDS->length(), OS);
OS << '"';
} else if (const MDNode *N = dyn_cast<MDNode>(this)) {
SlotTracker SlotTable(N);
diff --git a/llvm/lib/VMCore/LLVMContext.cpp b/llvm/lib/VMCore/LLVMContext.cpp
index c88ecba..d0abb2d 100644
--- a/llvm/lib/VMCore/LLVMContext.cpp
+++ b/llvm/lib/VMCore/LLVMContext.cpp
@@ -551,12 +551,12 @@
}
// MDString accessors
-MDString* LLVMContext::getMDString(const char *StrBegin, const char *StrEnd) {
- return pImpl->getMDString(StrBegin, StrEnd);
+MDString* LLVMContext::getMDString(const char *StrBegin, unsigned StrLength) {
+ return pImpl->getMDString(StrBegin, StrLength);
}
MDString* LLVMContext::getMDString(const std::string &Str) {
- return getMDString(Str.data(), Str.data()+Str.size());
+ return getMDString(Str.data(), Str.size());
}
// FunctionType accessors
diff --git a/llvm/lib/VMCore/LLVMContextImpl.cpp b/llvm/lib/VMCore/LLVMContextImpl.cpp
index 8ad46a0..79b33c1 100644
--- a/llvm/lib/VMCore/LLVMContextImpl.cpp
+++ b/llvm/lib/VMCore/LLVMContextImpl.cpp
@@ -394,13 +394,13 @@
}
MDString *LLVMContextImpl::getMDString(const char *StrBegin,
- const char *StrEnd) {
+ unsigned StrLength) {
sys::SmartScopedWriter<true> Writer(ConstantsLock);
- StringMapEntry<MDString *> &Entry = MDStringCache.GetOrCreateValue(
- StrBegin, StrEnd);
+ StringMapEntry<MDString *> &Entry =
+ MDStringCache.GetOrCreateValue(StrBegin, StrBegin + StrLength);
MDString *&S = Entry.getValue();
if (!S) S = new MDString(Entry.getKeyData(),
- Entry.getKeyData() + Entry.getKeyLength());
+ Entry.getKeyLength());
return S;
}
@@ -460,7 +460,8 @@
void LLVMContextImpl::erase(MDString *M) {
sys::SmartScopedWriter<true> Writer(ConstantsLock);
- MDStringCache.erase(MDStringCache.find(M->StrBegin, M->StrEnd));
+ MDStringCache.erase(MDStringCache.find(M->StrBegin,
+ M->StrBegin + M->length()));
}
void LLVMContextImpl::erase(MDNode *M) {
diff --git a/llvm/lib/VMCore/LLVMContextImpl.h b/llvm/lib/VMCore/LLVMContextImpl.h
index 03f0e7b..392e500 100644
--- a/llvm/lib/VMCore/LLVMContextImpl.h
+++ b/llvm/lib/VMCore/LLVMContextImpl.h
@@ -133,7 +133,7 @@
ConstantFP *getConstantFP(const APFloat &V);
- MDString *getMDString(const char *StrBegin, const char *StrEnd);
+ MDString *getMDString(const char *StrBegin, unsigned StrLength);
MDNode *getMDNode(Value*const* Vals, unsigned NumVals);
diff --git a/llvm/unittests/VMCore/MetadataTest.cpp b/llvm/unittests/VMCore/MetadataTest.cpp
index 732719e..d612808 100644
--- a/llvm/unittests/VMCore/MetadataTest.cpp
+++ b/llvm/unittests/VMCore/MetadataTest.cpp
@@ -23,9 +23,9 @@
// MDString objects, even with the same string pointer and nulls in the string.
TEST(MDStringTest, CreateDifferent) {
char x[3] = { 'f', 0, 'A' };
- MDString *s1 = getGlobalContext().getMDString(&x[0], &x[3]);
+ MDString *s1 = getGlobalContext().getMDString(&x[0], 3);
x[2] = 'B';
- MDString *s2 = getGlobalContext().getMDString(&x[0], &x[3]);
+ MDString *s2 = getGlobalContext().getMDString(&x[0], 3);
EXPECT_NE(s1, s2);
}
@@ -35,8 +35,8 @@
char x[4] = { 'a', 'b', 'c', 'X' };
char y[4] = { 'a', 'b', 'c', 'Y' };
- MDString *s1 = getGlobalContext().getMDString(&x[0], &x[3]);
- MDString *s2 = getGlobalContext().getMDString(&y[0], &y[3]);
+ MDString *s1 = getGlobalContext().getMDString(&x[0], 3);
+ MDString *s2 = getGlobalContext().getMDString(&y[0], 3);
EXPECT_EQ(s1, s2);
}
@@ -44,7 +44,7 @@
TEST(MDStringTest, PrintingSimple) {
char *str = new char[13];
strncpy(str, "testing 1 2 3", 13);
- MDString *s = getGlobalContext().getMDString(str, str+13);
+ MDString *s = getGlobalContext().getMDString(str, 13);
strncpy(str, "aaaaaaaaaaaaa", 13);
delete[] str;
@@ -56,7 +56,7 @@
// Test printing of MDString with non-printable characters.
TEST(MDStringTest, PrintingComplex) {
char str[5] = {0, '\n', '"', '\\', -1};
- MDString *s = getGlobalContext().getMDString(str+0, str+5);
+ MDString *s = getGlobalContext().getMDString(str+0, 5);
std::ostringstream oss;
s->print(oss);
EXPECT_STREQ("metadata !\"\\00\\0A\\22\\5C\\FF\"", oss.str().c_str());
@@ -67,8 +67,8 @@
char x[3] = { 'a', 'b', 'c' };
char y[3] = { '1', '2', '3' };
- MDString *s1 = getGlobalContext().getMDString(&x[0], &x[3]);
- MDString *s2 = getGlobalContext().getMDString(&y[0], &y[3]);
+ MDString *s1 = getGlobalContext().getMDString(&x[0], 3);
+ MDString *s2 = getGlobalContext().getMDString(&y[0], 3);
ConstantInt *CI = getGlobalContext().getConstantInt(APInt(8, 0));
std::vector<Value *> V;