[PATCH] D97311: [WIP][DebugInfo] Introduce DIConstant metadata for representing named constants

[Sourabh Singh Tomar via Phabricator via llvm-commits]

SouraVX added inline comments.


================
Comment at: llvm/test/Assembler/diconstant.ll:5
+; CHECK-DAG: ![[SUBPROGRAM_SCOPE:.*]] = distinct !DISubprogram
+; CHECK-DAG: !{{.*}} = !DIConstant(name: "local_constant", scope: ![[SUBPROGRAM_SCOPE]], file: !3, line: 10, type: !{{.*}}, isLocal: true, isDefinition: true, value: 168)
+; CHECK-DAG: !{{.*}} = !DIConstant(name: "module_constant", scope: ![[MODULE_SCOPE]], file: !3, line: 4, type: !{{.*}}, isLocal: false, isDefinition: true, value: 96)
----------------
You may notice un-related value (168), this could be due to in-correct signed and unsigned interpretation ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97311