[PATCH] D132221: [Assignment Tracking][2/*] Add flags to enable Assignment Tracking

[Orlando Cazalet-Hyams via Phabricator via llvm-commits]

Orlando added a comment.

In D132221#3758381 <https://reviews.llvm.org/D132221#3758381>, @jmorse wrote:

> I get the feeling that this is creating two places to store the flag rather than one -- there's the `EnableAssignmentTracking` flag which ends up in TargetOptions I think(?), I believe   `MarshallingInfoFlag<CodeGenOpts<"EnableAssignmentTracking">>;` causes the Xclang flag to be stored there.

I thought that it was putting it in `CodeGenOptions`, meaning we can check the value of the flag here in Clang - used in D132226 <https://reviews.llvm.org/D132226> (the `if` condition).

I'm not trying to push back on any suggested changes, I am just slightly lost on this side of LLVM. It could well be that storing in `CodeGenOptions` is undesirable too, but I thought it was worth checking.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132221/new/

https://reviews.llvm.org/D132221