[PATCH] D157750: Properly handle -fsplit-machine-functions for fatbinary compilation

Matt Arsenault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 14 12:05:01 PDT 2023


arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/TargetPassConfig.cpp:1281-1282
+    else
+      WithColor::warning()
+          << "-fsplit-machine-functions is only valid for X86.\n";
   }
----------------
shenhan wrote:
> arsenm wrote:
> > You cannot spam warnings here. The other instance of printing here looks like a new addition and should be removed
> Thanks. Do you suggest moving the warnings to the underlying pass? (Although that means we create passes that only issue warnings.)
Move it to the pass, and use a backend remark, not directly print to the console (e.g. DiagnosticInfoUnsupported)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157750



More information about the cfe-commits mailing list