[Lldb-commits] [lldb] [llvm] [polly] [ADT] Deprecate a soft-deprecated APInt constructor (PR #166314)

[Jakub Kuderski via lldb-commits]

================
@@ -154,6 +154,7 @@ class [[nodiscard]] APInt {
   /// Once all uses of this constructor are migrated to other constructors,
   /// consider marking this overload ""= delete" to prevent calls from being
   /// incorrectly bound to the APInt(unsigned, uint64_t, bool) constructor.
+  LLVM_DEPRECATED("Use other constructors of APInt", "APInt")
----------------
kuhar wrote:

I don't think this fixit is actionable because it doesn't produce working code when applied... I think this should be plain `[[deprecated("msg")]]`

https://github.com/llvm/llvm-project/pull/166314