Leave some FIXME comments

dianqk · dianqk · commit 1da22cd2fb1f · 2024-07-11T20:47:15.000+08:00
diff --git a/llvm/test/Transforms/MemCpyOpt/memcpy-memcpy-offset.ll b/llvm/test/Transforms/MemCpyOpt/memcpy-memcpy-offset.ll
@@ -58,6 +58,7 @@ define void @forward_offset_align_2(ptr %src, ptr %dest) {
 }
 
 ; If the copy destination can be used as the copy source, we don't need to create a GEP instruction.
+; FIXME: We can directly remove memcpy here.
 define void @forward_offset_without_gep(ptr %src) {
 ; CHECK-LABEL: define void @forward_offset_without_gep(
 ; CHECK-SAME: ptr [[SRC:%.*]]) {
diff --git a/llvm/test/Transforms/PhaseOrdering/memcpy-offset.ll b/llvm/test/Transforms/PhaseOrdering/memcpy-offset.ll
@@ -4,6 +4,7 @@
 
 %buf = type [7 x i8]
 
+; FIXME: This can be done independently in memcpyopt.
 ; Check that we eliminate all `memcpy` calls in this function.
 define void @forward_offset_and_store(ptr %dep_src) {
 ; CUSTOM-LABEL: define void @forward_offset_and_store(

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,7 @@ define void @forward_offset_align_2(ptr %src, ptr %dest) {`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`; If the copy destination can be used as the copy source, we don't need to create a GEP instruction.`
	`61`	`+; FIXME: We can directly remove memcpy here.`
`61`	`62`	`define void @forward_offset_without_gep(ptr %src) {`
`62`	`63`	`; CHECK-LABEL: define void @forward_offset_without_gep(`
`63`	`64`	`; CHECK-SAME: ptr [[SRC:%.*]]) {`