[flang] [clang] [flang] Enable alias tags pass by default (PR #73111)

Tom Eccles via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 27 07:09:56 PST 2023


tblah wrote:

> If you are in a rush to land this then this LGTM, but I would like the relationship between `-O{1|2|3|4}` and `-f{no}-alias-analysis` to be refined in a follow-up patch. Unless there's a good reason to avoid that? WDYT?

Thanks @banach-space, I will land this now and follow up later today.

The behavior you're commenting on was deliberate because to me, it feels wrong to enable alias analysis if there is an `-fno-alias-analysis` flag anywhere (and no `-falias-analysis`) as it it may not be obvious that -O implies `-falias-analysis`, and what `-O` does isn't documented anywhere.

But I don't feel strongly about it so I will follow up with a new patch later if you still feel that is worthwhile.

https://github.com/llvm/llvm-project/pull/73111


More information about the cfe-commits mailing list