[clang] No longer add enumeration constants to the wrong scope (PR #134998)

[Erich Keane via cfe-commits]

================
@@ -5585,6 +5585,18 @@ SourceRange EnumConstantDecl::getSourceRange() const {
   return SourceRange(getLocation(), End);
 }
 
+bool EnumConstantDecl::isOutOfLine() const {
+  if (Decl::isOutOfLine())
+    return true;
+
+  // In C++, if the enumeration is out of line, the enumeration constants are
+  // also out of line.
+  if (getLangOpts().CPlusPlus)
+    return cast<Decl>(getDeclContext())->isOutOfLine();
----------------
erichkeane wrote:

Curious; Can this EVER be anything but an `EnumDecl`? 

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