[PATCH] D109534: fixed ambiguous overload build failure

Jeremy Morse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 10 02:20:04 PDT 2021


jmorse added a comment.

For the VarLocBasedImpl.cpp change, I'd recommend explicitly constructing a Register from the RegNo field -- it'll achieve exactly the same thing, but avoids the cast and is a bit clearer to the reader.

I'm curious as to why this has cropped up in gcc now; however I'm not a language lawyer.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109534



More information about the llvm-commits mailing list