[PATCH] D157565: [CodeGen] Add AArch64 behavior to existing MFS tests

[Daniel Hoekwater via Phabricator via llvm-commits]

dhoekwater marked an inline comment as done.
dhoekwater added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineFunctionSplitter.cpp:73
+// function splitting is fully enabled on AArch64.
+static cl::opt<bool> IgnoreTripleForDebug(
+    "mfs-ignore-triple",
----------------
rahmanl wrote:
> How about introducing a permanent option named `SkipUnsupportedTriple` and default it True so we can use it for any future architectures?
I'm onboard with making this a permanent flag for testing purposes only, but I'm not sure about inverting and defaulting to true. Isn't it standard to have default-false hidden flags that enable some special behavior (as opposed to default-true hidden flags that enable the default behavior)?

For example, most of the `cl::opt<bool>` options in `llvm/lib/Analysis/InlineCost.cpp` follow this pattern.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157565