Add support for non-control-flow region-bearing ops in SecretnessAnalysis
AlexanderViand-Intel opened this issue · 0 comments
AlexanderViand-Intel commented
The fix for #839 conservatively marks region-bearing ops not handled by the analysis framework directly (i.e. no RegionBranchOpInterface
trait) as "unknown" secretness (unless there's a secret operand, in which case it's secret irrespective of regions). This is correct, but overly conservative, as we could set the secretness to false if none of the operations in the op's regions are "secret" (That's still somewhat conservative, but probably the best we can do without assuming something about the operation's semantics). Supporting this requires implementing visitNonControlFlowArguments
.