[PATCH] D116830: Initialize ExtAddrMode::Scale
Kevin Athey via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 7 14:36:54 PST 2022
kda accepted this revision.
kda added a comment.
This revision is now accepted and ready to land.
I wonder if this should be placed in the struct as a default initializer. (TargetInstrInfo.h)
Short term, I think you work is right and matches the spirit of existing code.
But it seems like a vulnerability if anyone else were to use the struct and not initialize all members.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116830/new/
https://reviews.llvm.org/D116830
More information about the llvm-commits
mailing list