[PATCH] D45608: [CallSiteSplit] Make sure we remove nonnull if the parameter turns out to be a constant.

[Jun Bum Lim via Phabricator via llvm-commits]

junbuml added a comment.

Thanks for handling this. Just curious if you see any issue with the non-null attribute in a constant ?



================
Comment at: test/Transforms/CallSiteSplitting/callsite-split.ll:44
+
+define void @caller2(i1 %c, %struct.bitmap* %a_elt, %struct.bitmap* %b_elt) {
+entry:
----------------
Can you move this in callsite-split-or-phi.ll where we run only callsite-splitting pass. Looks like you can also use just like i32* instead of %struct.bitmap*.


Repository:
  rL LLVM

https://reviews.llvm.org/D45608