[PATCH] D119482: [OpaquePtr][LLParser] Automatically detect opaque pointers in .ll files

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/include/llvm/IR/LLVMContext.h:309
-  /// pointer type.
-  void enableOpaquePointers() const;
-
----------------
nikic wrote:
> Why drop this? This is needed for per-frontend enablement, which we'll still need, right?
I was under the impression that frontends would just pass `-mllvm -opaque-pointers` (or whatever the frontend equivalent), since it doesn't seem necessary to support multiple LLVMContexts with different opaque pointers flags. But either way I've restored it in this change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119482