================
@@ -202,10 +202,10 @@ ToolChain::findMultilibsYAML(const llvm::opt::ArgList 
&Args, const Driver &D,
   }
 
   SmallString<128> MultilibPath;
-  if (!FallbackDir.empty())
----------------
lenary wrote:

Ok, I haven't had time to dig deeply into the code.

I have some questions:

- are both the `triple/multilib.yaml` and `multilib.yaml` loaded, or only the 
first one found?
- in the baremetal `multilib.yaml`, right now the main paths in the config are 
relative to the single `multilib.yaml` itself (which is in a top-level 
directory), rather than applying somewhere else (though resource dir path 
management is different and not very well tested). What's the plan here in the 
per-triple directories?

https://github.com/llvm/llvm-project/pull/227965
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to