================
@@ -51,10 +51,14 @@ PreservedAnalyses FEntryInserterPass::run(MachineFunction 
&MF,
 }
 
 bool FEntryInserter::run(MachineFunction &MF) {
-  const std::string FEntryName = std::string(
-      MF.getFunction().getFnAttribute("fentry-call").getValueAsString());
-  if (FEntryName != "true")
+  const Function &F = MF.getFunction();
+  if (F.getFnAttribute("fentry-call").getValueAsString() != "true") {
+    if (F.hasFnAttribute("mnop-mcount"))
+      report_fatal_error("mnop-mcount only supported with fentry-call");
----------------
nikic wrote:

```suggestion
      reportFatalUsageError("mnop-mcount only supported with fentry-call");
```
Here and for the next one, and then drop the `--crash` from the test.

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

Reply via email to