[PATCH] D143018: [DAGCombiner][NFC] Factor out ByteProvider

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/include/llvm/ADT/ByteProvider.h:23
+
+namespace {
+// TODO -- use constraint in c++20
----------------
Is there anyway that we can avoid an anonymous namespace in a header like this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143018