[PATCH] D15616: [EarlyCSE] Simplify target intrinsic support

[Sanjin Sijaric via llvm-commits]

ssijaric added a comment.

Hi Philip,

The intrinsics that were initially commoned in the original patch were AArch64 intrinsics that we generate with Polly.  These were strided loads and stores, for which there was no equivalent LLVM IR at the time.  The reason for commoning them them in EarlyCSE instead of GVN was that EarlyCSE was able to common the scalar version - it was mostly to keep it consistent that way.  We still have one case for which we need GVN to common intrinsics.

Can we incorporate the LoadSite and StoreSite changes that you suggested, after this patch is merged?  These are under http://reviews.llvm.org/D8313.

Thanks,
Sanjin


http://reviews.llvm.org/D15616