google/heir

Add support for non-control-flow region-bearing ops in SecretnessAnalysis

AlexanderViand-Intel opened this issue · 0 comments

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.