[Mlir-commits] [mlir] a6d76cf - Change the error message when `-split-input-file` is used with mlir-opt to make it recognizable by IDEs

Mehdi Amini llvmlistbot at llvm.org
Sat Feb 27 10:32:46 PST 2021


Author: Mehdi Amini
Date: 2021-02-27T18:32:34Z
New Revision: a6d76cf6101d0479af2d9c9266427c0ea51e3889

URL: https://github.com/llvm/llvm-project/commit/a6d76cf6101d0479af2d9c9266427c0ea51e3889
DIFF: https://github.com/llvm/llvm-project/commit/a6d76cf6101d0479af2d9c9266427c0ea51e3889.diff

LOG: Change the error message when `-split-input-file` is used with mlir-opt to make it recognizable by IDEs

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

Added: 
    

Modified: 
    mlir/lib/Support/ToolUtilities.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Support/ToolUtilities.cpp b/mlir/lib/Support/ToolUtilities.cpp
index 26a8b08f75af..ef1a41114fe9 100644
--- a/mlir/lib/Support/ToolUtilities.cpp
+++ b/mlir/lib/Support/ToolUtilities.cpp
@@ -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;
   }


        


More information about the Mlir-commits mailing list