[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