[PATCH] D120129: [NVPTX] Enhance vectorization of ld.param & st.param

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Mar 27 19:46:23 PDT 2022


jdoerfert requested changes to this revision.
jdoerfert added a comment.
This revision now requires changes to proceed.

Please revert this commit. It breaks any code using the cuda.11.0.2 libdevice.bc file, the source of the `null` `nvvm.annotations` and the annotations with 5 arguments. See below.
Once a fix is in, and a test is added, we can reapply this.

  $ opt -S .../cuda-11.0.2/nvvm/libdevice/libdevice.10.bc | tail
  !nvvm.annotations = !{!1, !2, !1, !3, !3, !3, !3, !4, !4, !3}
  
  !0 = !{i32 1, i32 4}
  !1 = !{null, !"align", i32 8}
  !2 = !{null, !"align", i32 8, !"align", i32 65544, !"align", i32 131080}
  !3 = !{null, !"align", i32 16}
  !4 = !{null, !"align", i32 16, !"align", i32 65552, !"align", i32 131088}
  !5 = distinct !{!5, !6}jdoerfert at jlselogin6 ~ ❯❯❯ opt -S /soft/compilers/cuda/cuda-11.0.2/nvvm/libdevice/libdevice.10.bc | tail
  !nvvm.annotations = !{!1, !2, !1, !3, !3, !3, !3, !4, !4, !3}
  
  !0 = !{i32 1, i32 4}
  !1 = !{null, !"align", i32 8}
  !2 = !{null, !"align", i32 8, !"align", i32 65544, !"align", i32 131080}
  !3 = !{null, !"align", i32 16}
  !4 = !{null, !"align", i32 16, !"align", i32 65552, !"align", i32 131088}
  !5 = distinct !{!5, !6}
  !6 = !{!"llvm.loop.unroll.count", i32 1}
  !7 = distinct !{!7, !6}


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120129



More information about the cfe-commits mailing list