[PATCH] D70263: [Error] Add source location macro

[Lang Hames via Phabricator via llvm-commits]

lhames added a comment.

I'm not sure I follow this. My understanding is that FileError was designed to make it easy to attach a source location to an error generated while operating on a file, but this patch is adapting it to track the LLVM source location that generated the error, right?

I don't think we want to do that -- it makes it very difficult to reason about what FileError's fields mean.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70263/new/

https://reviews.llvm.org/D70263