Jan =?utf-8?q?Górski?= <jan.a.gor...@wp.pl>, Jan =?utf-8?q?Górski?= <jan.a.gor...@wp.pl>, Jan =?utf-8?q?Górski?= <jan.a.gor...@wp.pl>, Jan =?utf-8?q?Górski?= <jan.a.gor...@wp.pl>, Jan =?utf-8?q?Górski?= <jan.a.gor...@wp.pl> Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/133...@github.com>
efriedma-quic wrote: Oh, hmm, we have range metadata, but it gets killed by mem2reg. So instcombine could do the transform, but it runs too late. (In theory it could try to preserve range metadata; see convertMetadataToAssumes . But probably simpler here to just generate the markings.) > make getRangeForLoadFromType a public method and use it for atomic load as > well This seems fine. https://github.com/llvm/llvm-project/pull/133546 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
