Skip to content

Commit bb5c510

Browse files
committed
Address reviewer feedback:
- Change assertion to comment
1 parent 5ca8bbd commit bb5c510

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

llvm/lib/Target/AMDGPU/Disassembler/AMDGPUDisassembler.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2366,10 +2366,9 @@ const MCExpr *AMDGPUDisassembler::createConstantSymbolExpr(StringRef Id,
23662366
int64_t Val) {
23672367
MCContext &Ctx = getContext();
23682368
MCSymbol *Sym = Ctx.getOrCreateSymbol(Id);
2369-
int64_t Res = ~Val;
2370-
assert(!Sym->isVariable() ||
2371-
(Sym->getVariableValue()->evaluateAsAbsolute(Res) && Res == Val));
2372-
(void)Res;
2369+
// Note: only set value to Val on a new symbol.
2370+
// Existing symbol may potentially have a different value to the one
2371+
// requested, which allows for user redefinition of symbols.
23732372
if (!Sym->isVariable())
23742373
Sym->setVariableValue(MCConstantExpr::create(Val, Ctx));
23752374
return MCSymbolRefExpr::create(Sym, Ctx);

0 commit comments

Comments
 (0)