Many DebugInfo tests now pass with native builds.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Is this a WIP patch? Will you add some tests?
How about @wangleiat's opinion?
llvm/lib/Target/LoongArch/MCTargetDesc/LoongArchAsmBackend.cpp | ||
---|---|---|
80–81 | Can these 2 lines be moved before line-79? |
Comment Actions
The patch is ready for review.
This may not need a dedicated test case, because before the change most if not all DebugInfo tests fail on native LoongArch builds, and this change (together with the parent change) fixes them.
llvm/lib/Target/LoongArch/MCTargetDesc/LoongArchAsmBackend.cpp | ||
---|---|---|
80–81 | fixed |
Comment Actions
Thanks for the reminder, test cases will be added in the subsequent complete implementation of applyFixup.
Can these 2 lines be moved before line-79?