Patch r323922 changed the sigil for physical registers to '$', instead of '%'.
An error message was missed during this change, and reports the wrong sigil.
This patch corrects that diagnostic and the tests that check that error string.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
I did the sigil change and now I'm curious why I hadn't hit this a few months back. Do you know why?
Comment Actions
It looks like you simply missed the diagnostic, and not any functional part. I only discovered it because I was messing around with a .mir test and noticed the error message. I'm guessing your search/replace didn't cover the case of an isolated '$'