Is it a good idea to mark class as deprecated?

Hi All,

  During a discussion about instruction scheduling in LLVM [1], I
realize that class PostRASchedulerList
actually is deprecated. The documentation of instruction scheduling is
pretty poor, so I cannot find such
information. Removing the obsoleted class might be too aggressive. I
am wondering if we can mark
the class as deprecated [2] (seems C++14 feature required), or at
least making a note on [3].

  Thoughts? :slight_smile:

[1] http://lists.llvm.org/pipermail/llvm-dev/2017-April/112358.html
[2] Marking as deprecated in C++14 — Joseph Mansfield
[3] http://llvm.org/docs/CodeGenerator.html#selectiondag-scheduling-and-formation-phase

Not sure about that class specifically, but we have
LLVM_ATTRIBUTE_DEPRECATED in Support/Compiler.h. However I'm not sure it
will work for class types.

- Michael Spencer

Not sure about that class specifically, but we have
LLVM_ATTRIBUTE_DEPRECATED in Support/Compiler.h. However I'm not sure it
will work for class types.

​A quick `grep -r "LLVM_ATTRIBUTE_DEPRECATED"` shows no one use it. ​Well,
at least a web page
(doxygen maybe?) saying the class is deprecated would be helpful.

Regards,
chenwj