[PATCH] D138496: [OMPIRBuilder] Add OpenMPDefaultSimdAlignment field to TargetMachine class

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp:3074
+  static std::string PreviousFeaturesString;
+  static std::unique_ptr<TargetMachine> TgtInfo;
+  const Target *TheTarget;
----------------
No static variables here, please.
These should be members of the class or a helper class.
Should we have a map of these to the respective TargetMachine?


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

https://reviews.llvm.org/D138496