[llvm] [AMDGPU] Do not preserve UniformityInfo (PR #76696)

Sameer Sahasrabuddhe via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 2 01:35:21 PST 2024


================
@@ -33,6 +33,12 @@
 /// the propagation of the impact of divergent control flow on the divergence of
 /// values (sync dependencies).
 ///
+/// NOTE: In general, no interface exists for a transform to update
+/// (Machine)UniformityInfo. Additionally, (Machine)CycleAnalysis is a
+/// transitive dependence, but it also does not provide an interface for
+/// updation. Given that, transforms should not preserve uniformity in their
----------------
ssahasra wrote:

I had looked it up ... "updation" is a word, and it's use as a noun is not wrong here. "updating" requires an object after it. But it was not clear to me if the verb "updating" is preferred over the noun "updation" in general ... so went with @arsenm's preference, writing it off as an ESL thing, and/or a tussle between en_US, en_UK and en_IN :)

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


More information about the llvm-commits mailing list