[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