[PATCH] D112387: [clang] [MinGW] Rename the 'Arch' member to 'SysrootName'. NFC.

Martin Storsjö via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 27 10:31:42 PDT 2021


mstorsjo added a comment.

In D112387#3090707 <https://reviews.llvm.org/D112387#3090707>, @MaskRay wrote:

> How is this different from `clang::driver::Driver::SysRoot`?
>
> From `SysrootName + "/sys-root"` looks like the `sys-root` directory is the sysroot? Perhaps there is a better way describing `SysrootName` but I cannot think of it...
> `SysrootName` may be good enough.

Yeah, it’s kinda vague what it is. It’s mainly a triple, except that it also can be “mingw32” or similar, in some cases. I guess it could be `SubdirName` too, or something related to that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112387



More information about the cfe-commits mailing list