[PATCH] D82553: [ARM] Allow rounding intrinsics to be tail predicated

[Sam Tebbs via Phabricator via llvm-commits]

samtebbs marked 3 inline comments as done.
samtebbs added inline comments.


================
Comment at: llvm/test/CodeGen/Thumb2/LowOverheadLoops/tail-pred-intrinsic-round.ll:259
+
+attributes #0 = { nofree nounwind "correctly-rounded-divide-sqrt-fp-math"="false" "disable-tail-calls"="false" "frame-pointer"="all" "less-precise-fpmad"="false" "min-legal-vector-width"="0" "no-infs-fp-math"="true" "no-jump-tables"="false" "no-nans-fp-math"="true" "no-signed-zeros-fp-math"="true" "no-trapping-math"="true" "stack-protector-buffer-size"="8" "target-cpu"="cortex-m55" "target-features"="+armv8.1-m.main,+dsp,+fp-armv8d16,+fp-armv8d16sp,+fp16,+fp64,+fullfp16,+hwdiv,+lob,+mve,+mve.fp,+ras,+strict-align,+thumb-mode,+vfp2,+vfp2sp,+vfp3d16,+vfp3d16sp,+vfp4d16,+vfp4d16sp,-aes,-bf16,-cdecp0,-cdecp1,-cdecp2,-cdecp3,-cdecp4,-cdecp5,-cdecp6,-cdecp7,-crc,-crypto,-dotprod,-fp16fml,-hwdiv-arm,-i8mm,-sb,-sha2" "unsafe-fp-math"="true" "use-soft-float"="false" }
+attributes #1 = { nosync nounwind readnone willreturn }
----------------
dmgreen wrote:
> I think you can remove all this and the code below. It complains about stuff then use -mattr=+mve.fp in the run line.
> 
> You will probably have to remove the !tbaa info too.
I did try removing them before but ran into an abort, which https://reviews.llvm.org/D82292 has since fixed.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82553/new/

https://reviews.llvm.org/D82553