[PATCH] D32241: Don't process debug intrinsics in InstCombine

[Dmitry Mikulin via Phabricator via llvm-commits]

dmikulin added inline comments.


================
Comment at: test/Transforms/InstCombine/debuginfo-skip.ll:54-58
+attributes #0 = { nounwind ssp uwtable "correctly-rounded-divide-sqrt-fp-math"="false" "disable-tail-calls"="false" "less-precise-fpmad"="false" "no-frame-pointer-elim"="true" "no-frame-pointer-elim-non-leaf" "no-infs-fp-math"="false" "no-jump-tables"="false" "no-nans-fp-math"="false" "no-signed-zeros-fp-math"="false" "no-trapping-math"="false" "stack-protector-buffer-size"="8" "target-cpu"="core2" "target-features"="+cx16,+fxsr,+mmx,+sse,+sse2,+sse3,+ssse3,+x87" "unsafe-fp-math"="false" "use-soft-float"="false" }
+attributes #1 = { nounwind readnone }
+attributes #2 = { argmemonly nounwind }
+attributes #3 = { nounwind }
+
----------------
davide wrote:
> For example, I'm pretty sure all these attributes are unneeded.
This test case was generated from a small source. I wanted to make sure that both @llvm.dbg.decalre and @llvm.dbg.value are present in the IR. I removed the attributes and shortened some strings. Some debug info can probably be removed, but I didn't want to mess with the consistency of the IR to reduce the size.


https://reviews.llvm.org/D32241