[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