[PATCH] D149464: [clang][dataflow] Expose DataflowAnalysisContext from DataflowEnvironment.

[Samira Bazuzi via Phabricator via cfe-commits]

bazuzi added inline comments.


================
Comment at: clang/include/clang/Analysis/FlowSensitive/DataflowEnvironment.h:189
 
+  /// Deprecated. Use *getDataflowAnalysisContext().getOptions().Log instead.
   Logger &logger() const { return *DACtx->getOptions().Log; }
----------------
xazax.hun wrote:
> Any reason for a comment as opposed to the deprecated attribute? 
I couldn't tell exactly what the conventions were or whether all of the chained replacements could be used as the FIX very well.

If you can confirm that `LLVM_DEPRECATED("DataflowAnalysisContext is now directly exposed.", "*getDataflowAnalysisContext().getOptions().Log")`, 
`LLVM_DEPRECATED(..., "getDataflowAnalysisContext().arena")`, etc. will provide useful fixes, happy to replace with those. Or with replacements just in the MSG and "" for FIX otherwise.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149464