Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

X86 IntelAsmPrinter can't print jumptables in memrefs #1433

Closed
asl opened this issue Dec 19, 2006 · 2 comments
Closed

X86 IntelAsmPrinter can't print jumptables in memrefs #1433

asl opened this issue Dec 19, 2006 · 2 comments
Assignees
Labels
backend:X86 bugzilla Issues migrated from bugzilla compile-fail Use [accepts-invalid] and [rejects-valid] instead

Comments

@asl
Copy link
Collaborator

asl commented Dec 19, 2006

Bugzilla Link 1061
Resolution FIXED
Resolved on Feb 22, 2010 12:47
Version 1.9
OS Linux
Attachments Failed bitecode

Extended Description

$ ./llc -f test-j.bc -x86-asm-syntax=intel
llc: /home/asl/proj/llvm/src/include/llvm/CodeGen/MachineInstr.h:138: int64_t
llvm::MachineOperand::getImmedValue() const: Assertion `isImm() && "Wrong
MachineOperand accessor"' failed.
./llc((anonymous namespace)::PrintStackTrace()+0x1f)[0x85a77cf]
/lib/libc.so.6(abort+0xeb)[0xb7cf6133]
/lib/libc.so.6(__assert_fail+0xeb)[0xb7cee4f3]
./llc(llvm::X86IntelAsmPrinter::printMemReference(llvm::MachineInstr const*,
unsigned int, char const*)+0x49c)[0x827468c]

Don't know MF machinery yet to investigate/fix :(

@asl
Copy link
Collaborator Author

asl commented Dec 19, 2006

assigned to @lattner

@lattner
Copy link
Collaborator

lattner commented Dec 19, 2006

Fixed. Testcase here: CodeGen/X86/2006-12-19-IntelSyntax.ll

Patch here:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20061218/041591.html

-Chris

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
clementval pushed a commit to clementval/llvm-project that referenced this issue Feb 4, 2022
…around for the (llvm#1433)

* Write a pass to annotate constant operands on FIR ops. This fixes the
bug in MLIR's canonicalizer that treats constants differently based on
how they are packaged and not their values and semantics.

* dyn_cast_or_null chang.
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend:X86 bugzilla Issues migrated from bugzilla compile-fail Use [accepts-invalid] and [rejects-valid] instead
Projects
None yet
Development

No branches or pull requests

2 participants