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

Jonas Hahnfeld via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 17 01:07:20 PDT 2023


Hahnfeld added a comment.

I dsabled two tests without `{arm,aarch64}-registered-target` in rGeeac4321c517ee8afc30ebe62c5b1778efc1173d <https://reviews.llvm.org/rGeeac4321c517ee8afc30ebe62c5b1778efc1173d>; two post-commit comments inline



================
Comment at: llvm/test/CodeGen/Generic/machine-function-splitter.ll:18
+; MFS_ON: Machine Function Splitter Transformation
+; MFS_ON_NO: warning: -fsplit-machine-functions is not valid for
+;; Check that MFS is not on for non-X86 targets.
----------------
shouldn't this be `MFS_ON-NOT`?


================
Comment at: llvm/test/CodeGen/Generic/machine-function-splitter.ll:21
+; MFS_OFF: warning: -fsplit-machine-functions is not valid for
+; MFS_OFF_NO: Machine Function Splitter Transformation
 
----------------



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