================
@@ -1755,6 +1755,18 @@ Compilation *Driver::BuildCompilation(ArrayRef<const 
char *> ArgList) {
         << TC.getTriple().str();
   }
 
+  // Throw diagnosis if mcmodel=tiny option is passed for targets other than 
ARM
+  // or AArch64.
+  if (Arg *A = UArgs->getLastArg(options::OPT_mcmodel_EQ)) {
----------------
ShashwathiNavada wrote:

@MaskRay Thank you for your response!
I noticed that in the function you mentioned, there’s a check like this:

```
else if (Triple.getArch() == llvm::Triple::x86_64) {
    Ok = llvm::is_contained({"small", "kernel", "medium", "large", "tiny"}, CM);
}
```
I found that removing the tiny option here gives the desired result.
Would you mind confirming if this approach looks good before I push the changes?

https://github.com/llvm/llvm-project/pull/125643
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to