[libcxx-commits] [PATCH] D111947: [Demangle] Rename OutputStream to OutputString

Mon Oct 18 10:37:05 PDT 2021

dblaikie added inline comments.

================
Comment at: llvm/include/llvm/Demangle/ItaniumDemangle.h:15
-#ifndef LLVM_DEMANGLE_ITANIUMDEMANGLE_H
-#define LLVM_DEMANGLE_ITANIUMDEMANGLE_H

----------------
ljmf00 wrote:
> Geod24 wrote:
> > Why those changes ? IIUC the CI is failing on this file, too.
> This is because the script was verbatim copied. I think LLVM should address this in a more clean way, to be honest. These specific files are literally copied from one place to another and stuff like this can happen without being noticed. @dblaikie Currently, there is an effort to change this or any specific rationale to keep it this way?
Not quite following - but no, I don't think there's any effort at the moment to change this situation that these two files need to be kept in sync.

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

https://reviews.llvm.org/D111947