================
@@ -79,7 +87,6 @@ class LLVM_LIBRARY_VISIBILITY BareMetal : public ToolChain {
OrderedMultilibs getOrderedMultilibs() const;
std::string SysRoot;
----------------
quic-garvgupt wrote:
Removing this private variable was making the code messy because there have
been updates to this variable in some member functions and the updated value is
then further used in other functions. Removing this variable would need to
pass this as an argument to all those functions which would make the code less
readable.
To avoid the confusion, all local variables for sysroot are named as
`SysRootDir` and any references to this variable as `SysRoot` only
https://github.com/llvm/llvm-project/pull/121829
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
