[FA] Fix compile failure on advanced path (#3567)

whitneywhtsang · web-flow · commit cd4f49b40fe2 · 2025-02-27T22:47:48.000Z
The workaround for advanced path was accidentally removed in dc237c2. Signed-off-by: Whitney Tsang <whitney.tsang@intel.com>
diff --git a/lib/Analysis/Utility.cpp b/lib/Analysis/Utility.cpp
@@ -108,6 +108,11 @@ unsigned ReduceOpHelper::getIntraWarpSizeWithUniqueData() {
 }
 
 bool ReduceOpHelper::isWarpSynchronous() {
+  // FIXME: In the default path tensors will always have a layout. Tensors do
+  // not have a layout only in the advanced path. We need to find a workaround
+  // in order to remove this change.
+  if (!srcEncoding)
+    return true;
   return getWarpsPerCTAWithUniqueData(srcEncoding, srcShape)[axis] == 1;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -108,6 +108,11 @@ unsigned ReduceOpHelper::getIntraWarpSizeWithUniqueData() {`
`108`	`108`	`}`
`109`	`109`
`110`	`110`	`bool ReduceOpHelper::isWarpSynchronous() {`
	`111`	`+ // FIXME: In the default path tensors will always have a layout. Tensors do`
	`112`	`+ // not have a layout only in the advanced path. We need to find a workaround`
	`113`	`+ // in order to remove this change.`
	`114`	`+ if (!srcEncoding)`
	`115`	`+ return true;`
`111`	`116`	`return getWarpsPerCTAWithUniqueData(srcEncoding, srcShape)[axis] == 1;`
`112`	`117`	`}`
`113`	`118`