[PATCH] D127835: [SCEV] recognize llvm.annotation intrinsic

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/docs/LangRef.rst:23580
+annotations on a best-effort basis but are allowed to replace them with their
+first argument without breaking semantics and are completely dropped during
+instruction selection.
----------------
"and they are"?


================
Comment at: llvm/lib/Analysis/ScalarEvolution.cpp:7628
+      case Intrinsic::annotation:
+      case Intrinsic::ptr_annotation:
+        // A start_loop_iterations or llvm.annotation or llvm.prt.annotation is
----------------
Please also add these to getOperandsToCreate() (probably after a rebase).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127835