diff options
author | Richard Trieu <rtrieu@google.com> | 2014-06-09 22:53:16 +0000 |
---|---|---|
committer | Richard Trieu <rtrieu@google.com> | 2014-06-09 22:53:16 +0000 |
commit | a23043cb9c1ef021a9cf05cd62cce76cd03c0ba2 (patch) | |
tree | 65eda585bf8ca912429cd6c56f02c614337aa6a2 /llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp | |
parent | d3d017cf00714c7d376a3cf87d2a17002e12cac7 (diff) | |
download | bcm5719-llvm-a23043cb9c1ef021a9cf05cd62cce76cd03c0ba2.tar.gz bcm5719-llvm-a23043cb9c1ef021a9cf05cd62cce76cd03c0ba2.zip |
Removing an "if (!this)" check from two print methods. The condition will
never be true in a well-defined context. The checking for null pointers
has been moved into the caller logic so it does not rely on undefined behavior.
llvm-svn: 210497
Diffstat (limited to 'llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp')
-rw-r--r-- | llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp b/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp index 9d822d3cb1a..67b57fa2b4c 100644 --- a/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp +++ b/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp @@ -1867,6 +1867,7 @@ static void emitGlobalConstantFP(const ConstantFP *CFP, AsmPrinter &AP) { SmallString<8> StrVal; CFP->getValueAPF().toString(StrVal); + assert(CFP->getType() != nullptr && "Expecting non-null Type"); CFP->getType()->print(AP.OutStreamer.GetCommentOS()); AP.OutStreamer.GetCommentOS() << ' ' << StrVal << '\n'; } |