[clang] [flang] [flang] Update -falias-analysis help text (PR #73548)

2023-12-05 Thread Tom Eccles via cfe-commits 

tblah wrote:

Closing this because I decided to just remove the `-f[no-]alias-analysis` flags 
instead. https://github.com/llvm/llvm-project/pull/74343

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

[clang] [flang] [flang] Update -falias-analysis help text (PR #73548)

2023-11-28 Thread Tom Eccles via cfe-commits 


@@ -6334,8 +6334,8 @@ defm loop_versioning : BoolOptionWithoutMarshalling<"f", 
"version-loops-for-stri
   PosFlag,
NegFlag>;
 defm alias_analysis : BoolOptionWithoutMarshalling<"f", "alias-analysis",
-  PosFlag,
-  NegFlag>;
+  PosFlag,
+  NegFlag>;
 } // let Visibility = [FC1Option, FlangOption]

tblah wrote:

Sure. Would it be okay to do that in a few weeks once the dust has settled from 
enabling it by default? `-fno-alias-analysis` is an easy work around if any 
problems show up

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

[clang] [flang] [flang] Update -falias-analysis help text (PR #73548)

2023-11-28 Thread Andrzej Warzyński via cfe-commits 


@@ -6334,8 +6334,8 @@ defm loop_versioning : BoolOptionWithoutMarshalling<"f", 
"version-loops-for-stri
   PosFlag,
NegFlag>;
 defm alias_analysis : BoolOptionWithoutMarshalling<"f", "alias-analysis",
-  PosFlag,
-  NegFlag>;
+  PosFlag,
+  NegFlag>;
 } // let Visibility = [FC1Option, FlangOption]

banach-space wrote:

Perhaps restrict to `FC1Option`?

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