Change the error message when `-split-input-file` is used with mlir-opt to make it...
authorMehdi Amini <joker.eph@gmail.com>
Sat, 27 Feb 2021 18:32:21 +0000 (18:32 +0000)
committerMehdi Amini <joker.eph@gmail.com>
Sat, 27 Feb 2021 18:32:34 +0000 (18:32 +0000)
By adding the line number of the split point immediately after the file
name (separated by `:`) this is recognized by various tool as a proper
location.

Ideally we would want to point to the line of the error, but that would
require some very invasive changes I suspect.

Reviewed By: jpienaar

Differential Revision: https://reviews.llvm.org/D93363

mlir/lib/Support/ToolUtilities.cpp

index 26a8b08..ef1a411 100644 (file)
@@ -37,8 +37,9 @@ mlir::splitAndProcessBuffer(std::unique_ptr<llvm::MemoryBuffer> originalBuffer,
     auto splitLoc = llvm::SMLoc::getFromPointer(subBuffer.data());
     unsigned splitLine = fileSourceMgr.getLineAndColumn(splitLoc).first;
     auto subMemBuffer = llvm::MemoryBuffer::getMemBufferCopy(
-        subBuffer, origMemBuffer->getBufferIdentifier() +
-                       Twine(" split at line #") + Twine(splitLine));
+        subBuffer, Twine("within split at ") +
+                       origMemBuffer->getBufferIdentifier() + ":" +
+                       Twine(splitLine) + " offset ");
     if (failed(processChunkBuffer(std::move(subMemBuffer), os)))
       hadFailure = true;
   }