[llvm] [AMDGPU] AMDGPULateCodeGenPrepare Legacy PM: replace `setPreservesAll()` with `setPreservesCFG()` (PR #148167)

[Jay Foad via llvm-commits]

Jim =?utf-8?q?Teichgräber?= <Jim.Teichgraber at amd.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/148167 at github.com>


================
@@ -546,7 +546,9 @@ class AMDGPULateCodeGenPrepareLegacy : public FunctionPass {
     AU.addRequired<TargetPassConfig>();
     AU.addRequired<AssumptionCacheTracker>();
     AU.addRequired<UniformityInfoWrapperPass>();
-    AU.setPreservesAll();
+    // makes changes that can invalidate Uniformity Analysis,
+    // so don't preserveAll here (see new PM version above)
----------------
jayfoad wrote:

Nit: comments should be full sentences with a capital letter and a period.

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