[PATCH] D61489: RegAlloc: try to fail more gracefully when out of registers

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/CodeGen/RegAllocBase.cpp:129
+                ? "inline assembly requires more registers than available"
+                : "ran out of registers during register allocation");
+      } else {
----------------
qcolombet wrote:
> Although I find the patch sensible, the new behavior is somewhat misleading:
> If MI is not an inline asm statement, we are still going to report a source location (if any), whereas unlike inline asm statement, there is little to nothing that the user can do with that information. It could actually be counter productive to give them that location.
It should probably report the source location for the function, rather than the instruction in that case


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61489