[llvm] [GOFF] Refactor writing GOFF records (PR #93855)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 20 02:11:39 PDT 2024


================
@@ -0,0 +1,7 @@
+# RUN: not yaml2obj %s 2>&1 | FileCheck %s
+
+# CHECK: yaml2obj: error: conversion to EBCDIC 1047 failed on 'Euro symbol €'
+
+--- !GOFF
+- End:
+    EntryName: "Euro symbol €"
----------------
jh7370 wrote:

I'm sure it's fine, but non-ascii characters in input files make me nervous because of issues with different default encodings with different programs. If I were to open this file in e.g. notepad on Windows, I'd expect the € character to not be interpreted correctly.

That being said, since the pre-merge check passes, this is probably fine for now (we can tackle any issues in the future).

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


More information about the llvm-commits mailing list